Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Validate presence of upstream pipe in ngx_http_upstream_finalize_request

Maxim Dounin
December 04, 2017 09:24AM
Hello!

On Fri, Dec 01, 2017 at 04:26:55PM -0800, Robert Paprocki wrote:

> # HG changeset patch
> # User Robert Paprocki <robert@cryptobells.com>
> # Date 1512174351 28800
> # Fri Dec 01 16:25:51 2017 -0800
> # Node ID a200e7c746dc7853b051f9b357a5b9c24d6b0dd0
> # Parent fc0d06224edac2c7cfbfd9a4def478f285d9957b
> Validate presence of upstream pipe in ngx_http_upstream_finalize_request
>
> diff -r fc0d06224eda -r a200e7c746dc src/http/ngx_http_upstream.c
> --- a/src/http/ngx_http_upstream.c Tue Nov 28 13:09:54 2017 +0300
> +++ b/src/http/ngx_http_upstream.c Fri Dec 01 16:25:51 2017 -0800
> @@ -4353,7 +4353,9 @@
> }
> }
>
> - ngx_http_file_cache_free(r->cache, u->pipe->temp_file);
> + if (u->pipe) {
> + ngx_http_file_cache_free(r->cache, u->pipe->temp_file);
> + }
> }
>
> #endif

No, thanks. See detailed explanation here:

http://mailman.nginx.org/pipermail/nginx-devel/2017-December/010665.html

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

[PATCH] Validate presence of upstream pipe in ngx_http_upstream_finalize_request

Robert Paprocki 565 December 01, 2017 07:28PM

Re: [PATCH] Validate presence of upstream pipe in ngx_http_upstream_finalize_request

Maxim Dounin 189 December 04, 2017 09:24AM



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

Online Users

Guests: 289
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