Mercurial > hg > nginx-vendor-current
diff src/http/ngx_http_parse.c @ 472:09f0ef15d544 NGINX_0_7_48
nginx 0.7.48
*) Feature: the "proxy_cache_key" directive.
*) Bugfix: now nginx takes into account the "X-Accel-Expires",
"Expires", and "Cache-Control" header lines in a backend response.
*) Bugfix: now nginx caches responses for the GET requests only.
*) Bugfix: the "fastcgi_cache_key" directive was not inherited.
*) Bugfix: the $arg_... variables did not work with SSI subrequests.
Thanks to Maxim Dounin.
*) Bugfix: nginx could not be built with uclibc library.
Thanks to Timothy Redaelli.
*) Bugfix: nginx could not be built on OpenBSD; the bug had
appeared in 0.7.46.
author | Igor Sysoev <http://sysoev.ru> |
---|---|
date | Mon, 06 Apr 2009 00:00:00 +0400 |
parents | 6866b490272e |
children | 392c16f2d858 |
line wrap: on
line diff
--- a/src/http/ngx_http_parse.c +++ b/src/http/ngx_http_parse.c @@ -1486,20 +1486,20 @@ ngx_http_parse_multi_header_lines(ngx_ar ngx_int_t ngx_http_arg(ngx_http_request_t *r, u_char *name, size_t len, ngx_str_t *value) { - u_char *p; + u_char *p, *last; if (r->args.len == 0) { return NGX_DECLINED; } - for (p = r->args.data; *p && *p != ' '; p++) { + p = r->args.data; + last = p + r->args.len; - /* - * although r->args.data is not null-terminated by itself, - * however, there is null in the end of request line - */ + for ( /* void */ ; p < last; p++) { - p = ngx_strcasestrn(p, (char *) name, len - 1); + /* we need '=' after name, so drop one char from last */ + + p = ngx_strlcasestrn(p, last - 1, name, len - 1); if (p == NULL) { return NGX_DECLINED; @@ -1509,7 +1509,7 @@ ngx_http_arg(ngx_http_request_t *r, u_ch value->data = p + len + 1; - p = (u_char *) ngx_strchr(p, '&'); + p = ngx_strlchr(p, last, '&'); if (p == NULL) { p = r->args.data + r->args.len;