Welcome! Log In Create A New Profile

Advanced

Re: Fix buffer size comparison for the case header_start + 256 has same value to buffer_size (ticket #17

Maxim Dounin
February 12, 2019 02:44PM
Hello!

On Tue, Feb 12, 2019 at 11:38:41AM +0900, 정찬훈 wrote:

> # HG changeset patch
> # User Chanhun Jeong <chanhun.jeong@navercorp.com>
> # Date 1549859813 -32400
> # Mon Feb 11 13:36:53 2019 +0900
> # Node ID f2648ca4ed03a907e6d3c22729be82b3bc4027a0
> # Parent 992bf7540a980c48a37c305d2c511cf9a48fe701
> Fix buffer size comparison for the case header_start + 256 has same value to buffer_size (ticket #1722).
> ​
> diff -r 992bf7540a98 -r f2648ca4ed03 src/http/ngx_http_upstream.c
> --- a/src/http/ngx_http_upstream.c Thu Feb 07 19:39:35 2019 +0300
> +++ b/src/http/ngx_http_upstream.c Mon Feb 11 13:36:53 2019 +0900
> @@ -850,7 +850,7 @@
> ​
> ngx_http_file_cache_create_key(r);
> ​
> - if (r->cache->header_start + 256 >= u->conf->buffer_size) {
> + if (r->cache->header_start + 256 > u->conf->buffer_size) {
> ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
> "%V_buffer_size %uz is not enough for cache key, "
> "it should be increased to at least %uz",
> ​
> ​

Committed (with a fixed commit message), thanks.

--
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

Fix buffer size comparison for the case header_start + 256 has same value to buffer_size (ticket #17

정찬훈 282 February 11, 2019 09:40PM

Re: Fix buffer size comparison for the case header_start + 256 has same value to buffer_size (ticket #17

Maxim Dounin 133 February 12, 2019 02:44PM



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

Online Users

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