avformat/mov: don't parse reserved ISOBMFF fields as if they were QT

Signed-off-by: James Almer <jamrial@gmail.com>
(cherry picked from commit c0044ec9c4)
This commit is contained in:
James Almer 2025-11-21 16:58:13 -03:00
parent 5acd4d17cc
commit 19df26bc82
10 changed files with 18 additions and 18 deletions

View file

@ -2676,12 +2676,18 @@ static void mov_parse_stsd_video(MOVContext *c, AVIOContext *pb,
* read in ff_mov_read_stsd_entries() */
stsd_start = avio_tell(pb) - 16;
if (c->isom) {
avio_skip(pb, 2); /* pre_defined */
avio_skip(pb, 2); /* reserved */
avio_skip(pb, 12); /* pre_defined */
} else {
avio_rb16(pb); /* version */
avio_rb16(pb); /* revision level */
id = avio_rl32(pb); /* vendor */
av_dict_set(&st->metadata, "vendor_id", av_fourcc2str(id), 0);
avio_rb32(pb); /* temporal quality */
avio_rb32(pb); /* spatial quality */
}
st->codecpar->width = avio_rb16(pb); /* width */
st->codecpar->height = avio_rb16(pb); /* height */
@ -2731,9 +2737,13 @@ static void mov_parse_stsd_audio(MOVContext *c, AVIOContext *pb,
AVDictionaryEntry *compatible_brands = av_dict_get(c->fc->metadata, "compatible_brands", NULL, AV_DICT_MATCH_CASE);
int channel_count;
if (c->isom)
avio_skip(pb, 6); /* reserved */
else {
avio_rb16(pb); /* revision level */
id = avio_rl32(pb); /* vendor */
av_dict_set(&st->metadata, "vendor_id", av_fourcc2str(id), 0);
}
channel_count = avio_rb16(pb);

View file

@ -1,5 +1,5 @@
92fe70291f72acf94ba56b426bbaccb0 *tests/data/fate/h264_redundant_pps-side_data.nut
596100 tests/data/fate/h264_redundant_pps-side_data.nut
18c64ec6b4f0cc39c5f8f3564b372fef *tests/data/fate/h264_redundant_pps-side_data.nut
596084 tests/data/fate/h264_redundant_pps-side_data.nut
#extradata 0: 34, 0x850408e3
#tb 0: 1/48000
#media_type 0: video

View file

@ -1,5 +1,5 @@
dd953f8d95d2927703ce9593a07fe2e7 *tests/data/fate/h264_redundant_pps-side_data2.nut
5162 tests/data/fate/h264_redundant_pps-side_data2.nut
f94ed2c25b6bbe63160743f08de33665 *tests/data/fate/h264_redundant_pps-side_data2.nut
5138 tests/data/fate/h264_redundant_pps-side_data2.nut
#tb 0: 1/25
#media_type 0: video
#codec_id 0: rawvideo

View file

@ -1,5 +1,5 @@
90c54a00ad8662c3eb93150791fa8328 *tests/data/fate/matroska-alac-remux.matroska
1293824 tests/data/fate/matroska-alac-remux.matroska
6075eb35f53692596194f3b0175cb184 *tests/data/fate/matroska-alac-remux.matroska
1293794 tests/data/fate/matroska-alac-remux.matroska
#extradata 0: 36, 0x562b05d8
#tb 0: 1/1000
#media_type 0: audio

View file

@ -1,5 +1,5 @@
7adef53df9e14358e0b99f8a829e2d97 *tests/data/fate/matroska-dovi-write-config7.matroska
72700 tests/data/fate/matroska-dovi-write-config7.matroska
adafbb4d021db027f4ae4ef7ca1c56c2 *tests/data/fate/matroska-dovi-write-config7.matroska
72640 tests/data/fate/matroska-dovi-write-config7.matroska
#extradata 0: 116, 0x2b8d1669
#extradata 1: 116, 0x2b8d1669
#tb 0: 1/1000

View file

@ -160,7 +160,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x0
@ -395,7 +394,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x0

View file

@ -207,7 +207,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=1
id=0x2
@ -465,7 +464,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=1
id=0x2

View file

@ -207,7 +207,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x9
@ -465,7 +464,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x9

View file

@ -99,7 +99,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x0
@ -264,7 +263,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x0

View file

@ -52,7 +52,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x0
@ -187,7 +186,6 @@ DISPOSITION:still_image=0
DISPOSITION:multilayer=0
TAG:language=und
TAG:handler_name=SoundHandler
TAG:vendor_id=[0][0][0][0]
[STREAM]
index=0
id=0x0