mirror of https://github.com/openresty/openresty
upgraded the nginx core to 1.2.1. bumped the version of ngx_openresty to 1.2.1.1rc1.
parent
139b2911ea
commit
2a871e5740
@ -0,0 +1,26 @@
|
|||||||
|
--- nginx-1.2.1/src/http/ngx_http_variables.c 2012-03-05 20:36:51.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_variables.c 2012-05-17 16:21:42.120968722 +0800
|
||||||
|
@@ -1898,6 +1898,7 @@
|
||||||
|
{
|
||||||
|
ngx_int_t rc;
|
||||||
|
ngx_http_variable_t *v;
|
||||||
|
+ ngx_http_variable_t *value;
|
||||||
|
ngx_http_core_main_conf_t *cmcf;
|
||||||
|
|
||||||
|
cmcf = ngx_http_conf_get_module_main_conf(cf, ngx_http_core_module);
|
||||||
|
@@ -1918,7 +1919,14 @@
|
||||||
|
}
|
||||||
|
|
||||||
|
for (v = ngx_http_core_variables; v->name.len; v++) {
|
||||||
|
- rc = ngx_hash_add_key(cmcf->variables_keys, &v->name, v,
|
||||||
|
+ value = ngx_palloc(cf->pool, sizeof(ngx_http_variable_t));
|
||||||
|
+ if (value == NULL) {
|
||||||
|
+ return NGX_ERROR;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ *value = *v;
|
||||||
|
+
|
||||||
|
+ rc = ngx_hash_add_key(cmcf->variables_keys, &value->name, value,
|
||||||
|
NGX_HASH_READONLY_KEY);
|
||||||
|
|
||||||
|
if (rc == NGX_OK) {
|
@ -0,0 +1,113 @@
|
|||||||
|
diff -ur nginx-1.2.1/src/http/ngx_http_request_body.c nginx-1.2.1-patched/src/http/ngx_http_request_body.c
|
||||||
|
--- nginx-1.2.1/src/http/ngx_http_request_body.c 2011-09-30 22:36:19.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_request_body.c 2011-10-21 21:54:08.460350482 +0800
|
||||||
|
@@ -38,7 +38,7 @@
|
||||||
|
|
||||||
|
r->main->count++;
|
||||||
|
|
||||||
|
- if (r->request_body || r->discard_body) {
|
||||||
|
+ if (r->request_body || r->discard_body || r->content_length_n == 0) {
|
||||||
|
post_handler(r);
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
@@ -440,7 +440,7 @@
|
||||||
|
ssize_t size;
|
||||||
|
ngx_event_t *rev;
|
||||||
|
|
||||||
|
- if (r != r->main || r->discard_body) {
|
||||||
|
+ if (r != r->main || r->discard_body || r->content_length_n == 0) {
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -456,20 +456,22 @@
|
||||||
|
ngx_del_timer(rev);
|
||||||
|
}
|
||||||
|
|
||||||
|
- if (r->headers_in.content_length_n <= 0 || r->request_body) {
|
||||||
|
+ r->content_length_n = r->headers_in.content_length_n;
|
||||||
|
+
|
||||||
|
+ if (r->content_length_n <= 0 || r->request_body) {
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
|
||||||
|
size = r->header_in->last - r->header_in->pos;
|
||||||
|
|
||||||
|
if (size) {
|
||||||
|
- if (r->headers_in.content_length_n > size) {
|
||||||
|
+ if (r->content_length_n > size) {
|
||||||
|
r->header_in->pos += size;
|
||||||
|
- r->headers_in.content_length_n -= size;
|
||||||
|
+ r->content_length_n -= size;
|
||||||
|
|
||||||
|
} else {
|
||||||
|
- r->header_in->pos += (size_t) r->headers_in.content_length_n;
|
||||||
|
- r->headers_in.content_length_n = 0;
|
||||||
|
+ r->header_in->pos += (size_t) r->content_length_n;
|
||||||
|
+ r->content_length_n = 0;
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -568,7 +570,7 @@
|
||||||
|
"http read discarded body");
|
||||||
|
|
||||||
|
for ( ;; ) {
|
||||||
|
- if (r->headers_in.content_length_n == 0) {
|
||||||
|
+ if (r->content_length_n == 0) {
|
||||||
|
r->read_event_handler = ngx_http_block_reading;
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
@@ -577,9 +579,9 @@
|
||||||
|
return NGX_AGAIN;
|
||||||
|
}
|
||||||
|
|
||||||
|
- size = (r->headers_in.content_length_n > NGX_HTTP_DISCARD_BUFFER_SIZE) ?
|
||||||
|
+ size = (r->content_length_n > NGX_HTTP_DISCARD_BUFFER_SIZE) ?
|
||||||
|
NGX_HTTP_DISCARD_BUFFER_SIZE:
|
||||||
|
- (size_t) r->headers_in.content_length_n;
|
||||||
|
+ (size_t) r->content_length_n;
|
||||||
|
|
||||||
|
n = r->connection->recv(r->connection, buffer, size);
|
||||||
|
|
||||||
|
@@ -596,7 +598,7 @@
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
|
||||||
|
- r->headers_in.content_length_n -= n;
|
||||||
|
+ r->content_length_n -= n;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
diff -ur nginx-1.2.1/src/http/ngx_http_request.c nginx-1.2.1-patched/src/http/ngx_http_request.c
|
||||||
|
--- nginx-1.2.1/src/http/ngx_http_request.c 2011-09-30 22:36:19.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_request.c 2011-10-21 19:06:38.404350692 +0800
|
||||||
|
@@ -286,6 +286,8 @@
|
||||||
|
|
||||||
|
r->pipeline = hc->pipeline;
|
||||||
|
|
||||||
|
+ r->content_length_n = -1;
|
||||||
|
+
|
||||||
|
if (hc->nbusy) {
|
||||||
|
r->header_in = hc->busy[0];
|
||||||
|
}
|
||||||
|
@@ -297,6 +299,8 @@
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ r->content_length_n = -1;
|
||||||
|
+
|
||||||
|
hc->request = r;
|
||||||
|
}
|
||||||
|
|
||||||
|
diff -ur nginx-1.2.1/src/http/ngx_http_request.h nginx-1.2.1-patched/src/http/ngx_http_request.h
|
||||||
|
--- nginx-1.2.1/src/http/ngx_http_request.h 2011-08-29 18:39:23.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_request.h 2011-10-21 17:26:13.203807584 +0800
|
||||||
|
@@ -366,6 +366,9 @@
|
||||||
|
ngx_pool_t *pool;
|
||||||
|
ngx_buf_t *header_in;
|
||||||
|
|
||||||
|
+ off_t content_length_n;
|
||||||
|
+ /* for discarding request body */
|
||||||
|
+
|
||||||
|
ngx_http_headers_in_t headers_in;
|
||||||
|
ngx_http_headers_out_t headers_out;
|
||||||
|
|
@ -0,0 +1,115 @@
|
|||||||
|
--- nginx-1.2.1/src/http/modules/ngx_http_log_module.c 2011-11-01 21:24:50.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/modules/ngx_http_log_module.c 2011-11-10 16:17:29.599039534 +0800
|
||||||
|
@@ -61,6 +61,8 @@
|
||||||
|
time_t open_file_cache_valid;
|
||||||
|
ngx_uint_t open_file_cache_min_uses;
|
||||||
|
|
||||||
|
+ ngx_flag_t escape_non_ascii;
|
||||||
|
+
|
||||||
|
ngx_uint_t off; /* unsigned off:1 */
|
||||||
|
} ngx_http_log_loc_conf_t;
|
||||||
|
|
||||||
|
@@ -104,7 +106,8 @@
|
||||||
|
uintptr_t data);
|
||||||
|
static u_char *ngx_http_log_variable(ngx_http_request_t *r, u_char *buf,
|
||||||
|
ngx_http_log_op_t *op);
|
||||||
|
-static uintptr_t ngx_http_log_escape(u_char *dst, u_char *src, size_t size);
|
||||||
|
+static uintptr_t ngx_http_log_escape(ngx_http_log_loc_conf_t *lcf, u_char *dst,
|
||||||
|
+ u_char *src, size_t size);
|
||||||
|
|
||||||
|
|
||||||
|
static void *ngx_http_log_create_main_conf(ngx_conf_t *cf);
|
||||||
|
@@ -146,6 +149,13 @@
|
||||||
|
0,
|
||||||
|
NULL },
|
||||||
|
|
||||||
|
+ { ngx_string("log_escape_non_ascii"),
|
||||||
|
+ NGX_HTTP_MAIN_CONF|NGX_HTTP_SRV_CONF|NGX_HTTP_LOC_CONF|NGX_CONF_FLAG,
|
||||||
|
+ ngx_conf_set_flag_slot,
|
||||||
|
+ NGX_HTTP_LOC_CONF_OFFSET,
|
||||||
|
+ offsetof(ngx_http_log_loc_conf_t, escape_non_ascii),
|
||||||
|
+ NULL },
|
||||||
|
+
|
||||||
|
ngx_null_command
|
||||||
|
};
|
||||||
|
|
||||||
|
@@ -637,6 +647,7 @@
|
||||||
|
ngx_http_log_variable_getlen(ngx_http_request_t *r, uintptr_t data)
|
||||||
|
{
|
||||||
|
uintptr_t len;
|
||||||
|
+ ngx_http_log_loc_conf_t *lcf;
|
||||||
|
ngx_http_variable_value_t *value;
|
||||||
|
|
||||||
|
value = ngx_http_get_indexed_variable(r, data);
|
||||||
|
@@ -645,7 +656,9 @@
|
||||||
|
return 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
- len = ngx_http_log_escape(NULL, value->data, value->len);
|
||||||
|
+ lcf = ngx_http_get_module_loc_conf(r, ngx_http_log_module);
|
||||||
|
+
|
||||||
|
+ len = ngx_http_log_escape(lcf, NULL, value->data, value->len);
|
||||||
|
|
||||||
|
value->escape = len ? 1 : 0;
|
||||||
|
|
||||||
|
@@ -656,6 +669,7 @@
|
||||||
|
static u_char *
|
||||||
|
ngx_http_log_variable(ngx_http_request_t *r, u_char *buf, ngx_http_log_op_t *op)
|
||||||
|
{
|
||||||
|
+ ngx_http_log_loc_conf_t *lcf;
|
||||||
|
ngx_http_variable_value_t *value;
|
||||||
|
|
||||||
|
value = ngx_http_get_indexed_variable(r, op->data);
|
||||||
|
@@ -669,16 +683,18 @@
|
||||||
|
return ngx_cpymem(buf, value->data, value->len);
|
||||||
|
|
||||||
|
} else {
|
||||||
|
- return (u_char *) ngx_http_log_escape(buf, value->data, value->len);
|
||||||
|
+ lcf = ngx_http_get_module_loc_conf(r, ngx_http_log_module);
|
||||||
|
+ return (u_char *) ngx_http_log_escape(lcf, buf, value->data, value->len);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
static uintptr_t
|
||||||
|
-ngx_http_log_escape(u_char *dst, u_char *src, size_t size)
|
||||||
|
+ngx_http_log_escape(ngx_http_log_loc_conf_t *lcf, u_char *dst, u_char *src,
|
||||||
|
+ size_t size)
|
||||||
|
{
|
||||||
|
- ngx_uint_t n;
|
||||||
|
- static u_char hex[] = "0123456789ABCDEF";
|
||||||
|
+ ngx_uint_t n;
|
||||||
|
+ static u_char hex[] = "0123456789ABCDEF";
|
||||||
|
|
||||||
|
static uint32_t escape[] = {
|
||||||
|
0xffffffff, /* 1111 1111 1111 1111 1111 1111 1111 1111 */
|
||||||
|
@@ -698,6 +714,12 @@
|
||||||
|
0xffffffff, /* 1111 1111 1111 1111 1111 1111 1111 1111 */
|
||||||
|
};
|
||||||
|
|
||||||
|
+ if (lcf->escape_non_ascii) {
|
||||||
|
+ ngx_memset(&escape[4], 0xff, sizeof(uint32_t) * 4);
|
||||||
|
+
|
||||||
|
+ } else {
|
||||||
|
+ ngx_memzero(&escape[4], sizeof(uint32_t) * 4);
|
||||||
|
+ }
|
||||||
|
|
||||||
|
if (dst == NULL) {
|
||||||
|
|
||||||
|
@@ -781,6 +803,7 @@
|
||||||
|
}
|
||||||
|
|
||||||
|
conf->open_file_cache = NGX_CONF_UNSET_PTR;
|
||||||
|
+ conf->escape_non_ascii = NGX_CONF_UNSET;
|
||||||
|
|
||||||
|
return conf;
|
||||||
|
}
|
||||||
|
@@ -796,6 +819,8 @@
|
||||||
|
ngx_http_log_fmt_t *fmt;
|
||||||
|
ngx_http_log_main_conf_t *lmcf;
|
||||||
|
|
||||||
|
+ ngx_conf_merge_value(conf->escape_non_ascii, prev->escape_non_ascii, 1);
|
||||||
|
+
|
||||||
|
if (conf->open_file_cache == NGX_CONF_UNSET_PTR) {
|
||||||
|
|
||||||
|
conf->open_file_cache = prev->open_file_cache;
|
@ -0,0 +1,24 @@
|
|||||||
|
diff -ur nginx-1.2.1/auto/cc/gcc nginx-1.2.1-patched/auto/cc/gcc
|
||||||
|
--- nginx-1.2.1/auto/cc/gcc 2011-06-27 19:53:00.205737804 +0800
|
||||||
|
+++ nginx-1.2.1-patched/auto/cc/gcc 2011-06-27 19:53:13.837741087 +0800
|
||||||
|
@@ -169,7 +169,7 @@
|
||||||
|
|
||||||
|
|
||||||
|
# stop on warning
|
||||||
|
-CFLAGS="$CFLAGS -Werror"
|
||||||
|
+#CFLAGS="$CFLAGS -Werror"
|
||||||
|
|
||||||
|
# debug
|
||||||
|
CFLAGS="$CFLAGS -g"
|
||||||
|
diff -ur nginx-1.2.1/auto/cc/icc nginx-1.2.1-patched/auto/cc/icc
|
||||||
|
--- nginx-1.2.1/auto/cc/icc 2011-06-27 19:52:56.370157068 +0800
|
||||||
|
+++ nginx-1.2.1-patched/auto/cc/icc 2011-06-27 19:53:19.508916811 +0800
|
||||||
|
@@ -139,7 +139,7 @@
|
||||||
|
esac
|
||||||
|
|
||||||
|
# stop on warning
|
||||||
|
-CFLAGS="$CFLAGS -Werror"
|
||||||
|
+#CFLAGS="$CFLAGS -Werror"
|
||||||
|
|
||||||
|
# debug
|
||||||
|
CFLAGS="$CFLAGS -g"
|
@ -0,0 +1,90 @@
|
|||||||
|
--- nginx-1.2.1/src/http/ngx_http_core_module.c 2010-12-14 18:38:42.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_core_module.c 2011-01-30 19:24:34.956354518 +0800
|
||||||
|
@@ -57,6 +57,8 @@
|
||||||
|
void *conf);
|
||||||
|
static char *ngx_http_core_error_page(ngx_conf_t *cf, ngx_command_t *cmd,
|
||||||
|
void *conf);
|
||||||
|
+static char *ngx_http_core_no_error_pages(ngx_conf_t *cf, ngx_command_t *cmd,
|
||||||
|
+ void *conf);
|
||||||
|
static char *ngx_http_core_try_files(ngx_conf_t *cf, ngx_command_t *cmd,
|
||||||
|
void *conf);
|
||||||
|
static char *ngx_http_core_open_file_cache(ngx_conf_t *cf, ngx_command_t *cmd,
|
||||||
|
@@ -614,6 +616,14 @@
|
||||||
|
0,
|
||||||
|
NULL },
|
||||||
|
|
||||||
|
+ { ngx_string("no_error_pages"),
|
||||||
|
+ NGX_HTTP_MAIN_CONF|NGX_HTTP_SRV_CONF|NGX_HTTP_LOC_CONF|NGX_HTTP_LIF_CONF
|
||||||
|
+ |NGX_CONF_NOARGS,
|
||||||
|
+ ngx_http_core_no_error_pages,
|
||||||
|
+ NGX_HTTP_LOC_CONF_OFFSET,
|
||||||
|
+ 0,
|
||||||
|
+ NULL },
|
||||||
|
+
|
||||||
|
{ ngx_string("try_files"),
|
||||||
|
NGX_HTTP_SRV_CONF|NGX_HTTP_LOC_CONF|NGX_CONF_2MORE,
|
||||||
|
ngx_http_core_try_files,
|
||||||
|
@@ -3052,7 +3062,6 @@
|
||||||
|
* clcf->types = NULL;
|
||||||
|
* clcf->default_type = { 0, NULL };
|
||||||
|
* clcf->error_log = NULL;
|
||||||
|
- * clcf->error_pages = NULL;
|
||||||
|
* clcf->try_files = NULL;
|
||||||
|
* clcf->client_body_path = NULL;
|
||||||
|
* clcf->regex = NULL;
|
||||||
|
@@ -3062,6 +3071,7 @@
|
||||||
|
* clcf->gzip_proxied = 0;
|
||||||
|
*/
|
||||||
|
|
||||||
|
+ clcf->error_pages = NGX_CONF_UNSET_PTR;
|
||||||
|
clcf->client_max_body_size = NGX_CONF_UNSET;
|
||||||
|
clcf->client_body_buffer_size = NGX_CONF_UNSET_SIZE;
|
||||||
|
clcf->client_body_timeout = NGX_CONF_UNSET_MSEC;
|
||||||
|
@@ -3250,9 +3260,7 @@
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
- if (conf->error_pages == NULL && prev->error_pages) {
|
||||||
|
- conf->error_pages = prev->error_pages;
|
||||||
|
- }
|
||||||
|
+ ngx_conf_merge_ptr_value(conf->error_pages, prev->error_pages, NULL);
|
||||||
|
|
||||||
|
ngx_conf_merge_str_value(conf->default_type,
|
||||||
|
prev->default_type, "text/plain");
|
||||||
|
@@ -3988,6 +3996,10 @@
|
||||||
|
ngx_http_compile_complex_value_t ccv;
|
||||||
|
|
||||||
|
if (clcf->error_pages == NULL) {
|
||||||
|
+ return "conflicts with \"no_error_pages\"";
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (clcf->error_pages == NGX_CONF_UNSET_PTR) {
|
||||||
|
clcf->error_pages = ngx_array_create(cf->pool, 4,
|
||||||
|
sizeof(ngx_http_err_page_t));
|
||||||
|
if (clcf->error_pages == NULL) {
|
||||||
|
@@ -4095,6 +4107,25 @@
|
||||||
|
|
||||||
|
|
||||||
|
static char *
|
||||||
|
+ngx_http_core_no_error_pages(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||||
|
+{
|
||||||
|
+ ngx_http_core_loc_conf_t *clcf = conf;
|
||||||
|
+
|
||||||
|
+ if (clcf->error_pages == NULL) {
|
||||||
|
+ return "is duplicate";
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (clcf->error_pages != NGX_CONF_UNSET_PTR) {
|
||||||
|
+ return "conflicts with \"error_page\"";
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ clcf->error_pages = NULL;
|
||||||
|
+
|
||||||
|
+ return NGX_CONF_OK;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+
|
||||||
|
+static char *
|
||||||
|
ngx_http_core_try_files(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
|
||||||
|
{
|
||||||
|
ngx_http_core_loc_conf_t *clcf = conf;
|
@ -0,0 +1,501 @@
|
|||||||
|
diff -ur nginx-1.2.1/src/core/nginx.h nginx-1.2.1-patched/src/core/nginx.h
|
||||||
|
--- nginx-1.2.1/src/core/nginx.h 2011-08-29 17:30:22.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/core/nginx.h 2011-09-13 12:11:03.135622101 +0800
|
||||||
|
@@ -10,7 +10,7 @@
|
||||||
|
|
||||||
|
|
||||||
|
#define nginx_version 1002001
|
||||||
|
#define NGINX_VERSION "1.2.1"
|
||||||
|
-#define NGINX_VER "ngx_openresty/" NGINX_VERSION ".unknown"
|
||||||
|
+#define NGINX_VER "ngx_openresty/" NGINX_VERSION ".unknown (no pool)"
|
||||||
|
|
||||||
|
#define NGINX_VAR "NGINX"
|
||||||
|
diff -urx '*~' -x '*.swp' nginx-1.2.1/src/core/ngx_array.c nginx-1.2.1-patched/src/core/ngx_array.c
|
||||||
|
--- nginx-1.2.1/src/core/ngx_array.c 2012-02-06 04:02:59.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/core/ngx_array.c 2012-06-04 20:11:32.494144287 +0800
|
||||||
|
@@ -40,13 +40,11 @@
|
||||||
|
|
||||||
|
p = a->pool;
|
||||||
|
|
||||||
|
- if ((u_char *) a->elts + a->size * a->nalloc == p->d.last) {
|
||||||
|
- p->d.last -= a->size * a->nalloc;
|
||||||
|
+ if (a->elts) {
|
||||||
|
+ ngx_pfree(p, a->elts);
|
||||||
|
}
|
||||||
|
|
||||||
|
- if ((u_char *) a + sizeof(ngx_array_t) == p->d.last) {
|
||||||
|
- p->d.last = (u_char *) a;
|
||||||
|
- }
|
||||||
|
+ ngx_pfree(p, a);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@@ -65,29 +63,19 @@
|
||||||
|
|
||||||
|
p = a->pool;
|
||||||
|
|
||||||
|
- if ((u_char *) a->elts + size == p->d.last
|
||||||
|
- && p->d.last + a->size <= p->d.end)
|
||||||
|
- {
|
||||||
|
- /*
|
||||||
|
- * the array allocation is the last in the pool
|
||||||
|
- * and there is space for new allocation
|
||||||
|
- */
|
||||||
|
-
|
||||||
|
- p->d.last += a->size;
|
||||||
|
- a->nalloc++;
|
||||||
|
-
|
||||||
|
- } else {
|
||||||
|
- /* allocate a new array */
|
||||||
|
-
|
||||||
|
- new = ngx_palloc(p, 2 * size);
|
||||||
|
- if (new == NULL) {
|
||||||
|
- return NULL;
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- ngx_memcpy(new, a->elts, size);
|
||||||
|
- a->elts = new;
|
||||||
|
- a->nalloc *= 2;
|
||||||
|
+ /* allocate a new array */
|
||||||
|
+
|
||||||
|
+ new = ngx_palloc(p, 2 * size);
|
||||||
|
+ if (new == NULL) {
|
||||||
|
+ return NULL;
|
||||||
|
}
|
||||||
|
+
|
||||||
|
+ ngx_memcpy(new, a->elts, size);
|
||||||
|
+
|
||||||
|
+ ngx_pfree(p, a->elts);
|
||||||
|
+
|
||||||
|
+ a->elts = new;
|
||||||
|
+ a->nalloc *= 2;
|
||||||
|
}
|
||||||
|
|
||||||
|
elt = (u_char *) a->elts + a->size * a->nelts;
|
||||||
|
@@ -101,43 +89,28 @@
|
||||||
|
ngx_array_push_n(ngx_array_t *a, ngx_uint_t n)
|
||||||
|
{
|
||||||
|
void *elt, *new;
|
||||||
|
- size_t size;
|
||||||
|
ngx_uint_t nalloc;
|
||||||
|
ngx_pool_t *p;
|
||||||
|
|
||||||
|
- size = n * a->size;
|
||||||
|
-
|
||||||
|
if (a->nelts + n > a->nalloc) {
|
||||||
|
|
||||||
|
/* the array is full */
|
||||||
|
|
||||||
|
p = a->pool;
|
||||||
|
|
||||||
|
- if ((u_char *) a->elts + a->size * a->nalloc == p->d.last
|
||||||
|
- && p->d.last + size <= p->d.end)
|
||||||
|
- {
|
||||||
|
- /*
|
||||||
|
- * the array allocation is the last in the pool
|
||||||
|
- * and there is space for new allocation
|
||||||
|
- */
|
||||||
|
-
|
||||||
|
- p->d.last += size;
|
||||||
|
- a->nalloc += n;
|
||||||
|
-
|
||||||
|
- } else {
|
||||||
|
- /* allocate a new array */
|
||||||
|
-
|
||||||
|
- nalloc = 2 * ((n >= a->nalloc) ? n : a->nalloc);
|
||||||
|
-
|
||||||
|
- new = ngx_palloc(p, nalloc * a->size);
|
||||||
|
- if (new == NULL) {
|
||||||
|
- return NULL;
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- ngx_memcpy(new, a->elts, a->nelts * a->size);
|
||||||
|
- a->elts = new;
|
||||||
|
- a->nalloc = nalloc;
|
||||||
|
+ nalloc = 2 * ((n >= a->nalloc) ? n : a->nalloc);
|
||||||
|
+
|
||||||
|
+ new = ngx_palloc(p, nalloc * a->size);
|
||||||
|
+ if (new == NULL) {
|
||||||
|
+ return NULL;
|
||||||
|
}
|
||||||
|
+
|
||||||
|
+ ngx_memcpy(new, a->elts, a->nelts * a->size);
|
||||||
|
+
|
||||||
|
+ ngx_pfree(p, a->elts);
|
||||||
|
+
|
||||||
|
+ a->elts = new;
|
||||||
|
+ a->nalloc = nalloc;
|
||||||
|
}
|
||||||
|
|
||||||
|
elt = (u_char *) a->elts + a->size * a->nelts;
|
||||||
|
diff -urx '*~' -x '*.swp' nginx-1.2.1/src/core/ngx_palloc.c nginx-1.2.1-patched/src/core/ngx_palloc.c
|
||||||
|
--- nginx-1.2.1/src/core/ngx_palloc.c 2012-02-06 04:02:59.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/core/ngx_palloc.c 2012-06-04 20:02:26.667925891 +0800
|
||||||
|
@@ -9,32 +9,23 @@
|
||||||
|
#include <ngx_core.h>
|
||||||
|
|
||||||
|
|
||||||
|
-static void *ngx_palloc_block(ngx_pool_t *pool, size_t size);
|
||||||
|
-static void *ngx_palloc_large(ngx_pool_t *pool, size_t size);
|
||||||
|
-
|
||||||
|
-
|
||||||
|
ngx_pool_t *
|
||||||
|
ngx_create_pool(size_t size, ngx_log_t *log)
|
||||||
|
{
|
||||||
|
- ngx_pool_t *p;
|
||||||
|
+ ngx_pool_t *p;
|
||||||
|
|
||||||
|
- p = ngx_memalign(NGX_POOL_ALIGNMENT, size, log);
|
||||||
|
+ size = sizeof(ngx_pool_t);
|
||||||
|
+ p = ngx_alloc(size, log);
|
||||||
|
if (p == NULL) {
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
- p->d.last = (u_char *) p + sizeof(ngx_pool_t);
|
||||||
|
- p->d.end = (u_char *) p + size;
|
||||||
|
- p->d.next = NULL;
|
||||||
|
- p->d.failed = 0;
|
||||||
|
+ ngx_memzero(p, size);
|
||||||
|
|
||||||
|
size = size - sizeof(ngx_pool_t);
|
||||||
|
p->max = (size < NGX_MAX_ALLOC_FROM_POOL) ? size : NGX_MAX_ALLOC_FROM_POOL;
|
||||||
|
|
||||||
|
p->current = p;
|
||||||
|
- p->chain = NULL;
|
||||||
|
- p->large = NULL;
|
||||||
|
- p->cleanup = NULL;
|
||||||
|
p->log = log;
|
||||||
|
|
||||||
|
return p;
|
||||||
|
@@ -44,8 +35,7 @@
|
||||||
|
void
|
||||||
|
ngx_destroy_pool(ngx_pool_t *pool)
|
||||||
|
{
|
||||||
|
- ngx_pool_t *p, *n;
|
||||||
|
- ngx_pool_large_t *l;
|
||||||
|
+ ngx_pool_data_t *d, *n;
|
||||||
|
ngx_pool_cleanup_t *c;
|
||||||
|
|
||||||
|
for (c = pool->cleanup; c; c = c->next) {
|
||||||
|
@@ -56,13 +46,9 @@
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
- for (l = pool->large; l; l = l->next) {
|
||||||
|
-
|
||||||
|
- ngx_log_debug1(NGX_LOG_DEBUG_ALLOC, pool->log, 0, "free: %p", l->alloc);
|
||||||
|
-
|
||||||
|
- if (l->alloc) {
|
||||||
|
- ngx_free(l->alloc);
|
||||||
|
- }
|
||||||
|
+ if (pool->d == NULL) {
|
||||||
|
+ ngx_free(pool);
|
||||||
|
+ return;
|
||||||
|
}
|
||||||
|
|
||||||
|
#if (NGX_DEBUG)
|
||||||
|
@@ -72,9 +58,9 @@
|
||||||
|
* so we cannot use this log while free()ing the pool
|
||||||
|
*/
|
||||||
|
|
||||||
|
- for (p = pool, n = pool->d.next; /* void */; p = n, n = n->d.next) {
|
||||||
|
+ for (d = pool->d, n = d->next; ; d = n, n = n->next) {
|
||||||
|
ngx_log_debug2(NGX_LOG_DEBUG_ALLOC, pool->log, 0,
|
||||||
|
- "free: %p, unused: %uz", p, p->d.end - p->d.last);
|
||||||
|
+ "free: %p, unused: %d", d, 0);
|
||||||
|
|
||||||
|
if (n == NULL) {
|
||||||
|
break;
|
||||||
|
@@ -83,172 +69,82 @@
|
||||||
|
|
||||||
|
#endif
|
||||||
|
|
||||||
|
- for (p = pool, n = pool->d.next; /* void */; p = n, n = n->d.next) {
|
||||||
|
- ngx_free(p);
|
||||||
|
+ for (d = pool->d, n = d->next; ; d = n, n = n->next) {
|
||||||
|
+ ngx_free(d->alloc);
|
||||||
|
+ ngx_free(d);
|
||||||
|
|
||||||
|
if (n == NULL) {
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
-}
|
||||||
|
|
||||||
|
+ pool->d = NULL;
|
||||||
|
|
||||||
|
-void
|
||||||
|
-ngx_reset_pool(ngx_pool_t *pool)
|
||||||
|
-{
|
||||||
|
- ngx_pool_t *p;
|
||||||
|
- ngx_pool_large_t *l;
|
||||||
|
-
|
||||||
|
- for (l = pool->large; l; l = l->next) {
|
||||||
|
- if (l->alloc) {
|
||||||
|
- ngx_free(l->alloc);
|
||||||
|
- }
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- pool->large = NULL;
|
||||||
|
-
|
||||||
|
- for (p = pool; p; p = p->d.next) {
|
||||||
|
- p->d.last = (u_char *) p + sizeof(ngx_pool_t);
|
||||||
|
- }
|
||||||
|
+ ngx_free(pool);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
-void *
|
||||||
|
-ngx_palloc(ngx_pool_t *pool, size_t size)
|
||||||
|
+void
|
||||||
|
+ngx_reset_pool(ngx_pool_t *pool)
|
||||||
|
{
|
||||||
|
- u_char *m;
|
||||||
|
- ngx_pool_t *p;
|
||||||
|
-
|
||||||
|
- if (size <= pool->max) {
|
||||||
|
+ ngx_pool_data_t *d, *n;
|
||||||
|
+ ngx_pool_data_t *saved = NULL;
|
||||||
|
|
||||||
|
- p = pool->current;
|
||||||
|
-
|
||||||
|
- do {
|
||||||
|
- m = ngx_align_ptr(p->d.last, NGX_ALIGNMENT);
|
||||||
|
-
|
||||||
|
- if ((size_t) (p->d.end - m) >= size) {
|
||||||
|
- p->d.last = m + size;
|
||||||
|
-
|
||||||
|
- return m;
|
||||||
|
+ if (pool->d) {
|
||||||
|
+ for (d = pool->d, n = d->next; ; d = n, n = n->next) {
|
||||||
|
+ if (d->alloc == pool->log) {
|
||||||
|
+ saved = d;
|
||||||
|
+ continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
- p = p->d.next;
|
||||||
|
-
|
||||||
|
- } while (p);
|
||||||
|
-
|
||||||
|
- return ngx_palloc_block(pool, size);
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- return ngx_palloc_large(pool, size);
|
||||||
|
-}
|
||||||
|
-
|
||||||
|
+ ngx_free(d->alloc);
|
||||||
|
+ ngx_free(d);
|
||||||
|
|
||||||
|
-void *
|
||||||
|
-ngx_pnalloc(ngx_pool_t *pool, size_t size)
|
||||||
|
-{
|
||||||
|
- u_char *m;
|
||||||
|
- ngx_pool_t *p;
|
||||||
|
-
|
||||||
|
- if (size <= pool->max) {
|
||||||
|
-
|
||||||
|
- p = pool->current;
|
||||||
|
-
|
||||||
|
- do {
|
||||||
|
- m = p->d.last;
|
||||||
|
-
|
||||||
|
- if ((size_t) (p->d.end - m) >= size) {
|
||||||
|
- p->d.last = m + size;
|
||||||
|
-
|
||||||
|
- return m;
|
||||||
|
+ if (n == NULL) {
|
||||||
|
+ break;
|
||||||
|
}
|
||||||
|
+ }
|
||||||
|
|
||||||
|
- p = p->d.next;
|
||||||
|
-
|
||||||
|
- } while (p);
|
||||||
|
-
|
||||||
|
- return ngx_palloc_block(pool, size);
|
||||||
|
+ pool->d = saved;
|
||||||
|
}
|
||||||
|
-
|
||||||
|
- return ngx_palloc_large(pool, size);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
-static void *
|
||||||
|
-ngx_palloc_block(ngx_pool_t *pool, size_t size)
|
||||||
|
+void *
|
||||||
|
+ngx_malloc(ngx_pool_t *pool, size_t size)
|
||||||
|
{
|
||||||
|
- u_char *m;
|
||||||
|
- size_t psize;
|
||||||
|
- ngx_pool_t *p, *new, *current;
|
||||||
|
+ ngx_pool_data_t *new;
|
||||||
|
+ void *m;
|
||||||
|
|
||||||
|
- psize = (size_t) (pool->d.end - (u_char *) pool);
|
||||||
|
-
|
||||||
|
- m = ngx_memalign(NGX_POOL_ALIGNMENT, psize, pool->log);
|
||||||
|
+ m = ngx_alloc(size, pool->log);
|
||||||
|
if (m == NULL) {
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
- new = (ngx_pool_t *) m;
|
||||||
|
-
|
||||||
|
- new->d.end = m + psize;
|
||||||
|
- new->d.next = NULL;
|
||||||
|
- new->d.failed = 0;
|
||||||
|
-
|
||||||
|
- m += sizeof(ngx_pool_data_t);
|
||||||
|
- m = ngx_align_ptr(m, NGX_ALIGNMENT);
|
||||||
|
- new->d.last = m + size;
|
||||||
|
-
|
||||||
|
- current = pool->current;
|
||||||
|
-
|
||||||
|
- for (p = current; p->d.next; p = p->d.next) {
|
||||||
|
- if (p->d.failed++ > 4) {
|
||||||
|
- current = p->d.next;
|
||||||
|
- }
|
||||||
|
+ new = ngx_alloc(sizeof(ngx_pool_data_t), pool->log);
|
||||||
|
+ if (new == NULL){
|
||||||
|
+ ngx_free(m);
|
||||||
|
+ return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
- p->d.next = new;
|
||||||
|
-
|
||||||
|
- pool->current = current ? current : new;
|
||||||
|
-
|
||||||
|
+ new->alloc = m;
|
||||||
|
+ new->next = pool->d;
|
||||||
|
+ pool->d = new;
|
||||||
|
return m;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
-static void *
|
||||||
|
-ngx_palloc_large(ngx_pool_t *pool, size_t size)
|
||||||
|
+void *
|
||||||
|
+ngx_palloc(ngx_pool_t *pool, size_t size)
|
||||||
|
{
|
||||||
|
- void *p;
|
||||||
|
- ngx_uint_t n;
|
||||||
|
- ngx_pool_large_t *large;
|
||||||
|
-
|
||||||
|
- p = ngx_alloc(size, pool->log);
|
||||||
|
- if (p == NULL) {
|
||||||
|
- return NULL;
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- n = 0;
|
||||||
|
-
|
||||||
|
- for (large = pool->large; large; large = large->next) {
|
||||||
|
- if (large->alloc == NULL) {
|
||||||
|
- large->alloc = p;
|
||||||
|
- return p;
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- if (n++ > 3) {
|
||||||
|
- break;
|
||||||
|
- }
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
- large = ngx_palloc(pool, sizeof(ngx_pool_large_t));
|
||||||
|
- if (large == NULL) {
|
||||||
|
- ngx_free(p);
|
||||||
|
- return NULL;
|
||||||
|
- }
|
||||||
|
+ return ngx_malloc(pool, size);
|
||||||
|
+}
|
||||||
|
|
||||||
|
- large->alloc = p;
|
||||||
|
- large->next = pool->large;
|
||||||
|
- pool->large = large;
|
||||||
|
|
||||||
|
- return p;
|
||||||
|
+void *
|
||||||
|
+ngx_pnalloc(ngx_pool_t *pool, size_t size)
|
||||||
|
+{
|
||||||
|
+ return ngx_malloc(pool, size);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@@ -263,7 +159,7 @@
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
- large = ngx_palloc(pool, sizeof(ngx_pool_large_t));
|
||||||
|
+ large = ngx_malloc(pool, sizeof(ngx_pool_large_t));
|
||||||
|
if (large == NULL) {
|
||||||
|
ngx_free(p);
|
||||||
|
return NULL;
|
||||||
|
@@ -278,16 +174,27 @@
|
||||||
|
|
||||||
|
|
||||||
|
ngx_int_t
|
||||||
|
-ngx_pfree(ngx_pool_t *pool, void *p)
|
||||||
|
+ngx_pfree(ngx_pool_t *pool, void *data)
|
||||||
|
{
|
||||||
|
- ngx_pool_large_t *l;
|
||||||
|
+ ngx_pool_data_t *p, *d;
|
||||||
|
|
||||||
|
- for (l = pool->large; l; l = l->next) {
|
||||||
|
- if (p == l->alloc) {
|
||||||
|
- ngx_log_debug1(NGX_LOG_DEBUG_ALLOC, pool->log, 0,
|
||||||
|
- "free: %p", l->alloc);
|
||||||
|
- ngx_free(l->alloc);
|
||||||
|
- l->alloc = NULL;
|
||||||
|
+ p = NULL;
|
||||||
|
+ for (d = pool->d; d; p = d, d = d->next) {
|
||||||
|
+ if (data == d->alloc) {
|
||||||
|
+
|
||||||
|
+ ngx_log_debug1(NGX_LOG_DEBUG_ALLOC, pool->log, 0, "free: %p", d->alloc);
|
||||||
|
+
|
||||||
|
+ ngx_free(d->alloc);
|
||||||
|
+ d->alloc = NULL;
|
||||||
|
+
|
||||||
|
+ if (p) {
|
||||||
|
+ p->next = d->next;
|
||||||
|
+
|
||||||
|
+ } else {
|
||||||
|
+ pool->d = d->next;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ ngx_free(d);
|
||||||
|
|
||||||
|
return NGX_OK;
|
||||||
|
}
|
||||||
|
diff -urx '*~' -x '*.swp' nginx-1.2.1/src/core/ngx_palloc.h nginx-1.2.1-patched/src/core/ngx_palloc.h
|
||||||
|
--- nginx-1.2.1/src/core/ngx_palloc.h 2012-02-06 04:02:59.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/core/ngx_palloc.h 2012-06-04 19:39:32.803578356 +0800
|
||||||
|
@@ -39,6 +39,8 @@
|
||||||
|
|
||||||
|
|
||||||
|
typedef struct ngx_pool_large_s ngx_pool_large_t;
|
||||||
|
+typedef struct ngx_pool_data_s ngx_pool_data_t;
|
||||||
|
+
|
||||||
|
|
||||||
|
struct ngx_pool_large_s {
|
||||||
|
ngx_pool_large_t *next;
|
||||||
|
@@ -46,16 +48,14 @@
|
||||||
|
};
|
||||||
|
|
||||||
|
|
||||||
|
-typedef struct {
|
||||||
|
- u_char *last;
|
||||||
|
- u_char *end;
|
||||||
|
- ngx_pool_t *next;
|
||||||
|
- ngx_uint_t failed;
|
||||||
|
-} ngx_pool_data_t;
|
||||||
|
+struct ngx_pool_data_s {
|
||||||
|
+ ngx_pool_data_t *next;
|
||||||
|
+ void *alloc;
|
||||||
|
+};
|
||||||
|
|
||||||
|
|
||||||
|
struct ngx_pool_s {
|
||||||
|
- ngx_pool_data_t d;
|
||||||
|
+ ngx_pool_data_t *d;
|
||||||
|
size_t max;
|
||||||
|
ngx_pool_t *current;
|
||||||
|
ngx_chain_t *chain;
|
@ -0,0 +1,113 @@
|
|||||||
|
--- src/http/modules/ngx_http_fastcgi_module.c
|
||||||
|
+++ src/http/modules/ngx_http_fastcgi_module.c
|
||||||
|
@@ -1501,10 +1501,10 @@ ngx_http_fastcgi_process_header(ngx_http
|
||||||
|
h->lowcase_key = h->key.data + h->key.len + 1
|
||||||
|
+ h->value.len + 1;
|
||||||
|
|
||||||
|
- ngx_cpystrn(h->key.data, r->header_name_start,
|
||||||
|
- h->key.len + 1);
|
||||||
|
- ngx_cpystrn(h->value.data, r->header_start,
|
||||||
|
- h->value.len + 1);
|
||||||
|
+ ngx_memcpy(h->key.data, r->header_name_start, h->key.len);
|
||||||
|
+ h->key.data[h->key.len] = '\0';
|
||||||
|
+ ngx_memcpy(h->value.data, r->header_start, h->value.len);
|
||||||
|
+ h->value.data[h->value.len] = '\0';
|
||||||
|
}
|
||||||
|
|
||||||
|
h->hash = r->header_hash;
|
||||||
|
--- src/http/modules/ngx_http_proxy_module.c
|
||||||
|
+++ src/http/modules/ngx_http_proxy_module.c
|
||||||
|
@@ -1381,8 +1381,10 @@ ngx_http_proxy_process_header(ngx_http_r
|
||||||
|
h->value.data = h->key.data + h->key.len + 1;
|
||||||
|
h->lowcase_key = h->key.data + h->key.len + 1 + h->value.len + 1;
|
||||||
|
|
||||||
|
- ngx_cpystrn(h->key.data, r->header_name_start, h->key.len + 1);
|
||||||
|
- ngx_cpystrn(h->value.data, r->header_start, h->value.len + 1);
|
||||||
|
+ ngx_memcpy(h->key.data, r->header_name_start, h->key.len);
|
||||||
|
+ h->key.data[h->key.len] = '\0';
|
||||||
|
+ ngx_memcpy(h->value.data, r->header_start, h->value.len);
|
||||||
|
+ h->value.data[h->value.len] = '\0';
|
||||||
|
|
||||||
|
if (h->key.len == r->lowcase_index) {
|
||||||
|
ngx_memcpy(h->lowcase_key, r->lowcase_header, h->key.len);
|
||||||
|
--- src/http/modules/ngx_http_scgi_module.c
|
||||||
|
+++ src/http/modules/ngx_http_scgi_module.c
|
||||||
|
@@ -941,8 +941,10 @@ ngx_http_scgi_process_header(ngx_http_re
|
||||||
|
h->value.data = h->key.data + h->key.len + 1;
|
||||||
|
h->lowcase_key = h->key.data + h->key.len + 1 + h->value.len + 1;
|
||||||
|
|
||||||
|
- ngx_cpystrn(h->key.data, r->header_name_start, h->key.len + 1);
|
||||||
|
- ngx_cpystrn(h->value.data, r->header_start, h->value.len + 1);
|
||||||
|
+ ngx_memcpy(h->key.data, r->header_name_start, h->key.len);
|
||||||
|
+ h->key.data[h->key.len] = '\0';
|
||||||
|
+ ngx_memcpy(h->value.data, r->header_start, h->value.len);
|
||||||
|
+ h->value.data[h->value.len] = '\0';
|
||||||
|
|
||||||
|
if (h->key.len == r->lowcase_index) {
|
||||||
|
ngx_memcpy(h->lowcase_key, r->lowcase_header, h->key.len);
|
||||||
|
--- src/http/modules/ngx_http_uwsgi_module.c
|
||||||
|
+++ src/http/modules/ngx_http_uwsgi_module.c
|
||||||
|
@@ -985,8 +985,10 @@ ngx_http_uwsgi_process_header(ngx_http_r
|
||||||
|
h->value.data = h->key.data + h->key.len + 1;
|
||||||
|
h->lowcase_key = h->key.data + h->key.len + 1 + h->value.len + 1;
|
||||||
|
|
||||||
|
- ngx_cpystrn(h->key.data, r->header_name_start, h->key.len + 1);
|
||||||
|
- ngx_cpystrn(h->value.data, r->header_start, h->value.len + 1);
|
||||||
|
+ ngx_memcpy(h->key.data, r->header_name_start, h->key.len);
|
||||||
|
+ h->key.data[h->key.len] = '\0';
|
||||||
|
+ ngx_memcpy(h->value.data, r->header_start, h->value.len);
|
||||||
|
+ h->value.data[h->value.len] = '\0';
|
||||||
|
|
||||||
|
if (h->key.len == r->lowcase_index) {
|
||||||
|
ngx_memcpy(h->lowcase_key, r->lowcase_header, h->key.len);
|
||||||
|
--- src/http/ngx_http_parse.c
|
||||||
|
+++ src/http/ngx_http_parse.c
|
||||||
|
@@ -874,6 +874,10 @@ ngx_http_parse_header_line(ngx_http_requ
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ if (ch == '\0') {
|
||||||
|
+ return NGX_HTTP_PARSE_INVALID_HEADER;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
r->invalid_header = 1;
|
||||||
|
|
||||||
|
break;
|
||||||
|
@@ -936,6 +940,10 @@ ngx_http_parse_header_line(ngx_http_requ
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ if (ch == '\0') {
|
||||||
|
+ return NGX_HTTP_PARSE_INVALID_HEADER;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
r->invalid_header = 1;
|
||||||
|
|
||||||
|
break;
|
||||||
|
@@ -954,6 +962,8 @@ ngx_http_parse_header_line(ngx_http_requ
|
||||||
|
r->header_start = p;
|
||||||
|
r->header_end = p;
|
||||||
|
goto done;
|
||||||
|
+ case '\0':
|
||||||
|
+ return NGX_HTTP_PARSE_INVALID_HEADER;
|
||||||
|
default:
|
||||||
|
r->header_start = p;
|
||||||
|
state = sw_value;
|
||||||
|
@@ -975,6 +985,8 @@ ngx_http_parse_header_line(ngx_http_requ
|
||||||
|
case LF:
|
||||||
|
r->header_end = p;
|
||||||
|
goto done;
|
||||||
|
+ case '\0':
|
||||||
|
+ return NGX_HTTP_PARSE_INVALID_HEADER;
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
|
||||||
|
@@ -988,6 +1000,8 @@ ngx_http_parse_header_line(ngx_http_requ
|
||||||
|
break;
|
||||||
|
case LF:
|
||||||
|
goto done;
|
||||||
|
+ case '\0':
|
||||||
|
+ return NGX_HTTP_PARSE_INVALID_HEADER;
|
||||||
|
default:
|
||||||
|
state = sw_value;
|
||||||
|
break;
|
@ -0,0 +1,20 @@
|
|||||||
|
--- nginx-1.2.1/src/os/unix/ngx_process_cycle.c 2012-02-06 04:02:59.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/os/unix/ngx_process_cycle.c 2012-06-15 11:39:11.234286287 +0800
|
||||||
|
@@ -707,6 +707,8 @@
|
||||||
|
ngx_exit_log.file = &ngx_exit_log_file;
|
||||||
|
|
||||||
|
ngx_exit_cycle.log = &ngx_exit_log;
|
||||||
|
+ ngx_exit_cycle.files = ngx_cycle->files;
|
||||||
|
+ ngx_exit_cycle.files_n = ngx_cycle->files_n;
|
||||||
|
ngx_cycle = &ngx_exit_cycle;
|
||||||
|
|
||||||
|
ngx_destroy_pool(cycle->pool);
|
||||||
|
@@ -1063,6 +1065,8 @@
|
||||||
|
ngx_exit_log.file = &ngx_exit_log_file;
|
||||||
|
|
||||||
|
ngx_exit_cycle.log = &ngx_exit_log;
|
||||||
|
+ ngx_exit_cycle.files = ngx_cycle->files;
|
||||||
|
+ ngx_exit_cycle.files_n = ngx_cycle->files_n;
|
||||||
|
ngx_cycle = &ngx_exit_cycle;
|
||||||
|
|
||||||
|
ngx_destroy_pool(cycle->pool);
|
@ -0,0 +1,27 @@
|
|||||||
|
# HG changeset patch
|
||||||
|
# User Maxim Dounin <mdounin@mdounin.ru>
|
||||||
|
# Date 1309799136 -14400
|
||||||
|
# Node ID 99e276bba8596bc4df9e638482ee413f4c6bf700
|
||||||
|
# Parent e7b2f945d55ae44a2295facf9e3336dc4633e5b5
|
||||||
|
Core: fix body with request_body_in_single_buf.
|
||||||
|
|
||||||
|
If there were preread data and request body was big enough first part
|
||||||
|
of request body was duplicated.
|
||||||
|
|
||||||
|
See report here:
|
||||||
|
http://nginx.org/pipermail/nginx/2011-July/027756.html
|
||||||
|
|
||||||
|
diff --git a/src/http/ngx_http_request_body.c b/src/http/ngx_http_request_body.c
|
||||||
|
--- a/src/http/ngx_http_request_body.c
|
||||||
|
+++ b/src/http/ngx_http_request_body.c
|
||||||
|
@@ -372,7 +372,9 @@ ngx_http_do_read_client_request_body(ngx
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
- if (r->request_body_in_file_only && rb->bufs->next) {
|
||||||
|
+ if (rb->bufs->next
|
||||||
|
+ && (r->request_body_in_file_only || r->request_body_in_single_buf))
|
||||||
|
+ {
|
||||||
|
rb->bufs = rb->bufs->next;
|
||||||
|
}
|
||||||
|
|
@ -0,0 +1,17 @@
|
|||||||
|
# HG changeset patch
|
||||||
|
# User Maxim Dounin <mdounin@mdounin.ru>
|
||||||
|
# Date 1309776931 -14400
|
||||||
|
# Node ID e7b2f945d55ae44a2295facf9e3336dc4633e5b5
|
||||||
|
# Parent 610e909bb9e29766188aa86fae3abe0bd3432940
|
||||||
|
Core: fix body if it's preread and there are extra data.
|
||||||
|
|
||||||
|
--- nginx-1.2.1/src/http/ngx_http_request_body.c 2011-07-05 12:11:21.619264633 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_request_body.c 2011-07-05 12:14:30.694321554 +0800
|
||||||
|
@@ -141,6 +141,7 @@
|
||||||
|
|
||||||
|
/* the whole request body was pre-read */
|
||||||
|
|
||||||
|
+ b->last = b->pos + r->headers_in.content_length_n;
|
||||||
|
r->header_in->pos += (size_t) r->headers_in.content_length_n;
|
||||||
|
r->request_length += r->headers_in.content_length_n;
|
||||||
|
|
@ -0,0 +1,11 @@
|
|||||||
|
--- nginx-1.2.1/src/core/ngx_resolver.c 2012-03-05 21:17:56.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/core/ngx_resolver.c 2012-06-01 17:55:55.861130605 +0800
|
||||||
|
@@ -987,7 +987,7 @@
|
||||||
|
nan = (query->nan_hi << 8) + query->nan_lo;
|
||||||
|
|
||||||
|
ngx_log_debug6(NGX_LOG_DEBUG_CORE, r->log, 0,
|
||||||
|
- "resolver DNS response %ui fl:%04Xui %ui/%ui/%ui/%ui",
|
||||||
|
+ "resolver DNS response %ui fl:%04Xui %ui/%ui/%ud/%ud",
|
||||||
|
ident, flags, nqs, nan,
|
||||||
|
(query->nns_hi << 8) + query->nns_lo,
|
||||||
|
(query->nar_hi << 8) + query->nar_lo);
|
@ -0,0 +1,26 @@
|
|||||||
|
diff -ur lz-nginx-1.2.1/nginx-1.2.1/src/core/nginx.h lz-nginx-1.2.1-patched/nginx-1.2.1/src/core/nginx.h
|
||||||
|
--- lz-nginx-1.2.1/nginx-1.2.1/src/core/nginx.h 2010-02-12 17:31:01.000000000 +0800
|
||||||
|
+++ lz-nginx-1.2.1-patched/nginx-1.2.1/src/core/nginx.h 2010-03-30 10:52:13.240702627 +0800
|
||||||
|
@@ -10,7 +10,7 @@
|
||||||
|
|
||||||
|
#define nginx_version 1002001
|
||||||
|
#define NGINX_VERSION "1.2.1"
|
||||||
|
-#define NGINX_VER "nginx/" NGINX_VERSION
|
||||||
|
+#define NGINX_VER "ngx_openresty/" NGINX_VERSION ".unknown"
|
||||||
|
|
||||||
|
#define NGINX_VAR "NGINX"
|
||||||
|
#define NGX_OLDPID_EXT ".oldbin"
|
||||||
|
Only in lz-nginx-1.2.1-patched/nginx-1.2.1/src/core: nginx.h.orig
|
||||||
|
Only in lz-nginx-1.2.1-patched/nginx-1.2.1/src/core: nginx.h.rej
|
||||||
|
diff -ur lz-nginx-1.2.1/nginx-1.2.1/src/http/ngx_http_header_filter_module.c lz-nginx-1.2.1-patched/nginx-1.2.1/src/http/ngx_http_header_filter_module.c
|
||||||
|
--- lz-nginx-1.2.1/nginx-1.2.1/src/http/ngx_http_header_filter_module.c 2010-03-03 23:14:04.000000000 +0800
|
||||||
|
+++ lz-nginx-1.2.1-patched/nginx-1.2.1/src/http/ngx_http_header_filter_module.c 2010-03-30 10:52:53.670909405 +0800
|
||||||
|
@@ -45,7 +45,7 @@
|
||||||
|
};
|
||||||
|
|
||||||
|
|
||||||
|
-static char ngx_http_server_string[] = "Server: nginx" CRLF;
|
||||||
|
+static char ngx_http_server_string[] = "Server: ngx_openresty" CRLF;
|
||||||
|
static char ngx_http_server_full_string[] = "Server: " NGINX_VER CRLF;
|
||||||
|
|
||||||
|
|
@ -0,0 +1,32 @@
|
|||||||
|
diff -ur nginx-1.2.1/src/http/ngx_http_upstream.c nginx-1.2.1-patched/src/http/ngx_http_upstream.c
|
||||||
|
--- nginx-1.2.1/src/http/ngx_http_upstream.c 2011-12-14 02:34:34.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_upstream.c 2012-03-21 21:20:17.333111806 +0800
|
||||||
|
@@ -1385,6 +1385,8 @@
|
||||||
|
|
||||||
|
/* rc == NGX_OK */
|
||||||
|
|
||||||
|
+ u->request_all_sent = 1;
|
||||||
|
+
|
||||||
|
if (c->tcp_nopush == NGX_TCP_NOPUSH_SET) {
|
||||||
|
if (ngx_tcp_push(c->fd) == NGX_ERROR) {
|
||||||
|
ngx_log_error(NGX_LOG_CRIT, c->log, ngx_socket_errno,
|
||||||
|
@@ -1451,7 +1453,7 @@
|
||||||
|
|
||||||
|
#endif
|
||||||
|
|
||||||
|
- if (u->header_sent) {
|
||||||
|
+ if (u->request_all_sent) {
|
||||||
|
u->write_event_handler = ngx_http_upstream_dummy_handler;
|
||||||
|
|
||||||
|
(void) ngx_handle_write_event(c->write, 0);
|
||||||
|
diff -ur nginx-1.2.1/src/http/ngx_http_upstream.h nginx-1.2.1-patched/src/http/ngx_http_upstream.h
|
||||||
|
--- nginx-1.2.1/src/http/ngx_http_upstream.h 2011-11-01 22:18:10.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_upstream.h 2012-03-21 21:18:21.041237173 +0800
|
||||||
|
@@ -313,6 +313,7 @@
|
||||||
|
unsigned buffering:1;
|
||||||
|
|
||||||
|
unsigned request_sent:1;
|
||||||
|
+ unsigned request_all_sent:1;
|
||||||
|
unsigned header_sent:1;
|
||||||
|
};
|
||||||
|
|
@ -0,0 +1,73 @@
|
|||||||
|
--- nginx-1.2.1/src/http/ngx_http_variables.c 2011-05-30 20:36:17.000000000 +0800
|
||||||
|
+++ nginx-1.2.1-patched/src/http/ngx_http_variables.c 2011-11-08 22:21:55.229247198 +0800
|
||||||
|
@@ -648,7 +648,17 @@
|
||||||
|
|
||||||
|
a = (ngx_array_t *) ((char *) r + data);
|
||||||
|
|
||||||
|
- n = a->nelts;
|
||||||
|
+ h = a->elts;
|
||||||
|
+ n = 0;
|
||||||
|
+
|
||||||
|
+ for (i = 0; i < a->nelts; i++) {
|
||||||
|
+
|
||||||
|
+ if (h[i]->hash == 0) {
|
||||||
|
+ continue;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ n++;
|
||||||
|
+ }
|
||||||
|
|
||||||
|
if (n == 0) {
|
||||||
|
v->not_found = 1;
|
||||||
|
@@ -659,9 +669,7 @@
|
||||||
|
v->no_cacheable = 0;
|
||||||
|
v->not_found = 0;
|
||||||
|
|
||||||
|
- h = a->elts;
|
||||||
|
-
|
||||||
|
- if (n == 1) {
|
||||||
|
+ if (n == 1 && a->nelts == 1) {
|
||||||
|
v->len = (*h)->value.len;
|
||||||
|
v->data = (*h)->value.data;
|
||||||
|
|
||||||
|
@@ -670,7 +678,12 @@
|
||||||
|
|
||||||
|
len = - (ssize_t) (sizeof("; ") - 1);
|
||||||
|
|
||||||
|
- for (i = 0; i < n; i++) {
|
||||||
|
+ for (i = 0; i < a->nelts; i++) {
|
||||||
|
+
|
||||||
|
+ if (h[i]->hash == 0) {
|
||||||
|
+ continue;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
len += h[i]->value.len + sizeof("; ") - 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -683,9 +696,14 @@
|
||||||
|
v->data = p;
|
||||||
|
|
||||||
|
for (i = 0; /* void */ ; i++) {
|
||||||
|
+
|
||||||
|
+ if (h[i]->hash == 0) {
|
||||||
|
+ continue;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
p = ngx_copy(p, h[i]->value.data, h[i]->value.len);
|
||||||
|
|
||||||
|
- if (i == n - 1) {
|
||||||
|
+ if (--n == 0) {
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -738,6 +756,10 @@
|
||||||
|
i = 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ if (header[i].hash == 0) {
|
||||||
|
+ continue;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
for (n = 0; n + prefix < var->len && n < header[i].key.len; n++) {
|
||||||
|
ch = header[i].key.data[n];
|
||||||
|
|
Loading…
Reference in New Issue