[PATCH] ustream-ssl: openssl: fix bio memory leak

[Rosen Penev]

On Sun, Nov 1, 2020 at 6:13 PM <serial115200 at outlook.com> wrote:
>
> From: Pan Chen <serial115200 at outlook.com>
>
> free memory of bio method when ustream be freed
>
> Signed-off-by: Pan Chen <serial115200 at outlook.com>
> ---
>  openssl_bio_compat.h | 7 +++++++
>  ustream-io-openssl.c | 1 +
>  2 files changed, 8 insertions(+)
>
> diff --git a/openssl_bio_compat.h b/openssl_bio_compat.h
> index 9355c86..39a0455 100644
> --- a/openssl_bio_compat.h
> +++ b/openssl_bio_compat.h
> @@ -28,6 +28,13 @@ static inline BIO_METHOD *BIO_meth_new(int type, const char *name)
>         return bm;
>  }
>
> +static inline void BIO_meth_free(BIO_METHOD *bm)
> +{
> +       if (bm != NULL) {
> +               free(bm);
> +       }
> +}
> +
>  #endif /* OPENSSL_VERSION_NUMBER */
>
>  #endif /* OPENSSL_BIO_COMPAT_H */
> diff --git a/ustream-io-openssl.c b/ustream-io-openssl.c
> index 606ed4a..c9d7fae 100644
> --- a/ustream-io-openssl.c
> +++ b/ustream-io-openssl.c
> @@ -42,6 +42,7 @@ s_ustream_free(BIO *b)
>         BIO_set_data(b, NULL);
>         BIO_set_init(b, 0);
>         BIO_clear_flags(b, ~0);
> +       BIO_meth_free(b->method);
Is this needed? AFAIK, OpenSSL 1.1 frees everything automatically.
>         return 1;
>  }
>
> --
> 2.24.1.windows.2
>
>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel