fix: copy entries slice before in-place reversal in ZRANGE

bootjp · bootjp · commit c27efd5a5b38 · 2026-04-12T23:04:57.000+09:00
The legacy blob path returns the unmarshalled struct Entries slice
directly. Reversing it in-place could mutate shared backing memory.
Clone the slice before calling reverseZSetEntries.
diff --git a/adapter/redis_compat_commands.go b/adapter/redis_compat_commands.go
@@ -1629,6 +1629,7 @@ func (r *RedisServer) zrangeRead(conn redcon.Conn, key []byte, start, stop int,
 		return
 	}
 	if opts.reverse {
+		entries = append([]redisZSetEntry(nil), entries...)
 		reverseZSetEntries(entries)
 	}
 	s, e := normalizeRankRange(start, stop, len(entries))

Original file line number	Diff line number	Diff line change
`@@ -1629,6 +1629,7 @@ func (r *RedisServer) zrangeRead(conn redcon.Conn, key []byte, start, stop int,`
`1629`	`1629`	`return`
`1630`	`1630`	`}`
`1631`	`1631`	`if opts.reverse {`
	`1632`	`+ entries = append([]redisZSetEntry(nil), entries...)`
`1632`	`1633`	`reverseZSetEntries(entries)`
`1633`	`1634`	`}`
`1634`	`1635`	`s, e := normalizeRankRange(start, stop, len(entries))`