Re: [FFmpeg-devel] [PATCH] avcodec/vvcdec: frame_context_setup, set fc->ref to NULL

2024-02-14 Thread Anton Khirnov
Quoting Frank Plowman (2024-02-13 11:38:50) > Btw, I don't think you should add Signed-off-by tags for other people. > Their exact meaning varies by project and I am not sure of their meaning > in FFmpeg (if there is one), There is no officially accepted one. E.g. I add my signoff to patches I pu

Re: [FFmpeg-devel] [PATCH] avcodec/vvcdec: frame_context_setup, set fc->ref to NULL

2024-02-13 Thread Nuo Mi
On Tue, Feb 13, 2024 at 6:39 PM Frank Plowman wrote: > On 13/02/2024 02:30, Nuo Mi wrote: > > fc->ref points to an old VVCFrame, which cannot be used after > frame_context_setup. > > This prevents crashes in decode_nal_units-->ff_vvc_report_frame_finished. > > > > Signed-off-by: Frank Plowman >

Re: [FFmpeg-devel] [PATCH] avcodec/vvcdec: frame_context_setup, set fc->ref to NULL

2024-02-13 Thread Frank Plowman
On 13/02/2024 02:30, Nuo Mi wrote: > fc->ref points to an old VVCFrame, which cannot be used after > frame_context_setup. > This prevents crashes in decode_nal_units-->ff_vvc_report_frame_finished. > > Signed-off-by: Frank Plowman > --- > libavcodec/vvc/vvcdec.c | 2 ++ > 1 file changed, 2 inse

[FFmpeg-devel] [PATCH] avcodec/vvcdec: frame_context_setup, set fc->ref to NULL

2024-02-12 Thread Nuo Mi
fc->ref points to an old VVCFrame, which cannot be used after frame_context_setup. This prevents crashes in decode_nal_units-->ff_vvc_report_frame_finished. Signed-off-by: Frank Plowman --- libavcodec/vvc/vvcdec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/vvc/vvcdec.c b/l