LGTM On 1/1/20, Jun Zhao <[email protected]> wrote: > From: Jun Zhao <[email protected]> > > ff_filter_frame always free the frame in case of error, so we don't > need to free the frame after ff_filter_frame fail. > > Signed-off-by: Jun Zhao <[email protected]> > --- > libavfilter/buffersrc.c | 4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > > diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c > index 64940d9..bf30f54 100644 > --- a/libavfilter/buffersrc.c > +++ b/libavfilter/buffersrc.c > @@ -240,10 +240,8 @@ static int > av_buffersrc_add_frame_internal(AVFilterContext *ctx, > } > > ret = ff_filter_frame(ctx->outputs[0], copy); > - if (ret < 0) { > - av_frame_free(©); > + if (ret < 0) > return ret; > - } > > if ((flags & AV_BUFFERSRC_FLAG_PUSH)) { > ret = push_frame(ctx->graph); > -- > 1.7.1 > > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
