[FFmpeg-devel] [PATCH 1/2] avcodec/cbs_vp9: fix parsing sRGB samples

James Almer jamrial at gmail.com
Sat Oct 27 22:14:57 EEST 2018


On 10/26/2018 6:56 PM, Hendrik Leppkes wrote:
> On Fri, Oct 26, 2018 at 9:37 PM James Almer <jamrial at gmail.com> wrote:
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/cbs_vp9_syntax_template.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/libavcodec/cbs_vp9_syntax_template.c b/libavcodec/cbs_vp9_syntax_template.c
>> index 0db0f52a6d..b4a7f65e85 100644
>> --- a/libavcodec/cbs_vp9_syntax_template.c
>> +++ b/libavcodec/cbs_vp9_syntax_template.c
>> @@ -65,6 +65,7 @@ static int FUNC(color_config)(CodedBitstreamContext *ctx, RWContext *rw,
>>          if (profile == 1 || profile == 3) {
>>              infer(subsampling_x, 0);
>>              infer(subsampling_y, 0);
>> +            f(1, color_config_reserved_zero);
>>          }
>>      }
>>
> 
> What an odd quirk in the bitstream, the entire RGB branch has no coded
> values but that single reserved one?
> LGTM.
> 
> - Hendrik

The RGB branch is weird in AV1 as well :p

Pushed, thanks.


More information about the ffmpeg-devel mailing list