[SCM] libav/experimental: frame: fix the error path in av_frame_copy_props()

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Aug 10 16:03:18 UTC 2014


The following commit has been merged in the experimental branch:
commit a53551cba86bb67efcb6105fdc337a36c43132bd
Author: Roman Savchenko <gmstima at gmail.com>
Date:   Wed May 28 22:16:56 2014 +0200

    frame: fix the error path in av_frame_copy_props()
    
    First free metadata, then the side data it is contained in.
    
    Signed-off-by: Anton Khirnov <anton at khirnov.net>

diff --git a/libavutil/frame.c b/libavutil/frame.c
index cc4bfcd..9048552 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -383,8 +383,8 @@ int av_frame_copy_props(AVFrame *dst, const AVFrame *src)
         if (!sd_dst) {
             for (i = 0; i < dst->nb_side_data; i++) {
                 av_freep(&dst->side_data[i]->data);
-                av_freep(&dst->side_data[i]);
                 av_dict_free(&dst->side_data[i]->metadata);
+                av_freep(&dst->side_data[i]);
             }
             av_freep(&dst->side_data);
             return AVERROR(ENOMEM);

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list