merge r3001, r3007, r3013:
fix variuos segfaults: *) fix segfault if 400 or 414 errors are handled intricately *) ngx_http_upstream_create() to cleanup the previous upstream after internal redirect
This commit is contained in:
parent
8d1d7b5aaa
commit
9a7f7e4578
|
@ -543,19 +543,16 @@ ngx_http_fastcgi_handler(ngx_http_request_t *r)
|
|||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
}
|
||||
|
||||
f = ngx_pcalloc(r->pool, sizeof(ngx_http_fastcgi_ctx_t));
|
||||
if (f == NULL) {
|
||||
return NGX_ERROR;
|
||||
}
|
||||
|
||||
ngx_http_set_ctx(r, f, ngx_http_fastcgi_module);
|
||||
|
||||
u = ngx_pcalloc(r->pool, sizeof(ngx_http_upstream_t));
|
||||
if (u == NULL) {
|
||||
if (ngx_http_upstream_create(r) != NGX_OK) {
|
||||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
}
|
||||
|
||||
r->upstream = u;
|
||||
f = ngx_pcalloc(r->pool, sizeof(ngx_http_fastcgi_ctx_t));
|
||||
if (f == NULL) {
|
||||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
}
|
||||
|
||||
ngx_http_set_ctx(r, f, ngx_http_fastcgi_module);
|
||||
|
||||
flcf = ngx_http_get_module_loc_conf(r, ngx_http_fastcgi_module);
|
||||
|
||||
|
@ -565,15 +562,11 @@ ngx_http_fastcgi_handler(ngx_http_request_t *r)
|
|||
}
|
||||
}
|
||||
|
||||
u = r->upstream;
|
||||
|
||||
u->schema.len = sizeof("fastcgi://") - 1;
|
||||
u->schema.data = (u_char *) "fastcgi://";
|
||||
|
||||
u->peer.log = r->connection->log;
|
||||
u->peer.log_error = NGX_ERROR_ERR;
|
||||
#if (NGX_THREADS)
|
||||
u->peer.lock = &r->connection->lock;
|
||||
#endif
|
||||
|
||||
u->output.tag = (ngx_buf_tag_t) &ngx_http_fastcgi_module;
|
||||
|
||||
u->conf = &flcf->upstream;
|
||||
|
|
|
@ -176,24 +176,19 @@ ngx_http_memcached_handler(ngx_http_request_t *r)
|
|||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
}
|
||||
|
||||
mlcf = ngx_http_get_module_loc_conf(r, ngx_http_memcached_module);
|
||||
|
||||
u = ngx_pcalloc(r->pool, sizeof(ngx_http_upstream_t));
|
||||
if (u == NULL) {
|
||||
if (ngx_http_upstream_create(r) != NGX_OK) {
|
||||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
}
|
||||
|
||||
u = r->upstream;
|
||||
|
||||
u->schema.len = sizeof("memcached://") - 1;
|
||||
u->schema.data = (u_char *) "memcached://";
|
||||
|
||||
u->peer.log = r->connection->log;
|
||||
u->peer.log_error = NGX_ERROR_ERR;
|
||||
#if (NGX_THREADS)
|
||||
u->peer.lock = &r->connection->lock;
|
||||
#endif
|
||||
|
||||
u->output.tag = (ngx_buf_tag_t) &ngx_http_memcached_module;
|
||||
|
||||
mlcf = ngx_http_get_module_loc_conf(r, ngx_http_memcached_module);
|
||||
|
||||
u->conf = &mlcf->upstream;
|
||||
|
||||
u->create_request = ngx_http_memcached_create_request;
|
||||
|
@ -202,8 +197,6 @@ ngx_http_memcached_handler(ngx_http_request_t *r)
|
|||
u->abort_request = ngx_http_memcached_abort_request;
|
||||
u->finalize_request = ngx_http_memcached_finalize_request;
|
||||
|
||||
r->upstream = u;
|
||||
|
||||
ctx = ngx_palloc(r->pool, sizeof(ngx_http_memcached_ctx_t));
|
||||
if (ctx == NULL) {
|
||||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
|
|
|
@ -590,13 +590,10 @@ ngx_http_proxy_handler(ngx_http_request_t *r)
|
|||
ngx_http_proxy_ctx_t *ctx;
|
||||
ngx_http_proxy_loc_conf_t *plcf;
|
||||
|
||||
u = ngx_pcalloc(r->pool, sizeof(ngx_http_upstream_t));
|
||||
if (u == NULL) {
|
||||
if (ngx_http_upstream_create(r) != NGX_OK) {
|
||||
return NGX_HTTP_INTERNAL_SERVER_ERROR;
|
||||
}
|
||||
|
||||
r->upstream = u;
|
||||
|
||||
ctx = ngx_pcalloc(r->pool, sizeof(ngx_http_proxy_ctx_t));
|
||||
if (ctx == NULL) {
|
||||
return NGX_ERROR;
|
||||
|
@ -606,6 +603,8 @@ ngx_http_proxy_handler(ngx_http_request_t *r)
|
|||
|
||||
plcf = ngx_http_get_module_loc_conf(r, ngx_http_proxy_module);
|
||||
|
||||
u = r->upstream;
|
||||
|
||||
if (plcf->proxy_lengths == 0) {
|
||||
ctx->vars = plcf->vars;
|
||||
u->schema = plcf->vars.schema;
|
||||
|
@ -619,12 +618,6 @@ ngx_http_proxy_handler(ngx_http_request_t *r)
|
|||
}
|
||||
}
|
||||
|
||||
u->peer.log = r->connection->log;
|
||||
u->peer.log_error = NGX_ERROR_ERR;
|
||||
#if (NGX_THREADS)
|
||||
u->peer.lock = &r->connection->lock;
|
||||
#endif
|
||||
|
||||
u->output.tag = (ngx_buf_tag_t) &ngx_http_proxy_module;
|
||||
|
||||
u->conf = &plcf->upstream;
|
||||
|
|
|
@ -384,6 +384,7 @@ ngx_http_init_request(ngx_event_t *rev)
|
|||
r->loc_conf = cscf->ctx->loc_conf;
|
||||
|
||||
rev->handler = ngx_http_process_request_line;
|
||||
r->read_event_handler = ngx_http_block_reading;
|
||||
|
||||
#if (NGX_HTTP_SSL)
|
||||
|
||||
|
|
|
@ -354,6 +354,35 @@ ngx_conf_bitmask_t ngx_http_upstream_cache_method_mask[] = {
|
|||
};
|
||||
|
||||
|
||||
ngx_int_t
|
||||
ngx_http_upstream_create(ngx_http_request_t *r)
|
||||
{
|
||||
ngx_http_upstream_t *u;
|
||||
|
||||
u = r->upstream;
|
||||
|
||||
if (u && u->cleanup) {
|
||||
ngx_http_upstream_cleanup(r);
|
||||
*u->cleanup = NULL;
|
||||
}
|
||||
|
||||
u = ngx_pcalloc(r->pool, sizeof(ngx_http_upstream_t));
|
||||
if (u == NULL) {
|
||||
return NGX_ERROR;
|
||||
}
|
||||
|
||||
r->upstream = u;
|
||||
|
||||
u->peer.log = r->connection->log;
|
||||
u->peer.log_error = NGX_ERROR_ERR;
|
||||
#if (NGX_THREADS)
|
||||
u->peer.lock = &r->connection->lock;
|
||||
#endif
|
||||
|
||||
return NGX_OK;
|
||||
}
|
||||
|
||||
|
||||
void
|
||||
ngx_http_upstream_init(ngx_http_request_t *r)
|
||||
{
|
||||
|
@ -2864,7 +2893,7 @@ ngx_http_upstream_finalize_request(ngx_http_request_t *r,
|
|||
|
||||
#if (NGX_HTTP_CACHE)
|
||||
|
||||
if (u->cacheable) {
|
||||
if (u->cacheable && r->cache) {
|
||||
time_t valid;
|
||||
|
||||
ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
|
||||
|
|
|
@ -317,6 +317,7 @@ typedef struct {
|
|||
ngx_int_t ngx_http_upstream_header_variable(ngx_http_request_t *r,
|
||||
ngx_http_variable_value_t *v, uintptr_t data);
|
||||
|
||||
ngx_int_t ngx_http_upstream_create(ngx_http_request_t *r);
|
||||
void ngx_http_upstream_init(ngx_http_request_t *r);
|
||||
ngx_http_upstream_srv_conf_t *ngx_http_upstream_add(ngx_conf_t *cf,
|
||||
ngx_url_t *u, ngx_uint_t flags);
|
||||
|
|
Loading…
Reference in New Issue