-
-
Notifications
You must be signed in to change notification settings - Fork 28
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: use after free in ffmpeg #248
fix: use after free in ffmpeg #248
Conversation
?brawl merge |
📌 Commit 51b98d1 has been approved and added to the merge queue. Requested by: @TroyKomodo Approved by: |
?brawl cancel |
🚨 Cancelled CI run |
?brawl merge |
📌 Commit 51b98d1 has been approved and added to the merge queue. Requested by: @TroyKomodo Approved by: @TroyKomodo |
🎉 Build successful! Approved by: @TroyKomodo |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #248 +/- ##
==========================================
- Coverage 50.24% 50.24% -0.01%
==========================================
Files 196 196
Lines 16202 16204 +2
==========================================
Hits 8141 8141
- Misses 8061 8063 +2
|
Motivation
there is a use after free in
encoder::Encoder::new
when codec-specific options are usedSolution
don't use after free