Mercurial > hg > nginx

changeset 1157:a4820184a618

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
*) 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
+++ b/src/core/ngx_string.h
@@ -40,6 +40,7 @@ typedef struct {
 
 
 #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
+++ b/src/http/ngx_http_core_module.c
@@ -2614,6 +2614,12 @@ ngx_http_core_server_name(ngx_conf_t *cf
             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;