From e938de22be4ee63d65152024f75d62482d3d69b4 Mon Sep 17 00:00:00 2001 From: Shenghou Ma Date: Wed, 11 Mar 2015 12:07:50 -0400 Subject: [PATCH] [release-branch.go1.4] runtime: don't return a slice with nil ptr but non-zero len from growslice Fixes #10135. Change-Id: Ic4c5ab15bcb7b9c3fcc685a788d3b59c60c26e1e Signed-off-by: Shenghou Ma Reviewed-on: https://go-review.googlesource.com/7400 Reviewed-by: Ian Lance Taylor Reviewed-on: https://go-review.googlesource.com/14248 Reviewed-by: Andrew Gerrand --- src/runtime/slice.go | 4 +++- test/fixedbugs/issue10135.go | 25 +++++++++++++++++++++++++ 2 files changed, 28 insertions(+), 1 deletion(-) create mode 100644 test/fixedbugs/issue10135.go diff --git a/src/runtime/slice.go b/src/runtime/slice.go index 171087d7f6..f457f41429 100644 --- a/src/runtime/slice.go +++ b/src/runtime/slice.go @@ -53,7 +53,9 @@ func growslice(t *slicetype, old sliceStruct, n int64) sliceStruct { et := t.elem if et.size == 0 { - return sliceStruct{old.array, old.len, cap} + // append should not create a slice with nil pointer but non-zero len. + // We assume that append doesn't need to preserve old.array in this case. + return sliceStruct{unsafe.Pointer(&zerobase), old.len, cap} } newcap := old.cap diff --git a/test/fixedbugs/issue10135.go b/test/fixedbugs/issue10135.go new file mode 100644 index 0000000000..9985e5a0ee --- /dev/null +++ b/test/fixedbugs/issue10135.go @@ -0,0 +1,25 @@ +// run + +// Copyright 2015 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 10135: append a slice with zero-sized element used +// to always return a slice with the same data pointer as the +// old slice, even if it's nil, so this program used to panic +// with nil pointer dereference because after append, s is a +// slice with nil data pointer but non-zero len and cap. + +package main + +type empty struct{} + +func main() { + var s []empty + + s = append(s, empty{}) + + for _, v := range s { + _ = v + } +}