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

[James Darnley]

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".

Other than that this looks fine.