[FFmpeg-cvslog] fftools/ffmpeg_enc: only promote first frame side data to global when meaningful

Anton Khirnov git at videolan.org
Wed Mar 27 12:41:51 EET 2024


ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Sat Mar 23 13:50:46 2024 +0100| [fabf148578c0d628a9207de3b60aba57ac1c4bc3] | committer: Anton Khirnov

fftools/ffmpeg_enc: only promote first frame side data to global when meaningful

Skip those side data types that do not make sense as global side data.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=fabf148578c0d628a9207de3b60aba57ac1c4bc3
---

 fftools/ffmpeg_enc.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/fftools/ffmpeg_enc.c b/fftools/ffmpeg_enc.c
index 5f7fcf8a5f..bcd88c7d3e 100644
--- a/fftools/ffmpeg_enc.c
+++ b/fftools/ffmpeg_enc.c
@@ -247,6 +247,11 @@ int enc_open(void *opaque, const AVFrame *frame)
         enc_ctx->chroma_sample_location = frame->chroma_location;
 
         for (int i = 0; i < frame->nb_side_data; i++) {
+            const AVSideDataDescriptor *desc = av_frame_side_data_desc(frame->side_data[i]->type);
+
+            if (!(desc->props & AV_SIDE_DATA_PROP_GLOBAL))
+                continue;
+
             ret = av_frame_side_data_clone(
                 &enc_ctx->decoded_side_data, &enc_ctx->nb_decoded_side_data,
                 frame->side_data[i], AV_FRAME_SIDE_DATA_FLAG_UNIQUE);



More information about the ffmpeg-cvslog mailing list