[FFmpeg-devel] Memory leaks in libavformat/segment.c

Hendrik Leppkes h.leppkes at gmail.com
Tue Dec 8 12:05:58 CET 2015

On Tue, Dec 8, 2015 at 11:46 AM, Lvqier <lvqier at gmail.com> wrote:
> On 12/8/15 6:26 PM, Hendrik Leppkes wrote:
>> On Tue, Dec 8, 2015 at 11:19 AM, Lvqier <lvqier at gmail.com> wrote:
>>> Hi Hendrik,
>>>      The attached is another patch which uses av_reallocp to fix this
>>> issue.
>> Thanks, that looks almost good. But you should check for a negative
>> return value, as all errors are negative.
> Do you mean that the set_segment_filename function should return the value
> which returns from av_reallocp instead of AVERROR(ENOMEM) when it fails?

That would be an option too, but right now you just do
if(av_reallop(..)), at the least this should be if (av_reallocp(...) <

or even better:
ret = av_reallocp(...);
if (ret < 0)
    return ret;

- Hendrik

More information about the ffmpeg-devel mailing list