diff mbox series

[FFmpeg-devel,1/3] avfilter/vf_guided: Don't free frame we don't own

Message ID HE1PR0301MB2154885D07EB9BB81A06A5418F509@HE1PR0301MB2154.eurprd03.prod.outlook.com
State Accepted
Commit 618d186b8c2bdb42cdbc722be42ab9efb1f7cdd1
Headers show
Series [FFmpeg-devel,1/3] avfilter/vf_guided: Don't free frame we don't own | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Andreas Rheinhardt May 14, 2021, 10:18 a.m. UTC 
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavfilter/vf_guided.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Liu Steven May 14, 2021, 12:21 p.m. UTC | #1 
> 2021年5月14日 下午6:18，Andreas Rheinhardt <andreas.rheinhardt@outlook.com> 写道：
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> libavfilter/vf_guided.c | 1 -
> 1 file changed, 1 deletion(-)
> 
> diff --git a/libavfilter/vf_guided.c b/libavfilter/vf_guided.c
> index e7c689e7be..e10d397f2e 100644
> --- a/libavfilter/vf_guided.c
> +++ b/libavfilter/vf_guided.c
> @@ -329,7 +329,6 @@ static int process_frame(FFFrameSync *fs)
>     out_frame = ff_get_video_buffer(outlink, outlink->w, outlink->h);
>     if (!out_frame) {
>         av_frame_free(&main_frame);
> -        av_frame_free(&ref_frame);
>         return AVERROR(ENOMEM);
>     }
>     av_frame_copy_props(out_frame, main_frame);
> -- 
> 2.27.0
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
> 
Looks simply  after patchset, lgtm

Thanks

Steven Liu
diff mbox series

Patch

diff --git a/libavfilter/vf_guided.c b/libavfilter/vf_guided.c
index e7c689e7be..e10d397f2e 100644
--- a/libavfilter/vf_guided.c
+++ b/libavfilter/vf_guided.c
@@ -329,7 +329,6 @@  static int process_frame(FFFrameSync *fs)
     out_frame = ff_get_video_buffer(outlink, outlink->w, outlink->h);
     if (!out_frame) {
         av_frame_free(&main_frame);
-        av_frame_free(&ref_frame);
         return AVERROR(ENOMEM);
     }
     av_frame_copy_props(out_frame, main_frame);