On Thu, Jul 18, 2024 at 07:19:17PM -0300, James Almer wrote: > On 7/18/2024 7:16 PM, Michael Niedermayer wrote: > > Fixes: MemLeak > > Fixes: > > 69853/clusterfuzz-testcase-minimized-ffmpeg_dem_MOV_fuzzer-4660448545275904 > > > > Found-by: continuous fuzzing process > > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > > --- > > libavformat/mov.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/libavformat/mov.c b/libavformat/mov.c > > index ce95842ce58..82fce7ef5c1 100644 > > --- a/libavformat/mov.c > > +++ b/libavformat/mov.c > > @@ -897,6 +897,8 @@ static int mov_read_iacb(MOVContext *c, AVIOContext > > *pb, MOVAtom atom) > > st = c->fc->streams[c->fc->nb_streams - 1]; > > sc = st->priv_data; > > + if (st->codecpar->extradata) > > + return AVERROR_INVALIDDATA; > > Maybe it's better to do like other atoms where we ignore duplicate entries > (See mov_read_glbl(), used for h264/hevc/etc).
IIRC its a mix of mov_read_iacb() and mov_read_stsd() both setting extradata i can certainly do a "return 0" with some warning if you prefer that thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB "Nothing to hide" only works if the folks in power share the values of you and everyone you know entirely and always will -- Tom Scott
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
