diff options
author | Ondřej Surý <ondrej@sury.org> | 2011-09-13 13:13:40 +0200 |
---|---|---|
committer | Ondřej Surý <ondrej@sury.org> | 2011-09-13 13:13:40 +0200 |
commit | 5ff4c17907d5b19510a62e08fd8d3b11e62b431d (patch) | |
tree | c0650497e988f47be9c6f2324fa692a52dea82e1 /test/fixedbugs/bug303.go | |
parent | 80f18fc933cf3f3e829c5455a1023d69f7b86e52 (diff) | |
download | golang-upstream/60.tar.gz |
Imported Upstream version 60upstream/60
Diffstat (limited to 'test/fixedbugs/bug303.go')
-rw-r--r-- | test/fixedbugs/bug303.go | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/test/fixedbugs/bug303.go b/test/fixedbugs/bug303.go new file mode 100644 index 000000000..3bd790f13 --- /dev/null +++ b/test/fixedbugs/bug303.go @@ -0,0 +1,37 @@ +// $G $D/$F.go && $L $F.$A && ./$A.out + +// Copyright 2010 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 1011. Removing either #1 or #3 avoided the crash at #2. + +package main + +import ( + "io" + "strings" +) + +func readU16BE(b []byte) uint16 { + b[0] = 0 + b[1] = 1 + return uint16(b[0])<<8 + uint16(b[1]) // #1 + n := uint16(b[0])<<8 + uint16(b[1]) + return n +} + +func readStr(r io.Reader, b []byte) string { + n := readU16BE(b) + if int(n) > len(b) { + return "err: n>b" + } + io.ReadFull(r, b[0:n]) // #2 + return string(b[0:n]) // #3 + return "ok" +} + +func main() { + br := strings.NewReader("abcd") + readStr(br, make([]byte, 256)) +} |