[FFmpeg-cvslog] avfilter/vf_copy: check for error cases and handle them

Paul B Mahol git at videolan.org
Tue Oct 1 14:47:43 EEST 2019


ffmpeg | branch: master | Paul B Mahol <onemda at gmail.com> | Tue Oct  1 13:45:29 2019 +0200| [a9500441a71805da13389d4a56188c90f6bc013e] | committer: Paul B Mahol

avfilter/vf_copy: check for error cases and handle them

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a9500441a71805da13389d4a56188c90f6bc013e
---

 libavfilter/vf_copy.c | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/libavfilter/vf_copy.c b/libavfilter/vf_copy.c
index b0159cff00..a59e024e87 100644
--- a/libavfilter/vf_copy.c
+++ b/libavfilter/vf_copy.c
@@ -48,15 +48,23 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
 {
     AVFilterLink *outlink = inlink->dst->outputs[0];
     AVFrame *out = ff_get_video_buffer(outlink, in->width, in->height);
+    int ret;
 
-    if (!out) {
-        av_frame_free(&in);
-        return AVERROR(ENOMEM);
-    }
-    av_frame_copy_props(out, in);
-    av_frame_copy(out, in);
+    if (!out)
+        ret = AVERROR(ENOMEM);
+
+    ret = av_frame_copy_props(out, in);
+    if (ret < 0)
+        goto fail;
+    ret = av_frame_copy(out, in);
+    if (ret < 0)
+        goto fail;
     av_frame_free(&in);
     return ff_filter_frame(outlink, out);
+fail:
+    av_frame_free(&in);
+    av_frame_free(&out);
+    return ret;
 }
 
 static const AVFilterPad avfilter_vf_copy_inputs[] = {



More information about the ffmpeg-cvslog mailing list