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 <jamr...@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 _______________________________________________ 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".