From: Russ Cox Date: Thu, 12 Jun 2014 00:42:13 +0000 (-0400) Subject: [release-branch.go1.3] runtime: add test for issue 8047. X-Git-Tag: go1.3rc2~7 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=e0d5179635a8749d194e19d29771d294812ad318;p=gostls13.git [release-branch.go1.3] runtime: add test for issue 8047. ««« CL 108840043 / 3a2306461574 runtime: add test for issue 8047. Make sure stack copier doesn't barf on a nil defer. Bug was fixed in https://golang.org/cl/101800043 This change just adds a test. Fixes #8047 LGTM=dvyukov, rsc R=dvyukov, rsc CC=golang-codereviews https://golang.org/cl/108840043 »»» TBR=adg CC=golang-codereviews https://golang.org/cl/102320043 --- diff --git a/test/fixedbugs/issue8047.go b/test/fixedbugs/issue8047.go new file mode 100644 index 0000000000..fe7ada5c0b --- /dev/null +++ b/test/fixedbugs/issue8047.go @@ -0,0 +1,29 @@ +// run + +// Copyright 2014 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. + +// Issue 8047. Stack copier shouldn't crash if there +// is a nil defer. + +package main + +func stackit(n int) { + if n == 0 { + return + } + stackit(n - 1) +} + +func main() { + defer func() { + // catch & ignore panic from nil defer below + err := recover() + if err == nil { + panic("defer of nil func didn't panic") + } + }() + defer ((func())(nil))() + stackit(1000) +}