diff options
| author | Maxim Dounin <mdounin@mdounin.ru> | 2019-03-03 16:48:39 +0300 |
|---|---|---|
| committer | Maxim Dounin <mdounin@mdounin.ru> | 2019-03-03 16:48:39 +0300 |
| commit | fe43346dc3151e80dae0acd751f0a94314dcb91c (patch) | |
| tree | 8da39450985963593478b57609232c43a4be4159 /src/http/modules/ngx_http_uwsgi_module.c | |
| parent | 99d7bb690924e60e9e03096ac5e507111f7c182d (diff) | |
| download | nginx-fe43346dc3151e80dae0acd751f0a94314dcb91c.tar.gz nginx-fe43346dc3151e80dae0acd751f0a94314dcb91c.zip | |
SSL: fixed potential leak on memory allocation errors.
If ngx_pool_cleanup_add() fails, we have to clean just created SSL context
manually, thus appropriate call added.
Additionally, ngx_pool_cleanup_add() moved closer to ngx_ssl_create() in
the ngx_http_ssl_module, to make sure there are no leaks due to intermediate
code.
Diffstat (limited to 'src/http/modules/ngx_http_uwsgi_module.c')
| -rw-r--r-- | src/http/modules/ngx_http_uwsgi_module.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/http/modules/ngx_http_uwsgi_module.c b/src/http/modules/ngx_http_uwsgi_module.c index 8b091101f..56dc236ef 100644 --- a/src/http/modules/ngx_http_uwsgi_module.c +++ b/src/http/modules/ngx_http_uwsgi_module.c @@ -2359,6 +2359,7 @@ ngx_http_uwsgi_set_ssl(ngx_conf_t *cf, ngx_http_uwsgi_loc_conf_t *uwcf) cln = ngx_pool_cleanup_add(cf->pool, 0); if (cln == NULL) { + ngx_ssl_cleanup_ctx(uwcf->upstream.ssl); return NGX_ERROR; } |