Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Add git.Blob Fuzz Target#1911

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

DaveLak
Copy link
Contributor

@DaveLakDaveLak commentedMay 4, 2024•
edited
Loading

Adds a new fuzz target forgit.Blob, using fuzzed data for all__init__ arguments.

60 second local test run output:

INFO: A corpus is not provided, starting from an empty corpus#2INITED cov: 211 ft: 211 corp: 1/1b exec/s: 0 rss: 72Mb#10NEW    cov: 230 ft: 232 corp: 2/4b lim: 4 exec/s: 0 rss: 74Mb L: 3/3 MS: 2 InsertByte-InsertByte-#26NEW    cov: 230 ft: 251 corp: 3/7b lim: 4 exec/s: 0 rss: 76Mb L: 3/3 MS: 1 ChangeByte-#128pulse  cov: 230 ft: 251 corp: 3/7b lim: 4 exec/s: 64 rss: 77Mb#181REDUCE cov: 230 ft: 251 corp: 3/6b lim: 4 exec/s: 60 rss: 77Mb L: 2/3 MS: 4 ChangeBinInt-CopyPart-EraseBytes-ChangeBit-#256pulse  cov: 230 ft: 251 corp: 3/6b lim: 4 exec/s: 64 rss: 77Mb#512pulse  cov: 230 ft: 251 corp: 3/6b lim: 6 exec/s: 56 rss: 77Mb#1024pulse  cov: 230 ft: 251 corp: 3/6b lim: 11 exec/s: 56 rss: 77Mb#1919NEW    cov: 239 ft: 260 corp: 4/27b lim: 21 exec/s: 56 rss: 77Mb L: 21/21 MS: 1 InsertRepeatedBytes-#1926REDUCE cov: 239 ft: 260 corp: 4/26b lim: 21 exec/s: 56 rss: 77Mb L: 20/20 MS: 2 ChangeBit-EraseBytes-#2048pulse  cov: 239 ft: 260 corp: 4/26b lim: 21 exec/s: 55 rss: 77Mb#2122REDUCE cov: 239 ft: 260 corp: 4/25b lim: 21 exec/s: 55 rss: 77Mb L: 1/20 MS: 1 EraseBytes-#2537REDUCE cov: 281 ft: 302 corp: 5/50b lim: 25 exec/s: 56 rss: 78Mb L: 25/25 MS: 4 ChangeBit-InsertRepeatedBytes-ChangeBinInt-ChangeBinInt-#3382DONE   cov: 281 ft: 302 corp: 5/50b lim: 33 exec/s: 55 rss: 78MbDone 3382 runs in 61 second(s)stat::number_of_executed_units: 3382stat::average_exec_per_sec:     55stat::new_units_added:          7stat::slowest_unit_time_sec:    0stat::peak_rss_mb:              78

Based on the `test_blob.py` unit test.
This increases the edges reached by the fuzzer, making for a moreeffective test with higher coverage.
Copy link
Member

@ByronByron left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

It's a bit unpleasant to see that that thegit.Blob has a path associated to it, as if it was always read from a loose object, with IO then dominating the runtime. I'd always hope there are ways to trigger the important code without touching disk for it.

@ByronByron merged commit2493c3a intogitpython-developers:mainMay 5, 2024
26 checks passed
@DaveLakDaveLak deleted the add-blob-fuzz-target branchMay 15, 2024 14:58
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@ByronByronByron approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@DaveLak@Byron
[8]ページ先頭
©2009-2025 Movatter.jp