summaryrefslogtreecommitdiffstats
path: root/common/collections/append.go
diff options
context:
space:
mode:
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2023-06-14 09:44:18 +0200
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2023-06-14 20:18:54 +0200
commitf73c5675341d41cb41b886b089544701022b5c12 (patch)
treea0815d89c447b9941f508ab3de3f78787499a6bc /common/collections/append.go
parentd178fe94fe26ce2cf8096f330f3d160af3ec818d (diff)
common/collections: Always make a copy of the input slice in Append
Fixes #10458.
Diffstat (limited to 'common/collections/append.go')
-rw-r--r--common/collections/append.go7
1 files changed, 7 insertions, 0 deletions
diff --git a/common/collections/append.go b/common/collections/append.go
index fe8792fc4..91abc46d3 100644
--- a/common/collections/append.go
+++ b/common/collections/append.go
@@ -31,6 +31,13 @@ func Append(to any, from ...any) (any, error) {
var tot reflect.Type
if !toIsNil {
+ if tov.Kind() == reflect.Slice {
+ // Create a copy of tov, so we don't modify the original.
+ c := reflect.MakeSlice(tov.Type(), tov.Len(), tov.Len()+len(from))
+ reflect.Copy(c, tov)
+ tov = c
+ }
+
if tov.Kind() != reflect.Slice {
return nil, fmt.Errorf("expected a slice, got %T", to)
}