changeset 1157:a4820184a618

*) introduce ngx_strchr() *) test server_name for '/': it's common configuration error when trailing ';' is omitted and a next directive is treated as server_name
author Igor Sysoev <igor@sysoev.ru>
date Mon, 02 Apr 2007 06:27:30 +0000
parents a9005d2e2c99
children d1ffcf84ea12
files src/core/ngx_string.h src/http/ngx_http_core_module.c
diffstat 2 files changed, 7 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/core/ngx_string.h	Mon Apr 02 05:43:21 2007 +0000
+++ b/src/core/ngx_string.h	Mon Apr 02 06:27:30 2007 +0000
@@ -40,6 +40,7 @@
 
 
 #define ngx_strstr(s1, s2)  strstr((const char *) s1, (const char *) s2)
+#define ngx_strchr(s1, c)   strchr((const char *) s1, (int) c)
 #define ngx_strlen(s)       strlen((const char *) s)
 
 
--- a/src/http/ngx_http_core_module.c	Mon Apr 02 05:43:21 2007 +0000
+++ b/src/http/ngx_http_core_module.c	Mon Apr 02 06:27:30 2007 +0000
@@ -2614,6 +2614,12 @@
             return NGX_CONF_ERROR;
         }
 
+        if (ngx_strchr(value[i].data, '/')) {
+            ngx_conf_log_error(NGX_LOG_WARN, cf, 0,
+                               "server name \"%V\" has strange symbols",
+                               &value[i]);
+        }
+
         sn = ngx_array_push(&cscf->server_names);
         if (sn == NULL) {
             return NGX_CONF_ERROR;