[PATCH] SSL: fixed $ssl_curves allocation error handling
Maxim Dounin
mdounin at mdounin.ru
Tue Jan 30 03:28:32 UTC 2024
Hello!
On Fri, Jan 26, 2024 at 02:36:29PM +0400, Sergey Kandaurov wrote:
> # HG changeset patch
> # User Sergey Kandaurov <pluknet at nginx.com>
> # Date 1706265240 -14400
> # Fri Jan 26 14:34:00 2024 +0400
> # Node ID 2f70dd17c16461f833eafec2dcf9193557bfb176
> # Parent ee40e2b1d0833b46128a357fbc84c6e23be9be07
> SSL: fixed $ssl_curves allocation error handling.
>
> diff --git a/src/event/ngx_event_openssl.c b/src/event/ngx_event_openssl.c
> --- a/src/event/ngx_event_openssl.c
> +++ b/src/event/ngx_event_openssl.c
> @@ -5187,6 +5187,9 @@ ngx_ssl_get_curves(ngx_connection_t *c,
> }
>
> curves = ngx_palloc(pool, n * sizeof(int));
> + if (curves == NULL) {
> + return NGX_ERROR;
> + }
>
> n = SSL_get1_curves(c->ssl->connection, curves);
> len = 0;
Looks good.
--
Maxim Dounin
http://mdounin.ru/
More information about the nginx-devel
mailing list