[FFmpeg-devel] [PATCH] avcodec/cbs_jpeg: Fix size of huffman symbol table array

[Mark Thompson]

On 08/02/2022 09:41, Andreas Rheinhardt wrote:
> L[i] can be in the range of 0-255, see table B.5 of ITU T.81.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>   libavcodec/cbs_jpeg.h                 | 2 +-
>   libavcodec/cbs_jpeg_syntax_template.c | 4 ++--
>   2 files changed, 3 insertions(+), 3 deletions(-)

Do you have a valid file showing this?  Not all values are allowed.

I guess I must have written it, but I have no idea where 224 came from.  As far as I know the worst case is in AC tables: 10 category values * 16 run lengths + 2 special cases = 162 (which could indeed all be dumped in the same code length if you want to be pathological).

- Mark