[FFmpeg-devel] [PATCH] lavc: fix h264 parser bug

[Lingjiang Fang]

On Fri, 13 May 2022 00:22:02 +0800
Lingjiang Fang <vacingfang at foxmail.com> wrote:

ping for review
padding zero after pps may lead to decode crash in same ipads
btw, I am working on update fate cases to fit this bugfix

> will give a wrong nalu length(+1) when next nalu start with 00000001
> this bug will lead to an padding zero to pps in AVC Sequence
> header(flv) or in avc1(mpr)
> 
> ---
> 
>  libavcodec/h2645_parse.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c
> index 03780680c6..2e941a2bd0 100644
> --- a/libavcodec/h2645_parse.c
> +++ b/libavcodec/h2645_parse.c
> @@ -134,6 +134,10 @@ int ff_h2645_extract_rbsp(const uint8_t *src,
> int length, dst[di++] = src[si++];
>  
>  nsc:
> +    if (src[si-1] == 0) {
> +        --di;
> +        --si;
> +    }
>      memset(dst + di, 0, AV_INPUT_BUFFER_PADDING_SIZE);
>  
>      nal->data = dst;



Regards,
Lingjiang Fang