Skip to content

Commit aa04caa

Browse files
randall77rsc
authored andcommitted
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
1 parent 3d68dc3 commit aa04caa

File tree

1 file changed

+29
-0
lines changed

1 file changed

+29
-0
lines changed

test/fixedbugs/issue8047.go

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
// run
2+
3+
// Copyright 2014 The Go Authors. All rights reserved.
4+
// Use of this source code is governed by a BSD-style
5+
// license that can be found in the LICENSE file.
6+
7+
// Issue 8047. Stack copier shouldn't crash if there
8+
// is a nil defer.
9+
10+
package main
11+
12+
func stackit(n int) {
13+
if n == 0 {
14+
return
15+
}
16+
stackit(n - 1)
17+
}
18+
19+
func main() {
20+
defer func() {
21+
// catch & ignore panic from nil defer below
22+
err := recover()
23+
if err == nil {
24+
panic("defer of nil func didn't panic")
25+
}
26+
}()
27+
defer ((func())(nil))()
28+
stackit(1000)
29+
}

0 commit comments

Comments
 (0)