aboutsummaryrefslogtreecommitdiff
path: root/src/http/ngx_http_request.c
diff options
context:
space:
mode:
authorMaxim Dounin <mdounin@mdounin.ru>2012-06-04 11:07:19 +0000
committerMaxim Dounin <mdounin@mdounin.ru>2012-06-04 11:07:19 +0000
commit69d94a9c6deae87f75e4bce4fe12b92a6a0e2a27 (patch)
treebc6698c506763a8488bdf3cfe58ed5e8d1906f74 /src/http/ngx_http_request.c
parent245cf55736ea05c075c3a1ef0f06afba22b4ab21 (diff)
downloadnginx-69d94a9c6deae87f75e4bce4fe12b92a6a0e2a27.tar.gz
nginx-69d94a9c6deae87f75e4bce4fe12b92a6a0e2a27.zip
Merge of r4618: rewrite escaping fix (ticket #162).
The following code resulted in incorrect escaping of uri and possible segfault: location / { rewrite ^(.*) $1?c=$1; return 200 "$uri"; } If there were arguments in a rewrite's replacement string, and length was actually calculated (due to duplicate captures as in the example above, or variables present), the is_args flag was set and incorrectly copied after length calculation. This resulted in escaping applied to the uri part of the replacement, resulting in incorrect escaping. Additionally, buffer was allocated without escaping expected, thus this also resulted in buffer overrun and possible segfault.
Diffstat (limited to 'src/http/ngx_http_request.c')
0 files changed, 0 insertions, 0 deletions