DevComp/zip-rs-wasm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Revert "chore: Use panic! rather than abort to ensure the fuzz harness can process the failure"

Browse source 
This reverts commit 0011370fdc.
This commit is contained in:
Chris Hennick 2024-05-05 20:19:09 -07:00
parent 0011370fdc
commit 30ef662aa2
No known key found for this signature in database
GPG key ID: DA47AABA4961C509
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
fuzz/fuzz_targets/fuzz_write.rs
View file

@ -2,7 +2,7 @@
use arbitrary::Arbitrary; use arbitrary::Arbitrary;
use libfuzzer_sys::fuzz_target; use libfuzzer_sys::fuzz_target;
use replace_with::replace_with; use replace_with::replace_with_or_abort;
use std::io::{Cursor, Read, Seek, Write}; use std::io::{Cursor, Read, Seek, Write};
use std::path::PathBuf; use std::path::PathBuf;
@ -83,9 +83,7 @@ where
} }
if operation.reopen { if operation.reopen {
let old_comment = writer.get_raw_comment().to_owned(); let old_comment = writer.get_raw_comment().to_owned();
replace_with(writer, || { replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| {
panic!("Failed to reopen writer");
}, |old_writer: zip::ZipWriter<T>| {
let new_writer = let new_writer =
zip::ZipWriter::new_append(old_writer.finish().unwrap()).unwrap(); zip::ZipWriter::new_append(old_writer.finish().unwrap()).unwrap();
assert_eq!(&old_comment, new_writer.get_raw_comment()); assert_eq!(&old_comment, new_writer.get_raw_comment());