[FFmpeg-devel] [PATCH] Rudimentary support for id3v2 APIC tags.

Clément Bœsch ubitux at gmail.com
Mon Jan 2 18:01:35 CET 2012


On Wed, Dec 28, 2011 at 09:24:18PM -0800, Adrian Drzewiecki wrote:
> (Sorry, I'm still not subscribed to the mailing list, so please CC: me.)
> 
> Thank you all for the feedback. I'll split out the patch as suggested by Clément.
> Regarding my changes to av_dict_set() -- the current code does not free memory in some of the error paths.
> Should that be fixed?
> 

I didn't look at the memleak you're mentionning; but if you see a memleak
in the current code base, please send a separate patch for it (or report a
bug). These kind of patchs are obviously welcome.

[...]

PS: sorry for the late reply.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120102/71dc0060/attachment.asc>


More information about the ffmpeg-devel mailing list