Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] avformat/mov: set primary extradata based on the first Sample only if it's not already in place (PR #20098)
@ 2025-08-03 16:54 James Almer
  0 siblings, 0 replies; only message in thread
From: James Almer @ 2025-08-03 16:54 UTC (permalink / raw)
  To: ffmpeg-devel

PR #20098 opened by James Almer (jamrial)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20098
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20098.patch

If the first Sample references the first stsd entry, then setting it here is redundant.

>From 925d0d2c21c97f9cae7b69409964adb6ed6af255 Mon Sep 17 00:00:00 2001
From: James Almer <jamrial@gmail.com>
Date: Sun, 3 Aug 2025 13:30:42 -0300
Subject: [PATCH] avformat/mov: set primary extradata based on the first Sample
 only if it's not already in place

If the first Sample references the first stsd entry, then setting it here is
redundant.

Signed-off-by: James Almer <jamrial@gmail.com>
---
 libavformat/mov.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 40105a408d..eb96136550 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -10685,9 +10685,9 @@ static int mov_read_header(AVFormatContext *s)
         uint32_t dvdsub_clut[FF_DVDCLUT_CLUT_LEN] = {0};
         fix_timescale(mov, sc);
 
-        /* Set the primary extradata based on the first Sample. */
+        /* Set the primary extradata based on the first Sample if it doesn't reference the first stsd entry. */
         if (sc->stsc_count && sc->extradata_size && !sc->iamf &&
-            sc->stsc_data[0].id > 0 && sc->stsc_data[0].id <= sc->stsd_count) {
+            sc->stsc_data[0].id > 1 && sc->stsc_data[0].id <= sc->stsd_count) {
             sc->last_stsd_index = sc->stsc_data[0].id - 1;
             av_freep(&st->codecpar->extradata);
             st->codecpar->extradata_size = sc->extradata_size[sc->last_stsd_index];
-- 
2.49.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2025-08-03 16:54 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-03 16:54 [FFmpeg-devel] [PATCH] avformat/mov: set primary extradata based on the first Sample only if it's not already in place (PR #20098) James Almer

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git