wrapper/libavc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Set correct defaults for video signal flags in VUI

Browse source 
When u1_video_signal_type_present_flag and
u1_colour_description_present_flag were set, we did write all
fields relating to video signal and color description into
the VUI, even if all those fields were uninitialized (left
at the default cleared zero, which for most fields are
invalid/reserved value zero).

Instead have these flags default to zero, making it clear
that the VUI doesn't (yet) contain correctly set values
for those fields.

Bug: 240470605
Change-Id: I86628cff190c8a40e9c6fd12d3fc8e06419ece7c
This commit is contained in:
Martin Storsjo 2022-01-27 15:41:02 +02:00
parent 91e3d65f9f
commit 8362f0939b
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
encoder/ih264e_api.c
View file

@ -2938,8 +2938,8 @@ static WORD32 ih264e_set_default_params(cfg_params_t *ps_cfg)
ps_cfg->u4_constrained_intra_pred = 0;
ps_cfg->u4_pic_info_type = 0;
ps_cfg->u4_mb_info_type = 0;
ps_cfg->s_vui.u1_video_signal_type_present_flag = 1;
ps_cfg->s_vui.u1_colour_description_present_flag = 1;
ps_cfg->s_vui.u1_video_signal_type_present_flag = 0;
ps_cfg->s_vui.u1_colour_description_present_flag = 0;
return ret;
}