[FFmpeg-devel] [PATCH] Fix memory leak in lrcdec.c

Carl Eugen Hoyos ceffmpeg at gmail.com
Sat Jan 20 03:39:02 EET 2018


2018-01-20 1:57 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:
> On Fri, Jan 19, 2018 at 01:17:07PM -0800, Nikolas Bowe wrote:
>> ---
>>  libavformat/lrcdec.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/libavformat/lrcdec.c b/libavformat/lrcdec.c
>> index 12f74b22a0..f4e9a4efa9 100644
>> --- a/libavformat/lrcdec.c
>> +++ b/libavformat/lrcdec.c
>> @@ -212,6 +212,7 @@ static int lrc_read_header(AVFormatContext *s)
>>      }
>>      ff_subtitles_queue_finalize(s, &lrc->q);
>>      ff_metadata_conv_ctx(s, NULL, ff_lrc_metadata_conv);
>> +    av_bprint_finalize(&line, NULL);
>>      return 0;
>>  }
>
> How did you find this ?
>
> iam asking because ideally all fuzzing stuff should be run by
> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> while currently thats not the case and disk space restrictions

> togther with "no shared libs allowed" and a few other things

(Sorry if I misread this)
Why is "no shared libs" an issue with FFmpeg?

> like one seperate tool for each test make it hard to extend it.

Carl Eugen


More information about the ffmpeg-devel mailing list