diff mbox

[FFmpeg-devel,2/2] lavfi/framesync: remove an invalid free.

Message ID 20180102135814.13657-2-george@nsup.org
State Accepted
Commit 29b5f3115d9f217758bebd1d00e541aa3e739d2a
Headers show

Commit Message

Nicolas George Jan. 2, 2018, 1:58 p.m. UTC 
Signed-off-by: Nicolas George <george@nsup.org>
---
 libavfilter/framesync.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


Not actually experienced a problem, but similar to Derek's fix.

Comments

Paul B Mahol Jan. 3, 2018, 3:41 p.m. UTC | #1 
On 1/2/18, Nicolas George <george@nsup.org> wrote:
> Signed-off-by: Nicolas George <george@nsup.org>
> ---
>  libavfilter/framesync.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>
> Not actually experienced a problem, but similar to Derek's fix.
>
>

LGTM
diff mbox

Patch

diff --git a/libavfilter/framesync.c b/libavfilter/framesync.c
index 82d715750c..da12c58a61 100644
--- a/libavfilter/framesync.c
+++ b/libavfilter/framesync.c
@@ -406,7 +406,7 @@  int ff_framesync_dualinput_get_writable(FFFrameSync *fs, AVFrame **f0, AVFrame *
     ret = ff_inlink_make_frame_writable(fs->parent->inputs[0], f0);
     if (ret < 0) {
         av_frame_free(f0);
-        av_frame_free(f1);
+        *f1 = NULL;
         return ret;
     }
     return 0;