Fix missing check for sanitize_dump in corrupt-dump-fuzzer test (#9285)
this means the assertion that checks that when deep sanitization is enabled, there are no crashes, was missing. (cherry picked from commit 3db0f1a284e4fba703419b892b2d5b8d385afc06)
This commit is contained in:
parent
0708720d3c
commit
e8874ee387
@ -117,7 +117,7 @@ foreach sanitize_dump {no yes} {
|
|||||||
set report_and_restart true
|
set report_and_restart true
|
||||||
incr stat_terminated_in_restore
|
incr stat_terminated_in_restore
|
||||||
write_log_line 0 "corrupt payload: $printable_dump"
|
write_log_line 0 "corrupt payload: $printable_dump"
|
||||||
if {$sanitize_dump == 1} {
|
if {$sanitize_dump == yes} {
|
||||||
puts "Server crashed in RESTORE with payload: $printable_dump"
|
puts "Server crashed in RESTORE with payload: $printable_dump"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -140,7 +140,7 @@ foreach sanitize_dump {no yes} {
|
|||||||
set by_signal [count_log_message 0 "crashed by signal"]
|
set by_signal [count_log_message 0 "crashed by signal"]
|
||||||
incr stat_terminated_by_signal $by_signal
|
incr stat_terminated_by_signal $by_signal
|
||||||
|
|
||||||
if {$by_signal != 0 || $sanitize_dump == 1 } {
|
if {$by_signal != 0 || $sanitize_dump == yes} {
|
||||||
puts "Server crashed (by signal: $by_signal), with payload: $printable_dump"
|
puts "Server crashed (by signal: $by_signal), with payload: $printable_dump"
|
||||||
set print_commands true
|
set print_commands true
|
||||||
}
|
}
|
||||||
@ -186,7 +186,7 @@ foreach sanitize_dump {no yes} {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
# if we run sanitization we never expect the server to crash at runtime
|
# if we run sanitization we never expect the server to crash at runtime
|
||||||
if { $sanitize_dump == 1} {
|
if {$sanitize_dump == yes} {
|
||||||
assert_equal $stat_terminated_in_restore 0
|
assert_equal $stat_terminated_in_restore 0
|
||||||
assert_equal $stat_terminated_in_traffic 0
|
assert_equal $stat_terminated_in_traffic 0
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user