diff options
author | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-04-28 13:18:25 +0000 |
---|---|---|
committer | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-04-28 13:18:25 +0000 |
commit | 109be507377fe7f6e8819ac94041d3fdcdf6fd2f (patch) | |
tree | 2806a689f8fab4a2ec9fc949830ef270a91d667d /test/fixedbugs/issue28797.go | |
parent | Initial commit. (diff) | |
download | golang-1.19-109be507377fe7f6e8819ac94041d3fdcdf6fd2f.tar.xz golang-1.19-109be507377fe7f6e8819ac94041d3fdcdf6fd2f.zip |
Adding upstream version 1.19.8.upstream/1.19.8upstream
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'test/fixedbugs/issue28797.go')
-rw-r--r-- | test/fixedbugs/issue28797.go | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/test/fixedbugs/issue28797.go b/test/fixedbugs/issue28797.go new file mode 100644 index 0000000..480c105 --- /dev/null +++ b/test/fixedbugs/issue28797.go @@ -0,0 +1,53 @@ +// run + +// Copyright 2018 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 ( + "fmt" +) + +// test expects f to panic, but not to run out of memory, +// which is a non-panic fatal error. OOM results from failure +// to properly check negative limit. +func test(f func()) { + defer func() { + r := recover() + if r == nil { + panic("panic wasn't recoverable") + } + }() + f() +} + +//go:noinline +func id(x int) int { + return x +} + +func main() { + test(foo) + test(bar) +} + +func foo() { + b := make([]byte, 0) + b = append(b, 1) + id(len(b)) + id(len(b) - 2) + s := string(b[1 : len(b)-2]) + fmt.Println(s) +} + +func bar() { + b := make([]byte, 1) + b = append(b, 1) + i := id(-1) + if i < len(b) { // establish value is not too large. + s := string(b[1:i]) // should check for negative also. + fmt.Println(s) + } +} |