[FFmpeg-devel] [PATCH 1/3] avcodec/bitpacked: ,

[Michael Niedermayer]

On Thu, Jun 04, 2020 at 03:11:12AM +0200, James Darnley wrote:
> On 2020-06-04 01:19, Michael Niedermayer wrote:
> > Fixes: array end overread
> > Fixes: 22395/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_BITPACKED_fuzzer-5760940300828672
> > 
> > Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> >  libavcodec/bitpacked.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/bitpacked.c b/libavcodec/bitpacked.c
> > index be7d1e3629..952ba73a32 100644
> > --- a/libavcodec/bitpacked.c
> > +++ b/libavcodec/bitpacked.c
> > @@ -147,7 +147,7 @@ AVCodec ff_bitpacked_decoder = {
> >      .decode = bitpacked_decode,
> >      .capabilities = AV_CODEC_CAP_EXPERIMENTAL,
> >      .codec_tags     = (const uint32_t []){
> > -        MKTAG('U', 'Y', 'V', 'Y')
> > +        MKTAG('U', 'Y', 'V', 'Y'),
> >          FF_CODEC_TAGS_END,
> >      },
> >  };
> > 
> 
> I think you should add to the commit title.  Something like "add missing
> comma to codec tags".

sure, will apply with that

thx

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

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200604/24328c98/attachment.sig>