Limit req: rate should be non-zero.

vbart · vbart · commit a3035758763a · 2013-04-03T14:13:35.000Z
Specifying zero rate caused division by zero when calculating delays.
diff --git a/src/http/modules/ngx_http_limit_req_module.c b/src/http/modules/ngx_http_limit_req_module.c
@@ -795,7 +795,7 @@ ngx_http_limit_req_zone(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
             }
 
             rate = ngx_atoi(value[i].data + 5, len - 5);
-            if (rate <= NGX_ERROR) {
+            if (rate <= 0) {
                 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
                                    "invalid rate \"%V\"", &value[i]);
                 return NGX_CONF_ERROR;

Original file line number	Diff line number	Diff line change
`@@ -795,7 +795,7 @@ ngx_http_limit_req_zone(ngx_conf_t cf, ngx_command_t cmd, void *conf)`
`795`	`795`	`}`
`796`	`796`
`797`	`797`	`rate = ngx_atoi(value[i].data + 5, len - 5);`
`798`		`- if (rate <= NGX_ERROR) {`
	`798`	`+ if (rate <= 0) {`
`799`	`799`	`ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,`
`800`	`800`	`"invalid rate \"%V\"", &value[i]);`
`801`	`801`	`return NGX_CONF_ERROR;`