Skip to content

feat(test): add fuzz test for range_reader#2609

Merged
Xuanwo merged 14 commits intoapache:mainfrom
dqhl76:fuzz-test-fsreader
Jul 17, 2023
Merged

feat(test): add fuzz test for range_reader#2609
Xuanwo merged 14 commits intoapache:mainfrom
dqhl76:fuzz-test-fsreader

Conversation

@dqhl76
Copy link
Copy Markdown
Member

@dqhl76 dqhl76 commented Jul 8, 2023

part of: #2551

For this PR, I used cargo fuzz to initialize the fuzz framework and added a fuzz target example for fs service's range reader.

I will continue to add more fuzz targets in the near future.


Why not put these files in the usual test folder?

We want to make the fuzz tests with oss-fuzz, which need us put them under a a top-level fuzz directory.
oss-fuzz with rust

dqhl76 added 2 commits July 6, 2023 11:15
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Comment thread core/fuzz/Cargo.toml Outdated
Comment thread core/fuzz/Cargo.toml Outdated
Comment thread core/fuzz/Cargo.toml Outdated
Comment thread core/fuzz/fuzz_targets/fuzz_reader_fs.rs Outdated
Comment thread core/fuzz/fuzz_targets/fuzz_reader_fs.rs Outdated
Comment thread core/fuzz/fuzz_targets/fuzz_reader_fs.rs Outdated
Comment thread core/fuzz/fuzz_targets/fuzz_reader_fs.rs Outdated
Comment thread core/fuzz/Cargo.toml Outdated
dqhl76 added 2 commits July 8, 2023 22:01
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
@dqhl76 dqhl76 marked this pull request as draft July 8, 2023 14:24
dqhl76 added 2 commits July 11, 2023 09:54
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
@dqhl76 dqhl76 marked this pull request as ready for review July 11, 2023 02:02
@dqhl76
Copy link
Copy Markdown
Member Author

dqhl76 commented Jul 11, 2023

@Xuanwo PTAL

Comment thread core/fuzz/Cargo.toml
Comment thread core/fuzz/README.md Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
Comment thread core/fuzz/fuzz_reader.rs Outdated
@dqhl76 dqhl76 marked this pull request as draft July 11, 2023 09:36
dqhl76 added 3 commits July 13, 2023 14:36
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
@dqhl76 dqhl76 force-pushed the fuzz-test-fsreader branch from 2cd7b87 to 05c7384 Compare July 15, 2023 14:09
dqhl76 added 3 commits July 16, 2023 17:10
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
Signed-off-by: dqhl76 <dqhl76@gmail.com>
@dqhl76 dqhl76 marked this pull request as ready for review July 16, 2023 09:38
@dqhl76
Copy link
Copy Markdown
Member Author

dqhl76 commented Jul 16, 2023

@Xuanwo PTAL

Signed-off-by: dqhl76 <dqhl76@gmail.com>
@dqhl76 dqhl76 changed the title test: add fuzz search for fs reader test: add fuzz test for range_reader Jul 16, 2023
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Great, thanks a lot!

@Xuanwo Xuanwo changed the title test: add fuzz test for range_reader feat(test): add fuzz test for range_reader Jul 17, 2023
@Xuanwo Xuanwo merged commit b2ae4a3 into apache:main Jul 17, 2023
@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Jul 17, 2023
@dqhl76
Copy link
Copy Markdown
Member Author

dqhl76 commented Jul 17, 2023

Thanks for your review!

@dqhl76 dqhl76 deleted the fuzz-test-fsreader branch July 17, 2023 08:30
@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

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants