Re: [FFmpeg-devel] [PATCH] lavc/vvc: Check fc->ref contains valid reference

2024-02-16 Thread Anton Khirnov
Quoting Nuo Mi (2024-02-13 04:03:11) > HI Lynee, > the name is from hevc. I do not like it either, but so many functions > copied from hevc will use it. ... The naming in hevcdec is really schizophrenic, I wish somebody did something about it. Certainly do not feel compelled to use it as a model.

Re: [FFmpeg-devel] [PATCH] lavc/vvc: Check fc->ref contains valid reference

2024-02-12 Thread Nuo Mi
On Fri, Feb 9, 2024 at 6:46 AM Frank Plowman wrote: > > > On 08/02/2024 21:50, Lynne wrote: > > Feb 8, 2024, 22:16 by p...@frankplowman.com: > > > >> From: Frank Plowman > >> > >> Depending on where exactly decode_nal_unit failed, it is possible that > >> fc->ref holds a VVCFrame which has had

Re: [FFmpeg-devel] [PATCH] lavc/vvc: Check fc->ref contains valid reference

2024-02-08 Thread Frank Plowman
On 08/02/2024 21:50, Lynne wrote: > Feb 8, 2024, 22:16 by p...@frankplowman.com: > >> From: Frank Plowman >> >> Depending on where exactly decode_nal_unit failed, it is possible that >> fc->ref holds a VVCFrame which has had ff_vvc_unref_frame called on it >> and not yet had ref_frame called

Re: [FFmpeg-devel] [PATCH] lavc/vvc: Check fc->ref contains valid reference

2024-02-08 Thread Lynne
Feb 8, 2024, 22:16 by p...@frankplowman.com: > From: Frank Plowman > > Depending on where exactly decode_nal_unit failed, it is possible that > fc->ref holds a VVCFrame which has had ff_vvc_unref_frame called on it > and not yet had ref_frame called on it. In this case, fc->ref most of > the

[FFmpeg-devel] [PATCH] lavc/vvc: Check fc->ref contains valid reference

2024-02-08 Thread post
From: Frank Plowman Depending on where exactly decode_nal_unit failed, it is possible that fc->ref holds a VVCFrame which has had ff_vvc_unref_frame called on it and not yet had ref_frame called on it. In this case, fc->ref most of the fields of fc->ref are NULL and attempting to call