[vlc-devel] [PATCH 2/2] flac: fix memory leak
ssbssa at yahoo.de
Tue Sep 2 18:49:20 CEST 2014
> Tristan Matthews <le.businessman at gmail.com> schrieb am 21:41 Montag, 1.September 2014:
> On Fri, Aug 29, 2014 at 12:26 PM, Hannes Domani <ssbssa at yahoo.de> wrote:
> > I'm not sure if this one is completely correct.
> > In this case p_sys->p_block was allocated by block_Alloc() inside
> > ProcessHeader().
> > But I don't see when this new block is actually used, because it's
> > overwritten immediately after ProcessHeader().
> > Can somebody please have a look at this?
> The new block is used inside ProcessHeader() (although it's not very
> obvious). FLAC__stream_decoder_process_until_end_of_metadata, invoked
> at the end of ProcessHeader(), eventually calls DecoderReadCallback,
> which uses the block. So you can free it immediately after that call.
> I would suggest moving this block_Release there (at the end of
> ProcessHeader) to make the connection clearer.
Here the new patch.
> Also note that the calls to block_Alloc in ProcessHeader() are not
> verified at the moment (they may return NULL), but that could be done
> in a separate patch.
It's the next one.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 768 bytes
Desc: not available
More information about the vlc-devel