aboutsummaryrefslogtreecommitdiffstats
path: root/libavformat/mov.c
diff options
context:
space:
mode:
authorJames Almer <jamrial@gmail.com>2024-04-13 10:11:03 -0300
committerJames Almer <jamrial@gmail.com>2024-04-18 13:00:11 -0300
commit5b9db32ccc9426dc2aa37f21eee4c9efd1baf75b (patch)
tree0e4016a233e6311415918ae8a5b6fe2e32b7fc69 /libavformat/mov.c
parent6927457443e6dc49b785a5b9e7e946313a3f2694 (diff)
downloadffmpeg-5b9db32ccc9426dc2aa37f21eee4c9efd1baf75b.tar.gz
avformat/mov: ignore old infe box versions
Some files with no image items have them, and were working prior to the recent HEIF parsing overhaul. Ignore such boxes instead, to recover the old behavior. Fixes a regression since d9fed9df2a9e70c9375d3b2591db35c09303d369. Tested-by: Wu Jianhua <toqsxw@outlook.com> Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavformat/mov.c')
-rw-r--r--libavformat/mov.c13
1 files changed, 8 insertions, 5 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c
index 0c892b56c8..ecd29a7d08 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -8137,8 +8137,8 @@ static int mov_read_infe(MOVContext *c, AVIOContext *pb, MOVAtom atom, int idx)
size -= 4;
if (version < 2) {
- av_log(c->fc, AV_LOG_ERROR, "infe: version < 2 not supported\n");
- return AVERROR_PATCHWELCOME;
+ avpriv_report_missing_feature(c->fc, "infe version < 2");
+ return 1;
}
item_id = version > 2 ? avio_rb32(pb) : avio_rb16(pb);
@@ -8209,6 +8209,8 @@ static int mov_read_iinf(MOVContext *c, AVIOContext *pb, MOVAtom atom)
ret = mov_read_infe(c, pb, infe, i);
if (ret < 0)
return ret;
+ if (ret)
+ return 0;
}
c->found_iinf = 1;
@@ -9508,14 +9510,15 @@ static int mov_read_header(AVFormatContext *s)
av_log(s, AV_LOG_ERROR, "error reading header\n");
return err;
}
- } while ((pb->seekable & AVIO_SEEKABLE_NORMAL) && !mov->found_moov && !mov->found_iloc && !mov->moov_retry++);
- if (!mov->found_moov && !mov->found_iloc) {
+ } while ((pb->seekable & AVIO_SEEKABLE_NORMAL) &&
+ !mov->found_moov && (!mov->found_iloc || !mov->found_iinf) && !mov->moov_retry++);
+ if (!mov->found_moov && !mov->found_iloc && !mov->found_iinf) {
av_log(s, AV_LOG_ERROR, "moov atom not found\n");
return AVERROR_INVALIDDATA;
}
av_log(mov->fc, AV_LOG_TRACE, "on_parse_exit_offset=%"PRId64"\n", avio_tell(pb));
- if (mov->found_iloc) {
+ if (mov->found_iloc && mov->found_iinf) {
for (i = 0; i < mov->nb_heif_item; i++) {
HEIFItem *item = &mov->heif_item[i];
MOVStreamContext *sc;