[FFmpeg-devel] [PATCH 25/29] avcodec/snowdec: use ff_frame_new_side_data

Anton Khirnov anton at khirnov.net
Mon Mar 4 15:06:40 EET 2024


From: Niklas Haas <git at haasn.dev>

For consistency, even though this cannot be overriden at the packet
level.
---
 libavcodec/snowdec.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/libavcodec/snowdec.c b/libavcodec/snowdec.c
index 70fbab9a49..97aea748b6 100644
--- a/libavcodec/snowdec.c
+++ b/libavcodec/snowdec.c
@@ -787,11 +787,10 @@ static int decode_frame(AVCodecContext *avctx, AVFrame *picture,
         res = av_frame_ref(picture, s->mconly_picture);
     if (res >= 0 && s->avmv_index) {
         AVFrameSideData *sd;
-
-        sd = av_frame_new_side_data(picture, AV_FRAME_DATA_MOTION_VECTORS, s->avmv_index * sizeof(AVMotionVector));
-        if (!sd)
-            return AVERROR(ENOMEM);
-        memcpy(sd->data, s->avmv, s->avmv_index * sizeof(AVMotionVector));
+        res = ff_frame_new_side_data(s->avctx, picture, AV_FRAME_DATA_MOTION_VECTORS,
+                                     s->avmv_index * sizeof(AVMotionVector), &sd);
+        if (sd)
+            memcpy(sd->data, s->avmv, s->avmv_index * sizeof(AVMotionVector));
     }
 
     if (res < 0)
-- 
2.43.0



More information about the ffmpeg-devel mailing list