Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Seek before the start of file should be invalid #2718

Merged
merged 2 commits into from
Jul 26, 2023

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jul 26, 2023

Fix #2717

The crash case has been fixed.

:) RUST_LOG=debug cargo +nightly fuzz run fuzz_range_reader /tmp/crash-4526ccffa285c58ef8c7731d4a5dfef69d38cfff
   Compiling opendal v0.38.1 (/home/xuanwo/Code/apache/incubator-opendal/core)
   Compiling opendal-fuzz v0.0.0 (/home/xuanwo/Code/apache/incubator-opendal/core/fuzz)
    Finished release [optimized] target(s) in 2m 06s
    Finished release [optimized] target(s) in 0.09s
     Running `/home/xuanwo/Code/apache/incubator-opendal/target/x86_64-unknown-linux-gnu/release/fuzz_range_reader -artifact_prefix=/home/xuanwo/Code/apache/incubator-opendal/core/fuzz/artifacts/fuzz_range_reader/ /tmp/crash-4526ccffa285c58ef8c7731d4a5dfef69d38cfff`
INFO: Running with entropic power schedule (0xFF, 100).
INFO: Seed: 3882736719
INFO: Loaded 1 modules   (1722493 inline 8-bit counters): 1722493 [0x5621e8ab8590, 0x5621e8c5ce0d), 
INFO: Loaded 1 PC tables (1722493 PCs): 1722493 [0x5621e8c5ce10,0x5621ea6a55e0), 
/home/xuanwo/Code/apache/incubator-opendal/target/x86_64-unknown-linux-gnu/release/fuzz_range_reader: Running 1 inputs 1 time(s) each.
Running: /tmp/crash-4526ccffa285c58ef8c7731d4a5dfef69d38cfff
Executed /tmp/crash-4526ccffa285c58ef8c7731d4a5dfef69d38cfff in 62 ms
***
*** NOTE: fuzzing was not performed, you have only
***       executed the target code on a fixed set of inputs.
***
RUST_LOG=debug cargo +nightly fuzz run fuzz_range_reader   124.20s user 2.24s system 99% cpu 2:06.52 total

Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Copy link
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Xuanwo Xuanwo merged commit a54a4b5 into main Jul 26, 2023
75 checks passed
@Xuanwo Xuanwo deleted the fix-fuzz-failure-2717 branch July 26, 2023 11:50
@oowl oowl mentioned this pull request Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fuzz-test-run-fs (fuzz_range_reader) failed for attempt to subtract with overflow
2 participants