commit ff63d7fcf97de55e717f94f7bcfb1642da298e72
author Gena Makhomed <gmm@csdoc.com> Mon Sep 24 20:26:46 2018 +0300
committer Gena Makhomed <gmm@csdoc.com> Mon Sep 24 20:26:46 2018 +0300
tree cf23c29201ffa1ee5f811480e77d0ce40ff46e1d
parent 6fb331ec39344719a7f1d936ad09f0c470b69ab9

Cache: status must be less then 599 in *_cache_valid directives.

Previously, configurations with typo, for example

    fastcgi_cache_valid 200301 302 5m;

successfully pass configuration test. Adding check for status
codes > 599, and such configurations are now properly rejected.

src/http/ngx_http_file_cache.c

diff --git a/src/http/ngx_http_file_cache.c b/src/http/ngx_http_file_cache.c
index 56866fa..330833d 100644
--- a/src/http/ngx_http_file_cache.c
+++ b/src/http/ngx_http_file_cache.c
@@ -2669,7 +2669,7 @@
         } else {
 
             status = ngx_atoi(value[i].data, value[i].len);
-            if (status < 100) {
+            if (status < 100 || status > 599) {
                 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
                                    "invalid status \"%V\"", &value[i]);
                 return NGX_CONF_ERROR;