Welcome! Log In Create A New Profile

Advanced

[PATCH] SSL: fixed possible memory and file descriptor leak on HUP signal.

Piotr Sikora
September 18, 2013 07:56PM
# HG changeset patch
# User Piotr Sikora <piotr@cloudflare.com>
# Date 1379548290 25200
# Wed Sep 18 16:51:30 2013 -0700
# Node ID c0be8de389be2012875a19a812ebf3ccc66c147d
# Parent e8d24b6d7f7304df77ccde7fc8223434c91b5322
SSL: fixed possible memory and file descriptor leak on HUP signal.

The problem appeared in 386a06a22c40 (1.3.7).

Signed-off-by: Piotr Sikora <piotr@cloudflare.com>

diff -r e8d24b6d7f73 -r c0be8de389be src/event/ngx_event_openssl.c
--- a/src/event/ngx_event_openssl.c Wed Sep 18 18:53:26 2013 +0400
+++ b/src/event/ngx_event_openssl.c Wed Sep 18 16:51:30 2013 -0700
@@ -280,6 +280,8 @@ ngx_ssl_certificate(ngx_conf_t *cf, ngx_
{
ngx_ssl_error(NGX_LOG_EMERG, ssl->log, 0,
"SSL_CTX_set_ex_data() failed");
+ X509_free(x509);
+ BIO_free(bio);
return NGX_ERROR;
}

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] SSL: fixed possible memory and file descriptor leak on HUP signal.

Piotr Sikora 643 September 18, 2013 07:56PM

Re: [PATCH] SSL: fixed possible memory and file descriptor leak on HUP signal.

Maxim Dounin 288 September 19, 2013 09:28AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 158
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready