[FFmpeg-devel] [PATCH 1/2] avcodec/cbs_av1: fix storage size for segmentation_params feature_value fields

[Mark Thompson]

On 11/11/18 02:24, James Almer wrote:
> The valid range is -255 to 255.
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavcodec/cbs_av1.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/cbs_av1.h b/libavcodec/cbs_av1.h
> index f662265f75..84622ed189 100644
> --- a/libavcodec/cbs_av1.h
> +++ b/libavcodec/cbs_av1.h
> @@ -210,7 +210,7 @@ typedef struct AV1RawFrameHeader {
>      uint8_t segmentation_temporal_update;
>      uint8_t segmentation_update_data;
>      uint8_t feature_enabled[AV1_MAX_SEGMENTS][AV1_SEG_LVL_MAX];
> -    uint8_t feature_value[AV1_MAX_SEGMENTS][AV1_SEG_LVL_MAX];
> +    int16_t feature_value[AV1_MAX_SEGMENTS][AV1_SEG_LVL_MAX];
>  
>      uint8_t delta_q_present;
>      uint8_t delta_q_res;
> 

Yep, LGTM.

Thanks,

- Mark