[FFmpeg-devel] [PATCH] avutil/downmix_info: check for side data allocation success before zeroing it

James Almer jamrial at gmail.com
Tue Jan 28 19:49:11 EET 2025


Fixes coverity #1641638.

Signed-off-by: James Almer <jamrial at gmail.com>
---
 libavutil/downmix_info.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavutil/downmix_info.c b/libavutil/downmix_info.c
index feaee28493..7e6c3e854d 100644
--- a/libavutil/downmix_info.c
+++ b/libavutil/downmix_info.c
@@ -30,7 +30,8 @@ AVDownmixInfo *av_downmix_info_update_side_data(AVFrame *frame)
     if (!side_data) {
         side_data = av_frame_new_side_data(frame, AV_FRAME_DATA_DOWNMIX_INFO,
                                            sizeof(AVDownmixInfo));
-        memset(side_data->data, 0, sizeof(AVDownmixInfo));
+        if (side_data)
+            memset(side_data->data, 0, sizeof(AVDownmixInfo));
     }
 
     if (!side_data)
-- 
2.48.1



More information about the ffmpeg-devel mailing list