[FFmpeg-devel] [PATCH] mov: Fix memory leaks on aborted header parsing.
michaelni at gmx.at
Fri Apr 13 12:10:06 CEST 2012
On Thu, Apr 12, 2012 at 05:50:55PM -0700, dalecurtis at chromium.org wrote:
> From: Dale Curtis <dalecurtis at chromium.org>
> If mov_read_header exits under error, the memory allocated is
> not freed.
> Signed-off-by: Dale Curtis <dalecurtis at chromium.org>
i wonder if we can simply call the close for all demuxers in utils.c,
as this likely affects more demuxers
but until thats tested and investigated, ill apply this patch.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel