Merge of r4815: map: fixed optimization of variables as values.
Previous code incorrectly used ctx->var_values as an array of pointers to ngx_http_variable_value_t, but the array contains structures, not pointers. Additionally, ctx->var_values inspection failed to properly set var on match.
This commit is contained in:
parent
6286f9c062
commit
8cc4f6248a
|
@ -416,11 +416,12 @@ ngx_http_map(ngx_conf_t *cf, ngx_command_t *dummy, void *conf)
|
|||
|
||||
for (i = 0; i < ctx->var_values.nelts; i++) {
|
||||
if (index == (ngx_int_t) var[i].data) {
|
||||
var = &var[i];
|
||||
goto found;
|
||||
}
|
||||
}
|
||||
|
||||
var = ngx_palloc(ctx->keys.pool, sizeof(ngx_http_variable_value_t));
|
||||
var = ngx_array_push(&ctx->var_values);
|
||||
if (var == NULL) {
|
||||
return NGX_CONF_ERROR;
|
||||
}
|
||||
|
@ -431,13 +432,6 @@ ngx_http_map(ngx_conf_t *cf, ngx_command_t *dummy, void *conf)
|
|||
var->len = 0;
|
||||
var->data = (u_char *) index;
|
||||
|
||||
vp = ngx_array_push(&ctx->var_values);
|
||||
if (vp == NULL) {
|
||||
return NGX_CONF_ERROR;
|
||||
}
|
||||
|
||||
*vp = var;
|
||||
|
||||
goto found;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue