On Sun, Oct 25, 2020 at 09:00:53PM -0300, James Almer wrote: > On 10/25/2020 8:00 PM, Michael Niedermayer wrote: > > Fixes: NULL pointer dereference > > Fixes: > > 26550/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AV1_fuzzer-5417762807349248 > > > > Found-by: continuous fuzzing process > > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > > --- > > libavcodec/av1dec.c | 7 ++++++- > > 1 file changed, 6 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c > > index a0a279d65b..bb4295aa6f 100644 > > --- a/libavcodec/av1dec.c > > +++ b/libavcodec/av1dec.c > > @@ -673,7 +673,12 @@ static int av1_decode_frame(AVCodecContext *avctx, > > void *frame, > > for (int i = 0; i < s->current_obu.nb_units; i++) { > > CodedBitstreamUnit *unit = &s->current_obu.units[i]; > > AV1RawOBU *obu = unit->content; > > - const AV1RawOBUHeader *header = &obu->header; > > + const AV1RawOBUHeader *header; > > + > > + if (!obu) > > This looks like a "regression" since ae7686a64f, where ENOSYS (an error > code that's not propagated to the caller) on unsupported OBU types is > now returned before cbs_av1 even gets a chance to allocate the unit's > content. >
> Should be ok. will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into democracies and democracies degenerate into despotisms. -- Aristotle
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".
