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