mirror of
https://github.com/nginx/nginx.git
synced 2024-11-30 18:29:08 +08:00
r3117, r3123, r3229 merge:
server name related merges: *) If .domain.com, .sub.domain.com, and .domain-some.com were defined, then .sub.domain.com was matched by .domain.com: wildcard names hash was built incorrectly due to sorting order issue of "." vs "-". They were sorted as com.domain com.domain-some com.domain.sub while they should be sorted as com.domain com.domain.sub com.domain-some for correct hash building *) test space between "~" and regex in server_name and invalid_referers *) do not run regex for empty host name since regex always fails in this case, the bug had been introduced in r2196
This commit is contained in:
parent
664ca88d33
commit
9a61c9e761
@ -819,6 +819,37 @@ ngx_memn2cmp(u_char *s1, u_char *s2, size_t n1, size_t n2)
|
||||
}
|
||||
|
||||
|
||||
ngx_int_t
|
||||
ngx_dns_strcmp(u_char *s1, u_char *s2)
|
||||
{
|
||||
ngx_uint_t c1, c2;
|
||||
|
||||
for ( ;; ) {
|
||||
c1 = (ngx_uint_t) *s1++;
|
||||
c2 = (ngx_uint_t) *s2++;
|
||||
|
||||
c1 = (c1 >= 'A' && c1 <= 'Z') ? (c1 | 0x20) : c1;
|
||||
c2 = (c2 >= 'A' && c2 <= 'Z') ? (c2 | 0x20) : c2;
|
||||
|
||||
if (c1 == c2) {
|
||||
|
||||
if (c1) {
|
||||
continue;
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
/* in ASCII '.' > '-', but we need '.' to be the lowest character */
|
||||
|
||||
c1 = (c1 == '.') ? ' ' : c1;
|
||||
c2 = (c2 == '.') ? ' ' : c2;
|
||||
|
||||
return c1 - c2;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
ngx_int_t
|
||||
ngx_atoi(u_char *line, size_t n)
|
||||
{
|
||||
|
@ -158,6 +158,7 @@ u_char *ngx_strlcasestrn(u_char *s1, u_char *last, u_char *s2, size_t n);
|
||||
ngx_int_t ngx_rstrncmp(u_char *s1, u_char *s2, size_t n);
|
||||
ngx_int_t ngx_rstrncasecmp(u_char *s1, u_char *s2, size_t n);
|
||||
ngx_int_t ngx_memn2cmp(u_char *s1, u_char *s2, size_t n1, size_t n2);
|
||||
ngx_int_t ngx_dns_strcmp(u_char *s1, u_char *s2);
|
||||
|
||||
ngx_int_t ngx_atoi(u_char *line, size_t n);
|
||||
ssize_t ngx_atosz(u_char *line, size_t n);
|
||||
|
@ -337,7 +337,7 @@ ngx_http_map_cmp_dns_wildcards(const void *one, const void *two)
|
||||
first = (ngx_hash_key_t *) one;
|
||||
second = (ngx_hash_key_t *) two;
|
||||
|
||||
return ngx_strcmp(first->key.data, second->key.data);
|
||||
return ngx_dns_strcmp(first->key.data, second->key.data);
|
||||
}
|
||||
|
||||
|
||||
|
@ -506,6 +506,11 @@ ngx_http_add_regex_referer(ngx_conf_t *cf, ngx_http_referer_conf_t *rlcf,
|
||||
ngx_regex_elt_t *re;
|
||||
u_char errstr[NGX_MAX_CONF_ERRSTR];
|
||||
|
||||
if (name->len == 1) {
|
||||
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, "empty regex in \"%V\"", name);
|
||||
return NGX_CONF_ERROR;
|
||||
}
|
||||
|
||||
if (rlcf->regex == NGX_CONF_UNSET_PTR) {
|
||||
rlcf->regex = ngx_array_create(cf->pool, 2, sizeof(ngx_regex_elt_t));
|
||||
if (rlcf->regex == NULL) {
|
||||
@ -562,5 +567,5 @@ ngx_http_cmp_referer_wildcards(const void *one, const void *two)
|
||||
first = (ngx_hash_key_t *) one;
|
||||
second = (ngx_hash_key_t *) two;
|
||||
|
||||
return ngx_strcmp(first->key.data, second->key.data);
|
||||
return ngx_dns_strcmp(first->key.data, second->key.data);
|
||||
}
|
||||
|
@ -1601,7 +1601,7 @@ ngx_http_cmp_dns_wildcards(const void *one, const void *two)
|
||||
first = (ngx_hash_key_t *) one;
|
||||
second = (ngx_hash_key_t *) two;
|
||||
|
||||
return ngx_strcmp(first->key.data, second->key.data);
|
||||
return ngx_dns_strcmp(first->key.data, second->key.data);
|
||||
}
|
||||
|
||||
|
||||
|
@ -3512,6 +3512,12 @@ ngx_http_core_server_name(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||
ngx_str_t err;
|
||||
u_char errstr[NGX_MAX_CONF_ERRSTR];
|
||||
|
||||
if (value[i].len == 1) {
|
||||
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
|
||||
"empty regex in server name \"%V\"", &value[i]);
|
||||
return NGX_CONF_ERROR;
|
||||
}
|
||||
|
||||
err.len = NGX_MAX_CONF_ERRSTR;
|
||||
err.data = errstr;
|
||||
|
||||
|
@ -1658,7 +1658,7 @@ ngx_http_find_virtual_server(ngx_http_request_t *r, u_char *host, size_t len)
|
||||
|
||||
#if (NGX_PCRE)
|
||||
|
||||
if (r->virtual_names->nregex) {
|
||||
if (len && r->virtual_names->nregex) {
|
||||
size_t ncaptures;
|
||||
ngx_int_t n;
|
||||
ngx_uint_t i;
|
||||
|
Loading…
Reference in New Issue
Block a user