[nginx] Limit req: don't truncate key value to 255 bytes.

Valentin Bartenev vbart at nginx.com
Wed Sep 17 08:08:18 UTC 2014


details:   http://hg.nginx.org/nginx/rev/cda4fcb9294c
branches:  
changeset: 5846:cda4fcb9294c
user:      Valentin Bartenev <vbart at nginx.com>
date:      Tue Sep 16 21:12:51 2014 +0400
description:
Limit req: don't truncate key value to 255 bytes.

While the module allows to use values up to 65535 bytes as a key,
that actually never worked properly.

diffstat:

 src/http/modules/ngx_http_limit_req_module.c |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r 40b24cfef18b -r cda4fcb9294c src/http/modules/ngx_http_limit_req_module.c
--- a/src/http/modules/ngx_http_limit_req_module.c	Wed Sep 17 12:04:47 2014 +0400
+++ b/src/http/modules/ngx_http_limit_req_module.c	Tue Sep 16 21:12:51 2014 +0400
@@ -461,7 +461,7 @@ ngx_http_limit_req_lookup(ngx_http_limit
 
     lr = (ngx_http_limit_req_node_t *) &node->color;
 
-    lr->len = (u_char) len;
+    lr->len = (u_short) len;
     lr->excess = 0;
 
     ngx_memcpy(lr->data, data, len);



More information about the nginx-devel mailing list