From ff7cb872a455a370beed530294838ab151e965c6 Mon Sep 17 00:00:00 2001 From: Andrew Gerrand Date: Wed, 5 Jun 2013 10:52:00 +1000 Subject: [release-branch.go1.1] runtime: zeroize g->fnstart to not prevent GC of the closure MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ««« CL 9557043 / 2c128d417029 runtime: zeroize g->fnstart to not prevent GC of the closure Fixes #5493. R=golang-dev, minux.ma, iant CC=golang-dev https://golang.org/cl/9557043 »»» R=dvyukov, iant, minux.ma, bradfitz, dave CC=golang-dev https://golang.org/cl/10031043 --- src/pkg/runtime/proc.c | 1 + test/fixedbugs/issue5493.go | 52 +++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 53 insertions(+) create mode 100644 test/fixedbugs/issue5493.go diff --git a/src/pkg/runtime/proc.c b/src/pkg/runtime/proc.c index 018a453d62..31876b62a9 100644 --- a/src/pkg/runtime/proc.c +++ b/src/pkg/runtime/proc.c @@ -1232,6 +1232,7 @@ static void goexit0(G *gp) { gp->status = Gdead; + gp->fnstart = nil; gp->m = nil; gp->lockedm = nil; m->curg = nil; diff --git a/test/fixedbugs/issue5493.go b/test/fixedbugs/issue5493.go new file mode 100644 index 0000000000..fe571bc085 --- /dev/null +++ b/test/fixedbugs/issue5493.go @@ -0,0 +1,52 @@ +// run + +// Copyright 2013 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +package main + +import ( + "runtime" + "sync" + "sync/atomic" + "time" +) + +const N = 10 +var count int64 + +func run() error { + f1 := func() {} + f2 := func() { + func() { + f1() + }() + } + runtime.SetFinalizer(&f1, func(f *func()) { + atomic.AddInt64(&count, -1) + }) + go f2() + return nil +} + +func main() { + count = N + var wg sync.WaitGroup + wg.Add(N) + for i := 0; i < N; i++ { + go func() { + run() + wg.Done() + }() + } + wg.Wait() + for i := 0; i < 2*N; i++ { + time.Sleep(10 * time.Millisecond) + runtime.GC() + } + if count != 0 { + panic("not all finalizers are called") + } +} + -- cgit v1.2.3-54-g00ecf