Skip to content

Commit

Permalink
fix: invalid memcopy bug in fuzzer
Browse files Browse the repository at this point in the history
  • Loading branch information
liuq19 committed Oct 18, 2023
1 parent c838482 commit 75075bf
Show file tree
Hide file tree
Showing 5 changed files with 39 additions and 224 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ jobs:
cargo check
cargo test
cargo bench
cargo fuzz check
cargo +nightly fuzz run fuzz_value
test-linux-aarch64:
runs-on: [self-hosted, arm]
Expand Down
252 changes: 31 additions & 221 deletions fuzz/Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion fuzz/fuzz_targets/from_slice.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,7 @@
use libfuzzer_sys::fuzz_target;

fuzz_target!(|data: &[u8]| {
_ = sonic_rs::value::to_dom(data);
if let Ok(dom) = sonic_rs::value::dom_from_slice(data) {
let _ = sonic_rs::to_string(&dom).unwrap();
}
});
3 changes: 2 additions & 1 deletion src/reader.rs
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@ pub trait Reader<'de>: Sealed {
fn slice_unchecked(&self, start: usize, end: usize) -> &'de [u8];

#[cold]
fn position_of_index(&self, i: usize) -> Position {
fn position_of_index(&self, mut i: usize) -> Position {
i = i.min(self.as_u8_slice().len());
let mut position = Position { line: 1, column: 0 };
for ch in &self.as_u8_slice()[..i] {
match *ch {
Expand Down
2 changes: 1 addition & 1 deletion src/value/node.rs
Original file line number Diff line number Diff line change
Expand Up @@ -976,7 +976,7 @@ impl Document {
let dst = alloc.try_alloc_layout(layout).map_err(Error::custom)?;
let json_buf = unsafe {
let dst = dst.as_ptr();
std::ptr::copy_nonoverlapping(json.as_ptr(), dst, real_size);
std::ptr::copy_nonoverlapping(json.as_ptr(), dst, len);
*(dst.add(len)) = b'x';
*(dst.add(len + 1)) = b'"';
*(dst.add(len + 2)) = b'x';
Expand Down

0 comments on commit 75075bf

Please sign in to comment.