[FFmpeg-devel] [PATCH] avformat/mov: don't unnecesarely replace extradata with the same one

[James Almer]

If the first STSC entry refers to the first extradata buffer, then don't
bother replacing the output stream's extradata as it's already set to it.

Signed-off-by: James Almer <jamrial at gmail.com>
---
 libavformat/mov.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 7f88ec8220..a3e1aa2933 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -10667,7 +10667,7 @@ static int mov_read_header(AVFormatContext *s)
         fix_timescale(mov, sc);
 
         /* Set the primary extradata based on the first Sample. */
-        if (sc->stsc_count && sc->extradata_size && !sc->iamf) {
+        if (sc->stsc_count && sc->extradata_size && sc->stsc_data[0].id != 1 && !sc->iamf) {
             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.50.1