[FFmpeg-devel] [PATCH] avcodec/hevcdec: don't store stale HDR10+ values

James Almer jamrial at gmail.com
Tue Nov 14 17:31:37 EET 2023


This metadata is signaled per frame. If not present in an AU, then the previous
values should not be used.

Should fix ticket #10541.

Signed-off-by: James Almer <jamrial at gmail.com>
---
 libavcodec/hevcdec.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
index 3ce845dddb..13c37e1e3b 100644
--- a/libavcodec/hevcdec.c
+++ b/libavcodec/hevcdec.c
@@ -2805,14 +2805,12 @@ static int set_side_data(HEVCContext *s)
     }
 
     if (s->sei.common.dynamic_hdr_plus.info) {
-        AVBufferRef *info_ref = av_buffer_ref(s->sei.common.dynamic_hdr_plus.info);
-        if (!info_ref)
-            return AVERROR(ENOMEM);
-
-        if (!av_frame_new_side_data_from_buf(out, AV_FRAME_DATA_DYNAMIC_HDR_PLUS, info_ref)) {
-            av_buffer_unref(&info_ref);
+        if (!av_frame_new_side_data_from_buf(out, AV_FRAME_DATA_DYNAMIC_HDR_PLUS,
+                                             s->sei.common.dynamic_hdr_plus.info)) {
+            av_buffer_unref(&s->sei.common.dynamic_hdr_plus.info);
             return AVERROR(ENOMEM);
         }
+        s->sei.common.dynamic_hdr_plus.info = NULL;
     }
 
     if (s->rpu_buf) {
-- 
2.42.1



More information about the ffmpeg-devel mailing list