From 3fc6f64bd297de220faab0f0c7abe8b67e1b870f Mon Sep 17 00:00:00 2001 From: Igor Sysoev Date: Thu, 10 Nov 2005 07:44:53 +0000 Subject: nginx-0.3.9-RELEASE import *) Bugfix: nginx considered URI as unsafe if two any symbols was between two slashes; the bug had appeared in 0.3.8. --- docs/xml/nginx/changes.xml | 19 ++++++++++++++++++- src/core/nginx.h | 2 +- src/http/ngx_http_parse.c | 6 ++++-- 3 files changed, 23 insertions(+), 4 deletions(-) diff --git a/docs/xml/nginx/changes.xml b/docs/xml/nginx/changes.xml index dfac2a065..4402afc04 100644 --- a/docs/xml/nginx/changes.xml +++ b/docs/xml/nginx/changes.xml @@ -9,6 +9,23 @@ nginx changelog + + + + +nginx считал небезопасными URI, в которых между двумя слэшами +находилось два любых символа; +ошибка появилась в 0.3.8. + + +nginx considered URI as unsafe if two any symbols was between two slashes; +bug appeared in 0.3.8. + + + + + + @@ -119,7 +136,7 @@ and temporary files with client requests in open state. -рабочие процессы не сбрасывал буферизированные логи при плавном выходе. +рабочие процессы не сбрасывали буферизированные логи при плавном выходе. the worker processes did not flush the buffered logs on graceful exit. diff --git a/src/core/nginx.h b/src/core/nginx.h index 966d405b0..c498e0719 100644 --- a/src/core/nginx.h +++ b/src/core/nginx.h @@ -8,7 +8,7 @@ #define _NGINX_H_INCLUDED_ -#define NGINX_VER "nginx/0.3.8" +#define NGINX_VER "nginx/0.3.9" #define NGINX_VAR "NGINX" #define NGX_OLDPID_EXT ".oldbin" diff --git a/src/http/ngx_http_parse.c b/src/http/ngx_http_parse.c index 0001286c6..b2e2f9e3f 100644 --- a/src/http/ngx_http_parse.c +++ b/src/http/ngx_http_parse.c @@ -1056,7 +1056,7 @@ ngx_http_parse_unsafe_uri(ngx_http_request_t *r, ngx_str_t *uri, /* detect "/../" */ - if (p[2] == '/') { + if (p[0] == '.' && p[1] == '.' && p[2] == '/') { goto unsafe; } @@ -1070,7 +1070,9 @@ ngx_http_parse_unsafe_uri(ngx_http_request_t *r, ngx_str_t *uri, /* detect "/.../" */ - if (p[3] == '/' || p[3] == '\\') { + if (p[0] == '.' && p[1] == '.' && p[2] == '.' + && (p[3] == '/' || p[3] == '\\')) + { goto unsafe; } } -- cgit v1.2.1