[FFmpeg-devel] [PATCH v2] avcodec/cbs_apv: don't return an error when reading empty buffers

[Mark Thompson]

On 30/04/2025 22:22, James Almer wrote:
> The output will be a fragment with zero units, which is a lot more user friendly
> than making them think something went wrong, as it already happens with cbs_av1.
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavcodec/cbs_apv.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/cbs_apv.c b/libavcodec/cbs_apv.c
> index 2c4a1daaf0..e7dd04825c 100644
> --- a/libavcodec/cbs_apv.c
> +++ b/libavcodec/cbs_apv.c
> @@ -168,8 +168,8 @@ static int cbs_apv_split_fragment(CodedBitstreamContext *ctx,
>      uint32_t signature;
>      int err, trace;
>  
> -    if (header) {
> -        // Ignore extradata fragments.
> +    if (header || !frag->data_size) {
> +        // Ignore empty or extradata fragments.
>          return 0;
>      }
>  

LGTM.

Thanks,

- Mark