diff options
author | Russ Cox <rsc@golang.org> | 2014-06-11 20:42:13 -0400 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2014-06-11 20:42:13 -0400 |
commit | 6fe3894300c40dfe3c28cb7d86db8f2b3fa301cd (patch) | |
tree | 6812117ce1168a8fe8435db129ecbb134612af40 | |
parent | 77cfef57c6ad486494942922c21dae450018855d (diff) | |
download | go-6fe3894300c40dfe3c28cb7d86db8f2b3fa301cd.tar.gz |
[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://codereview.appspot.com/101800043
This change just adds a test.
Fixes issue 8047
LGTM=dvyukov, rsc
R=dvyukov, rsc
CC=golang-codereviews
https://codereview.appspot.com/108840043
Committer: Russ Cox <rsc@golang.org>
???
TBR=adg
CC=golang-codereviews
https://codereview.appspot.com/102320043
-rw-r--r-- | test/fixedbugs/issue8047.go | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/test/fixedbugs/issue8047.go b/test/fixedbugs/issue8047.go new file mode 100644 index 000000000..fe7ada5c0 --- /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) +} |