mirror of
https://github.com/nginx/nginx.git
synced 2024-11-30 18:29:08 +08:00
Fixed variable syntax checking in "set", "geo", "limit_conn_zone",
and "perl_set" directives.
This commit is contained in:
parent
b3430993f1
commit
ba290091cf
@ -322,6 +322,13 @@ ngx_http_geo_block(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||
}
|
||||
|
||||
name = value[1];
|
||||
|
||||
if (name.len < 2 || name.data[0] != '$') {
|
||||
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
|
||||
"invalid variable name \"%V\"", &name);
|
||||
return NGX_CONF_ERROR;
|
||||
}
|
||||
|
||||
name.len--;
|
||||
name.data++;
|
||||
|
||||
|
@ -540,7 +540,7 @@ ngx_http_limit_conn_zone(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||
continue;
|
||||
}
|
||||
|
||||
if (value[i].data[0] == '$') {
|
||||
if (value[i].len > 1 && value[i].data[0] == '$') {
|
||||
|
||||
value[i].len--;
|
||||
value[i].data++;
|
||||
@ -613,7 +613,7 @@ ngx_http_limit_zone(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||
|
||||
value = cf->args->elts;
|
||||
|
||||
if (value[2].data[0] != '$') {
|
||||
if (value[2].len < 2 || value[2].data[0] != '$') {
|
||||
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
|
||||
"invalid variable name \"%V\"", &value[2]);
|
||||
return NGX_CONF_ERROR;
|
||||
|
@ -908,7 +908,7 @@ ngx_http_rewrite_set(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||
|
||||
value = cf->args->elts;
|
||||
|
||||
if (value[1].data[0] != '$') {
|
||||
if (value[1].len < 2 || value[1].data[0] != '$') {
|
||||
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
|
||||
"invalid variable name \"%V\"", &value[1]);
|
||||
return NGX_CONF_ERROR;
|
||||
|
@ -968,7 +968,7 @@ ngx_http_perl_set(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||
|
||||
value = cf->args->elts;
|
||||
|
||||
if (value[1].data[0] != '$') {
|
||||
if (value[1].len < 2 || value[1].data[0] != '$') {
|
||||
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
|
||||
"invalid variable name \"%V\"", &value[1]);
|
||||
return NGX_CONF_ERROR;
|
||||
|
Loading…
Reference in New Issue
Block a user