On Sun, 25 Aug 2013, Diego Biurrun wrote:

The second malloc is checked, but I think there is a memory leak if you
return after it in case of failure without freeing the memory from the
avcodec_alloc_context3() you added a check for.  Same for the av_malloc
below.

I also pointed out this risk when I reviewed John's patchset earlier last week, which is exactly why I now took the time to dig into the issue properly.

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to