Hi,
On 14-01-18 00:00, selva.n...@gmail.com wrote:
> From: Selva Nair <selva.n...@gmail.com>
>
> - RSA_meth_new allocates memory for the name string
> and must be released using RSA_meth_free
>
> Signed-off-by: Selva Nair <selva.n...@gmail.com>
> ---
> src/openvpn/ssl_openssl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/openvpn/ssl_openssl.c b/src/openvpn/ssl_openssl.c
> index 34c31b9..d6d9acf 100644
> --- a/src/openvpn/ssl_openssl.c
> +++ b/src/openvpn/ssl_openssl.c
> @@ -1144,7 +1144,7 @@ err:
> {
> if (rsa_meth)
> {
> - free(rsa_meth);
> + RSA_meth_free(rsa_meth);
> }
> }
> crypto_msg(M_FATAL, "Cannot enable SSL external private key capability");
>
Good find, seems we missed that in the transition to OpenSSL 1.1.
(I see we have the same pattern in cryptapi.c, but you fix that as part
of your cryptoapi.c patches.)
Acked-by: Steffan Karger <stef...@karger.me>
-Steffan
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
