diff mbox series

[FFmpeg-devel,v2] lavf/tls_mbedtls: fix resource leak

Message ID 1587370728-13725-1-git-send-email-mypopydev@gmail.com
State Accepted
Commit 36083450a4be9e3053f4254ca1f696b402dab8e0
Headers show
Series [FFmpeg-devel,v2] lavf/tls_mbedtls: fix resource leak | expand

Checks

Context Check Description
andriy/default pending
andriy/make success Make finished
andriy/make_fate success Make fate finished

Commit Message

Jun Zhao April 20, 2020, 8:18 a.m. UTC 
From: Jun Zhao <barryjzhao@tencent.com>

fix resource leak in mbedtls part.

fix #8614

Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
---
 libavformat/tls_mbedtls.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Jun Zhao April 22, 2020, 4:39 a.m. UTC | #1 
On Mon, Apr 20, 2020 at 4:24 PM Jun Zhao <mypopydev@gmail.com> wrote:
>
> From: Jun Zhao <barryjzhao@tencent.com>
>
> fix resource leak in mbedtls part.
>
> fix #8614
>
> Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
> ---
>  libavformat/tls_mbedtls.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavformat/tls_mbedtls.c b/libavformat/tls_mbedtls.c
> index 9b80a1e..965adf1 100644
> --- a/libavformat/tls_mbedtls.c
> +++ b/libavformat/tls_mbedtls.c
> @@ -62,6 +62,7 @@ static int tls_close(URLContext *h)
>      mbedtls_ctr_drbg_free(&tls_ctx->ctr_drbg_context);
>      mbedtls_entropy_free(&tls_ctx->entropy_context);
>
> +    ffurl_closep(&tls_ctx->tls_shared.tcp);
>      return 0;
>  }
>
> --
> 2.7.4
>
Will apply the patch, tks
diff mbox series

Patch

diff --git a/libavformat/tls_mbedtls.c b/libavformat/tls_mbedtls.c
index 9b80a1e..965adf1 100644
--- a/libavformat/tls_mbedtls.c
+++ b/libavformat/tls_mbedtls.c
@@ -62,6 +62,7 @@  static int tls_close(URLContext *h)
     mbedtls_ctr_drbg_free(&tls_ctx->ctr_drbg_context);
     mbedtls_entropy_free(&tls_ctx->entropy_context);
 
+    ffurl_closep(&tls_ctx->tls_shared.tcp);
     return 0;
 }