summaryrefslogtreecommitdiffhomepage
path: root/src
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 /src
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.
Diffstat (limited to 'src')
-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) {