]> git.kaiwu.me - nginx.git/commit
Avoid post-rewrite URI to path mapping with alias
authorRoman Arutyunyan <arut@nginx.com>
Fri, 10 Jul 2026 16:10:12 +0000 (20:10 +0400)
committerSergey Kandaurov <pluknet@nginx.com>
Thu, 1 Oct 2026 17:19:27 +0000 (21:19 +0400)
commitf8a23425bd6840c4c36b4264edf576ba3646156f
tree9d1bdc834f5b144c8326178584f0746160564d2e
parent6aa4b3fdd4fdc156153f2981eb4070690624447c
Avoid post-rewrite URI to path mapping with alias

Previously, after a "rewrite" had changed the URI, and before a new
location was chosen, the new URI might not match the current location
prefix.  This becomes a problem when the location uses "alias", which
is why normally such inconsistency is detected by checking the
r->valid_location flag.  Meanwhile this flag only handles the
inconsistency when using the "break" directive or parameter.  However
in absence of those, there's still an inconsistency window from URI
change up until the end of the current set of rewrite directives.
URI to path mapping within this window might result in a buffer
overflow if location's prefix was longer than the rewritten URI.

The following example resulted in a buffer overflow with the request
"/foo/1":

location /foo/ {
    alias /bar/;
    rewrite ^/foo/.*$ /x;
    return 200 $request_filename;
}

Alternatively, $request_filename might be used in access log format,
"set", "if", "add_header", etc.

The fix is to reject URI mapping in ngx_http_map_uri_to_path() if the
r->uri_changed flag is set.
src/http/ngx_http_core_module.c