commit ec582de2dd1ea4e55b0a326e1931593882593ba2
author Igor Sysoev <igor@sysoev.ru> Thu Jan 11 16:50:06 2007 +0000
committer Igor Sysoev <igor@sysoev.ru> Thu Jan 11 16:50:06 2007 +0000
tree b29b43aea990adb3d6ace84689d514e8b76f4dfb
parent 21dd4be0c0debea5f21b9991e8f7dd84a06beaa9

test length of variable and number of connections

src/http/modules/ngx_http_limit_zone_module.c

diff --git a/src/http/modules/ngx_http_limit_zone_module.c b/src/http/modules/ngx_http_limit_zone_module.c
index 1ec99cc..abab4ad 100644
--- a/src/http/modules/ngx_http_limit_zone_module.c
+++ b/src/http/modules/ngx_http_limit_zone_module.c
@@ -131,10 +131,22 @@
         return NGX_DECLINED;
     }
 
-    r->main->limit_zone_set = 1;
-
     len = vv->len;
 
+    if (len == 0) {
+        return NGX_DECLINED;
+    }
+
+    if (len > 255) {
+        ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
+                      "the value of the \"%V\" variable "
+                      "is more than 255 bytes: \"%V\"",
+                      &ctx->var, vv);
+        return NGX_DECLINED;
+    }
+
+    r->main->limit_zone_set = 1;
+
     hash = ngx_crc32_short(vv->data, len);
 
     cln = ngx_pool_cleanup_add(r->pool, sizeof(ngx_http_limit_zone_cleanup_t));
@@ -419,6 +431,12 @@
         return NGX_CONF_ERROR;
     }
 
+    if (n > 65535) {
+        ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
+                           "connection limit must be less 65536");
+        return NGX_CONF_ERROR;
+    }
+
     lzcf->conn = n;
 
     return NGX_CONF_OK;