Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] SSL: fixed $ssl_curves allocation error handling

Maxim Dounin
January 29, 2024 10:30PM
Hello!

On Fri, Jan 26, 2024 at 02:36:29PM +0400, Sergey Kandaurov wrote:

> # HG changeset patch
> # User Sergey Kandaurov <pluknet@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/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] SSL: fixed $ssl_curves allocation error handling

Sergey Kandaurov 130 January 26, 2024 05:38AM

Re: [PATCH] SSL: fixed $ssl_curves allocation error handling

Maxim Dounin 20 January 29, 2024 10:30PM



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

Online Users

Guests: 247
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready