upstream/redis
1
0
Fork 0
mirror of https://github.com/redis/redis.git synced 2026-05-28 04:02:46 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fixed: issue from review

Browse source
This commit is contained in:
Sergey Georgiev 2026-05-14 15:55:47 +03:00
parent 0b9e7ca6b1
commit 3d8beec04c
1 changed files with 0 additions and 30 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
tests/integration/corrupt-dump.tcl
View file

@ -1070,18 +1070,6 @@ test {corrupt payload: stream live entry count integer overflow bypasses length
}
}
test {corrupt payload: stream all-tombstone entries with non-zero length} {
start_server [list overrides [list loglevel verbose use-exit-on-panic yes crash-memcheck-enabled no]] {
r debug set-skip-checksum-validation 1
# Payload: stream where every entry is a tombstone
# s->length claims 1 live entry.
catch {r RESTORE mystream 0 "\x1A\x01\x10\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x1D\x1D\x00\x00\x00\x0A\x00\x00\x01\x01\x01\x01\x01\x81\x6B\x02\x00\x01\x03\x01\x00\x01\x00\x01\x81\x76\x02\x04\x01\xFF\x01\x01\x00\x01\x00\x00\x00\x01\x00\x40\x64\x40\x64\x00\x00\x00\x0D\x00\xBD\x89\x4D\xF3\x41\xC5\xE0\x8E" REPLACE} err
catch {r XREAD COUNT 1 STREAMS mystream $} _
assert_match "*Bad data format*" $err
r ping
}
}
test {corrupt payload: stream last_id smaller than actual tail entry} {
start_server [list overrides [list loglevel verbose use-exit-on-panic yes crash-memcheck-enabled no]] {
r config set sanitize-dump-payload yes
@ -1192,23 +1180,5 @@ test {corrupt payload: stream listpacks in non-ascending master order} {
}
}
test {corrupt payload: stream length inconsistent with summed live entries} {
start_server [list overrides [list loglevel verbose use-exit-on-panic yes crash-memcheck-enabled no]] {
r config set sanitize-dump-payload yes
r debug set-skip-checksum-validation 1
# Payload: one listpack with a single live entry at id 1-0
# (lp_live=1 in the listpack header, so the per-listpack
# lp_live>0 check passes), but the trailer declares length=2
# and entries_added=2. The summed live-entry count (1) does
# not match s->length (2), and entries_added>=length so the
# entries_added<length check cannot fire first. Only the
# post-loop "length inconsistent with live entries"
# cross-check rejects this.
catch {r RESTORE mystream 0 "\x1A\x01\x10\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x1D\x1D\x00\x00\x00\x0A\x00\x01\x01\x00\x01\x01\x01\x81\x6B\x02\x00\x01\x02\x01\x00\x01\x00\x01\x81\x76\x02\x04\x01\xFF\x02\x01\x00\x01\x00\x00\x00\x02\x00\x40\x64\x40\x64\x00\x00\x00\x0D\x00\xBD\x89\x4D\xF3\x41\xC5\xE0\x8E" REPLACE} err
assert_match "*Bad data format*" $err
r ping
}
}
} ;# tags