[FFmpeg-devel] [PATCH] ffprobe: Fix memory leak

James Almer jamrial at gmail.com
Fri Jun 21 17:25:33 EEST 2019


On 6/21/2019 11:13 AM, Derek Buitenhuis wrote:
> On 21/06/2019 14:46, James Almer wrote:
>> Why not just call this unconditionally instead of the init() + zero below?
> 
> I wasn't sure from a quick skim if these packets were
> referenced elsewhere (and thus unrefercing twice would
> be problematic).

Unreferencing a packet cleans the struct, so calling unref() on it again
is basically a no-op.

> 
> If it's safe to do so, I will.
> 
> - Derel
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> 



More information about the ffmpeg-devel mailing list