summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorMaxim Dounin <mdounin@mdounin.ru>2012-03-22 10:43:33 +0000
committerMaxim Dounin <mdounin@mdounin.ru>2012-03-22 10:43:33 +0000
commit3333c6c7c868d01cd940a5044cd31eadaea434d5 (patch)
tree5121141e4ae83b792860ca9e253b704126b268dd
parentae11f08cadf7b3ae01556115efdbce272c0fedb7 (diff)
downloadnginx-3333c6c7c868d01cd940a5044cd31eadaea434d5.tar.gz
nginx-3333c6c7c868d01cd940a5044cd31eadaea434d5.tar.bz2
Fixed off-by-one in xslt parameter parsing.
The problem was introduced in 0.7.44 (r2589) during conversion to complex values. Previously string.len included space for terminating NUL, but with complex values it doesn't.
-rw-r--r--src/http/modules/ngx_http_xslt_filter_module.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/http/modules/ngx_http_xslt_filter_module.c b/src/http/modules/ngx_http_xslt_filter_module.c
index f86c30381..0fb59f288 100644
--- a/src/http/modules/ngx_http_xslt_filter_module.c
+++ b/src/http/modules/ngx_http_xslt_filter_module.c
@@ -585,7 +585,7 @@ ngx_http_xslt_params(ngx_http_request_t *r, ngx_http_xslt_filter_ctx_t *ctx,
"xslt filter param: \"%s\"", string.data);
p = string.data;
- last = string.data + string.len - 1;
+ last = string.data + string.len;
while (p && *p) {