Skip to content

cifuzz: do not report non-reproducible crashes#2600

Merged
rminnich merged 1 commit into
u-root:mainfrom
RiSKeD:cifuzz-reproducible
Jan 10, 2023
Merged

cifuzz: do not report non-reproducible crashes#2600
rminnich merged 1 commit into
u-root:mainfrom
RiSKeD:cifuzz-reproducible

Conversation

@RiSKeD
Copy link
Copy Markdown
Contributor

@RiSKeD RiSKeD commented Jan 9, 2023

Added the report-unreproducible-crashes flag to the cifuzz config (just a verbose change, this is false per default anyway).
Limited the input byte size for the gosh fuzzing to avoid any timeouts due to it.

@RiSKeD RiSKeD force-pushed the cifuzz-reproducible branch from bc6c317 to f7830bc Compare January 9, 2023 11:47
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 9, 2023

Codecov Report

Base: 74.83% // Head: 74.83% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (f7830bc) compared to base (4c2d93f).
Patch has no changes to coverable lines.

❗ Current head f7830bc differs from pull request most recent head 8d2a784. Consider uploading reports for the commit 8d2a784 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2600      +/-   ##
==========================================
- Coverage   74.83%   74.83%   -0.01%     
==========================================
  Files         408      408              
  Lines       41344    41344              
==========================================
- Hits        30940    30939       -1     
- Misses      10404    10405       +1     
Impacted Files Coverage Δ
cmds/core/mv/mv.go 77.27% <0.00%> (-2.28%) ⬇️
pkg/libinit/root_linux.go 27.02% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@RiSKeD
Copy link
Copy Markdown
Contributor Author

RiSKeD commented Jan 9, 2023

I opened up an issue regarding CIFuzz failing on PRs.

@rminnich rminnich added the Awaiting author Waiting for new changes or feedback for author. label Jan 10, 2023
Signed-off-by: Fabian Wienand <fabian.wienand@9elements.com>
@rminnich rminnich merged commit b289ec4 into u-root:main Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Awaiting author Waiting for new changes or feedback for author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants