Skip to content

decoder/ffmpeg: free AVIO buffer#2519

Closed
hyder365 wants to merge 1 commit into
MusicPlayerDaemon:masterfrom
hyder365:fix-ffmpeg-avio-buffer
Closed

decoder/ffmpeg: free AVIO buffer#2519
hyder365 wants to merge 1 commit into
MusicPlayerDaemon:masterfrom
hyder365:fix-ffmpeg-avio-buffer

Conversation

@hyder365

Copy link
Copy Markdown
Contributor

Free the buffer allocated for avio_alloc_context() if creating the AVIOContext fails.

Without this cleanup, that failure path leaks the 8192-byte buffer.

@MaxKellermann

Copy link
Copy Markdown
Member

This PR contains text that is missing in the commit message.
This is annoying enough that I will close every PR from you that doesn't even pass the most basic smell test.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants