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

rust: update broti decompressor crate #9540

Closed
wants to merge 1 commit into from

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=59687

Describe changes:

  • rust: update broti decompressor crate patch version

@jasonish did I get it right ?
I ran cargo update -p brotli-decompressor and moved the changes to Cargo.lock.in

@codecov
Copy link

codecov bot commented Oct 3, 2023

Codecov Report

Merging #9540 (b44b589) into master (bb15a8f) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9540      +/-   ##
==========================================
- Coverage   82.19%   82.19%   -0.01%     
==========================================
  Files         968      968              
  Lines      274216   274216              
==========================================
- Hits       225405   225399       -6     
- Misses      48811    48817       +6     
Flag Coverage Δ
fuzzcorpus 64.07% <ø> (+<0.01%) ⬆️
suricata-verify 60.91% <ø> (-0.02%) ⬇️
unittests 62.88% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@catenacyber
Copy link
Contributor Author

@jasonish
Copy link
Member

jasonish commented Oct 3, 2023

Looks right for a transitive dependency.

@victorjulien
Copy link
Member

Looks right for a transitive dependency.

Please hit approve if you, well, approve :)

This was referenced Oct 5, 2023
@victorjulien victorjulien self-requested a review October 6, 2023 15:15
Copy link
Member

@victorjulien victorjulien left a comment

Choose a reason for hiding this comment

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

Typo in commit title broti should be brotli

@catenacyber
Copy link
Contributor Author

Typo in commit title broti should be brotli

Nice, I had to read twice your comment to see the diff

#9571

@catenacyber catenacyber closed this Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants