mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-12-28 20:53:54 +02:00
Merge commit '94603feb1b3ad01a821a1a1cef1570b13f471821'
* commit '94603feb1b3ad01a821a1a1cef1570b13f471821': h264_ps: when parsing a VUI fails, only abort when explode is set Merge only for metadata, ignoring invalid data can put the context into an invalid state and can possibly be used for remote code execution exploits. Also we support all non standard VUIs that have been reported so ignoring parsing failure makes no sense for us unless theres some issue that has not been reported Merged-by: Michael Niedermayer <michaelni@gmx.at>
This commit is contained in:
commit
ac3fa95e73
@ -520,9 +520,11 @@ int ff_h264_decode_seq_parameter_set(H264Context *h)
|
||||
}
|
||||
|
||||
sps->vui_parameters_present_flag = get_bits1(&h->gb);
|
||||
if (sps->vui_parameters_present_flag)
|
||||
if (decode_vui_parameters(h, sps) < 0)
|
||||
if (sps->vui_parameters_present_flag) {
|
||||
int ret = decode_vui_parameters(h, sps);
|
||||
if (ret < 0)
|
||||
goto fail;
|
||||
}
|
||||
|
||||
if (!sps->sar.den)
|
||||
sps->sar.den = 1;
|
||||
|
Loading…
Reference in New Issue
Block a user