[FFmpeg-devel] [PATCH] movenc: Check for more options incompatible with hybrid_fragmented (PR #20651)

[FFmpeg-devel] [PATCH] movenc: Check for more options incompatible with hybrid_fragmented (PR #20651)

[Martin Storsjö via ffmpeg-devel]

PR #20651 opened by Martin Storsjö (mstorsjo)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20651
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20651.patch

This follows up on c79bce92b8407199b6e70205393411650e4342ca with
more options that don't work together with hybrid fragmentation.


From 36f0efda74f306465c7bc10cadc1ad0e81909aa9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?= <martin@martin.st>
Date: Fri, 27 Jun 2025 14:53:02 +0300
Subject: [PATCH] movenc: Check for more options incompatible with
 hybrid_fragmented

This follows up on c79bce92b8407199b6e70205393411650e4342ca with
more options that don't work together with hybrid fragmentation.
---
 libavformat/movenc.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index 067d38b14b..f586a442b2 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -7858,8 +7858,10 @@ static int mov_init(AVFormatContext *s)
         mov->flags |= FF_MOV_FLAG_FRAGMENT;
 
     if (mov->flags & FF_MOV_FLAG_HYBRID_FRAGMENTED &&
-        mov->flags & FF_MOV_FLAG_FASTSTART) {
-        av_log(s, AV_LOG_ERROR, "Setting both hybrid_fragmented and faststart is not supported.\n");
+        (mov->flags & (FF_MOV_FLAG_FASTSTART | FF_MOV_FLAG_DELAY_MOOV) ||
+         mov->reserved_moov_size > 0)) {
+        av_log(s, AV_LOG_ERROR, "The hybrid_fragmented mode is incompatible "
+                                "with faststart, delay_moov and moov_size.\n");
         return AVERROR(EINVAL);
     }
 
-- 
2.49.1

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org