[FFmpeg-cvslog] lavu/frame: fix malloc error path in av_frame_copy_props()
wm4
git at videolan.org
Mon Dec 15 05:32:17 CET 2014
ffmpeg | branch: master | wm4 <nfxjfg at googlemail.com> | Mon Dec 15 04:32:58 2014 +0100| [a400edbb6d00c0211de38e4f1b4f593681db91d8] | committer: Michael Niedermayer
lavu/frame: fix malloc error path in av_frame_copy_props()
The error path frees all side data, but forgets to reset the side data
count. This can blow up later in av_frame_unref() and free_side_data().
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a400edbb6d00c0211de38e4f1b4f593681db91d8
---
libavutil/frame.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavutil/frame.c b/libavutil/frame.c
index 4ee0630..5c9aa29 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -503,6 +503,7 @@ int av_frame_copy_props(AVFrame *dst, const AVFrame *src)
free_side_data(&dst->side_data[i]);
}
av_freep(&dst->side_data);
+ dst->nb_side_data = 0;
return AVERROR(ENOMEM);
}
memcpy(sd_dst->data, sd_src->data, sd_src->size);
More information about the ffmpeg-cvslog
mailing list