[nginx] svn commit: r5078 - in trunk/src/http: . modules

vbart at nginx.com vbart at nginx.com
Tue Feb 19 17:48:46 UTC 2013


Author: vbart
Date: 2013-02-19 17:48:45 +0000 (Tue, 19 Feb 2013)
New Revision: 5078
URL: http://trac.nginx.org/nginx/changeset/5078/nginx

Log:
Removed zero termination of shm zone names.

It was added in r2717 and no longer needed since r2721,
where the termination was added to ngx_shm_alloc() and
ngx_init_zone_pool().  So then it only corrupts error
messages about ivalid zones.


Modified:
   trunk/src/http/modules/ngx_http_ssl_module.c
   trunk/src/http/ngx_http_file_cache.c

Modified: trunk/src/http/modules/ngx_http_ssl_module.c
===================================================================
--- trunk/src/http/modules/ngx_http_ssl_module.c	2013-02-19 17:45:12 UTC (rev 5077)
+++ trunk/src/http/modules/ngx_http_ssl_module.c	2013-02-19 17:48:45 UTC (rev 5078)
@@ -661,7 +661,6 @@
 
             for (j = sizeof("shared:") - 1; j < value[i].len; j++) {
                 if (value[i].data[j] == ':') {
-                    value[i].data[j] = '\0';
                     break;
                 }
 

Modified: trunk/src/http/ngx_http_file_cache.c
===================================================================
--- trunk/src/http/ngx_http_file_cache.c	2013-02-19 17:45:12 UTC (rev 5077)
+++ trunk/src/http/ngx_http_file_cache.c	2013-02-19 17:48:45 UTC (rev 5078)
@@ -1674,8 +1674,6 @@
             p = (u_char *) ngx_strchr(name.data, ':');
 
             if (p) {
-                *p = '\0';
-
                 name.len = p - name.data;
 
                 p++;



More information about the nginx-devel mailing list