mirror of
https://github.com/golang/go
synced 2024-09-04 23:44:16 +00:00
![Keith Randall](/assets/img/avatar_default.png)
staticcopy of a struct or array should recursively call itself, not staticassign. This fixes an issue where a struct with a slice in it is copied during static initialization. In this case, the backing array for the slice is duplicated, and each copy of the slice refers to a different backing array, which is incorrect. That issue has existed since at least Go 1.2. I'm not sure why this was never noticed. It seems like a pretty obvious bug if anyone modifies the resulting slice. In any case, we started to notice when the optimization in CL 140301 landed. Here is basically what happens in issue29013b.go: 1) The error above happens, so we get two backing stores for what should be the same slice. 2) The code for initializing those backing stores is reused. But not duplicated: they are the same Node structure. 3) The order pass allocates temporaries for the map operations. For the first instance, things work fine and two temporaries are allocated and stored in the OKEY nodes. For the second instance, the order pass decides new temporaries aren't needed, because the OKEY nodes already have temporaries in them. But the order pass also puts a VARKILL of the temporaries between the two instance initializations. 4) In this state, the code is technically incorrect. But before CL 140301 it happens to work because the temporaries are still correctly initialized when they are used for the second time. But then... 5) The new CL 140301 sees the VARKILLs and decides to reuse the temporary for instance 1 map 2 to initialize the instance 2 map 1 map. Because the keys aren't re-initialized, instance 2 map 1 gets the wrong key inserted into it. Fixes #29013 Change-Id: I840ce1b297d119caa706acd90e1517a5e47e9848 Reviewed-on: https://go-review.googlesource.com/c/152081 Run-TryBot: Keith Randall <khr@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Josh Bleecher Snyder <josharian@gmail.com>
44 lines
616 B
Go
44 lines
616 B
Go
// 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
|
|
|
|
type TestSuite struct {
|
|
Tests []Test
|
|
}
|
|
type Test struct {
|
|
Want interface{}
|
|
}
|
|
type Int struct {
|
|
i int
|
|
}
|
|
|
|
func NewInt(v int) Int {
|
|
return Int{i: v}
|
|
}
|
|
|
|
var Suites = []TestSuite{
|
|
Dicts,
|
|
}
|
|
var Dicts = TestSuite{
|
|
Tests: []Test{
|
|
{
|
|
Want: map[Int]bool{NewInt(1): true},
|
|
},
|
|
{
|
|
Want: map[Int]string{
|
|
NewInt(3): "3",
|
|
},
|
|
},
|
|
},
|
|
}
|
|
|
|
func main() {
|
|
if Suites[0].Tests[0].Want.(map[Int]bool)[NewInt(3)] {
|
|
panic("bad")
|
|
}
|
|
}
|