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

fix: use after free in ffmpeg #248

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

Juliapixel
Copy link
Contributor

@Juliapixel Juliapixel commented Jan 10, 2025

Motivation

there is a use after free in encoder::Encoder::new when codec-specific options are used

Solution

don't use after free

@TroyKomodo
Copy link
Member

?brawl merge

@scuffle-brawl
Copy link
Contributor

scuffle-brawl bot commented Jan 10, 2025

📌 Commit 51b98d1 has been approved and added to the merge queue.

Requested by: @TroyKomodo

Approved by:

@TroyKomodo
Copy link
Member

?brawl cancel

@scuffle-brawl
Copy link
Contributor

scuffle-brawl bot commented Jan 10, 2025

🚨 Cancelled CI run

@TroyKomodo
Copy link
Member

?brawl merge

@scuffle-brawl
Copy link
Contributor

scuffle-brawl bot commented Jan 10, 2025

📌 Commit 51b98d1 has been approved and added to the merge queue.

Requested by: @TroyKomodo

Approved by: @TroyKomodo

@scuffle-brawl
Copy link
Contributor

scuffle-brawl bot commented Jan 10, 2025

⌛ Trying commit 51b98d1 with merge 382cbb0...

@scuffle-brawl
Copy link
Contributor

scuffle-brawl bot commented Jan 10, 2025

🎉 Build successful!
Completed in 2:22

Approved by: @TroyKomodo
Pushing 382cbb0 to main

@scuffle-brawl scuffle-brawl bot merged commit 382cbb0 into ScuffleCloud:main Jan 10, 2025
6 checks passed
Copy link

codecov bot commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 50.24%. Comparing base (905efab) to head (4126e27).
Report is 3 commits behind head on main.

Current head 4126e27 differs from pull request most recent head 51b98d1

Please upload reports for the commit 51b98d1 to get more accurate results.

Files with missing lines Patch % Lines
crates/ffmpeg/src/encoder.rs 0.00% 5 Missing ⚠️
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     
Files with missing lines Coverage Δ
crates/ffmpeg/src/encoder.rs 0.00% <0.00%> (ø)
Components Coverage Δ
scuffle-batching 100.00% <ø> (ø)
scuffle-bootstrap 27.15% <ø> (ø)
scuffle-context 100.00% <ø> (ø)
scuffle-ffmpeg 0.00% <0.00%> (ø)
scuffle-h3-webtransport 0.00% <ø> (ø)
scuffle-http 0.00% <ø> (ø)
scuffle-metrics 42.69% <ø> (ø)
scuffle-pprof 0.00% <ø> (ø)
scuffle-settings 0.00% <ø> (ø)
scuffle-signal 100.00% <ø> (ø)
postcompile 0.00% <ø> (ø)
scuffle-image-processor ∅ <ø> (∅)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants