Mercurial > hg > nginx
diff src/core/ngx_resolver.c @ 2484:cf3cd450049c
store name pointer in variable allocated on stack
as resolver ctx may be already deallocated at this point
author | Igor Sysoev <igor@sysoev.ru> |
---|---|
date | Thu, 29 Jan 2009 14:35:23 +0000 |
parents | 29494780d978 |
children | 8de5dc3e7001 |
line wrap: on
line diff
--- a/src/core/ngx_resolver.c Thu Jan 29 14:32:58 2009 +0000 +++ b/src/core/ngx_resolver.c Thu Jan 29 14:35:23 2009 +0000 @@ -578,6 +578,7 @@ ngx_int_t ngx_resolve_addr(ngx_resolver_ctx_t *ctx) { + u_char *name; ngx_resolver_t *r; ngx_resolver_node_t *rn; @@ -601,19 +602,21 @@ ngx_queue_insert_head(&r->addr_expire_queue, &rn->queue); - ctx->name.len = rn->nlen; - ctx->name.data = ngx_resolver_dup(r, rn->name, rn->nlen); - if (ctx->name.data == NULL) { + name = ngx_resolver_dup(r, rn->name, rn->nlen); + if (name == NULL) { goto failed; } + ctx->name.len = rn->nlen; + ctx->name.data = name; + /* unlock addr mutex */ ctx->state = NGX_OK; ctx->handler(ctx); - ngx_resolver_free(r, ctx->name.data); + ngx_resolver_free(r, name); return NGX_OK; }