Re: [FFmpeg-devel] [PATCH v1] lavc/vvc: Validate num_signalled_palette_entries

2025-05-19 Thread Nuo Mi
On Sun, May 18, 2025 at 2:51 PM Frank Plowman wrote: > On 18/05/2025 02:42, Nuo Mi wrote: > > Hi Frank, > > 👍,your fuzzing infrastructure caught this issue as well. > > How about this: > > > https://patchwork.ffmpeg.org/project/ffmpeg/patch/20250517055150.807683-1-nuomi2...@gmail.com/ > > Sorry,

Re: [FFmpeg-devel] [PATCH v1] lavc/vvc: Validate num_signalled_palette_entries

2025-05-17 Thread Frank Plowman
On 18/05/2025 02:42, Nuo Mi wrote: > Hi Frank, > 👍,your fuzzing infrastructure caught this issue as well. > How about this: > https://patchwork.ffmpeg.org/project/ffmpeg/patch/20250517055150.807683-1-nuomi2...@gmail.com/ Sorry, I missed this. Your patch looks good to me: probably preferable in th

Re: [FFmpeg-devel] [PATCH v1] lavc/vvc: Validate num_signalled_palette_entries

2025-05-17 Thread Nuo Mi
Hi Frank, 👍,your fuzzing infrastructure caught this issue as well. How about this: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20250517055150.807683-1-nuomi2...@gmail.com/ On Sun, May 18, 2025 at 5:05 AM Frank Plowman wrote: > "The value of CurrentPaletteSize[ startComp ] shall be in the r

[FFmpeg-devel] [PATCH v1] lavc/vvc: Validate num_signalled_palette_entries

2025-05-17 Thread Frank Plowman
"The value of CurrentPaletteSize[ startComp ] shall be in the range of 0 to maxNumPaletteEntries, inclusive." Signed-off-by: Frank Plowman --- libavcodec/vvc/ctu.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/libavcodec/vvc/ctu.c b/libavcodec/vvc/ctu.c inde