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

Michael Niedermayer michael at niedermayer.cc
Sat Jan 20 04:10:00 EET 2018


On Sat, Jan 20, 2018 at 02:39:02AM +0100, Carl Eugen Hoyos wrote:
> 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?

its an issue for the oss fuzz testcases it seems.
all code is statically linked even libs which on a modern system
are linked as shared libs normally
dont ask me why. I very much would love to switch to shared linking
also i would be very happy to find out this is a misunderstanding 
somehow and we can use shared linking ...

that together with one binary per tested codec causes space issues
which should not come as much of a surprise. Duplicating libavcodec
for each test is not great.
On top of that fate samples are partly duplicated too

also see:
https://github.com/google/oss-fuzz/issues/567

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180120/ced02b4d/attachment.sig>


More information about the ffmpeg-devel mailing list