[FFmpeg-devel] [PATCH v1] lavf/tls_mbedtls: fix resource leak
mypopy at gmail.com
mypopy at gmail.com
Mon Apr 20 10:46:51 EEST 2020
On Mon, Apr 20, 2020 at 3:05 PM Andreas Rheinhardt
<andreas.rheinhardt at gmail.com> wrote:
>
> Jun Zhao:
> > From: Jun Zhao <barryjzhao at tencent.com>
> >
> > fix resource leak in mbedtls part.
> >
> > fix #8614
> >
> > Signed-off-by: Jun Zhao <barryjzhao at tencent.com>
> > ---
> > libavformat/tls_mbedtls.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/libavformat/tls_mbedtls.c b/libavformat/tls_mbedtls.c
> > index 9b80a1e..eb4b2a0 100644
> > --- a/libavformat/tls_mbedtls.c
> > +++ b/libavformat/tls_mbedtls.c
> > @@ -62,6 +62,8 @@ static int tls_close(URLContext *h)
> > mbedtls_ctr_drbg_free(&tls_ctx->ctr_drbg_context);
> > mbedtls_entropy_free(&tls_ctx->entropy_context);
> >
> > + if (tls_ctx->tls_shared.tcp)
> > + ffurl_close(tls_ctx->tls_shared.tcp);
> > return 0;
> > }
> >
> > You can simply use ffurl_closep() instead of these two lines.
>
Yes, will update the code,tks
More information about the ffmpeg-devel
mailing list