PR #24361 opened by lowkey
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/24361
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/24361.patch

This is a fix for https://crbug.com/556081479 which details a case of HarmonyOS 
adding a self referencing `cdsc` track reference, leading to `AV_EF_EXPLODE` to 
cause the provided mp4 in the issue tracker to refuse to play. Testing on 
Quicktime and Firefox the mp4 seems to play fine.

Signed-off-by: Syed AbuTalib <[email protected]>


>From 818e5d965be955be8842ee3a4cdd7b43ab81661d Mon Sep 17 00:00:00 2001
From: Syed AbuTalib <[email protected]>
Date: Thu, 3 Sep 2026 20:57:47 +0000
Subject: [PATCH] avformat/mov: Treat missing cdsc as nonfatal

Downgrade the log level to a warning and continue demuxing instead of
returning an error.

Signed-off-by: Syed AbuTalib <[email protected]>
---
 libavformat/mov.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index f53ce693f8..1255de95e8 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -11381,10 +11381,7 @@ static int 
mov_parse_cdsc_and_rndr_streams(AVFormatContext *s)
 
             st_ref = mov_find_reference_track(s, st, tag->id, tag->nb_id, 0);
             if (!st_ref) {
-                int loglevel = (s->error_recognition & AV_EF_EXPLODE) ? 
AV_LOG_ERROR : AV_LOG_WARNING;
-                av_log(s, loglevel, "Failed to find referenced stream\n");
-                if (s->error_recognition & AV_EF_EXPLODE)
-                    return AVERROR_INVALIDDATA;
+                av_log(s, AV_LOG_WARNING, "Failed to find referenced 
stream\n");
                 continue;
             }
 
-- 
2.52.0

_______________________________________________
ffmpeg-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to