2 silly bugs. First of all, make the code match the error log

(and allow 8192 to be valid). Secondly, this missplaced else
made the size part (8192) non-optional for shm:
PR:
Obtained from:
Submitted by:
Reviewed by:

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@97281 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/modules/ssl/ssl_engine_config.c b/modules/ssl/ssl_engine_config.c
index 1350e4fc566faae5e601637c6e40eeb94ff45a88..57fdc3e88bf98803a9bf5adedf87e795b7e68566 100644 (file)
--- a/modules/ssl/ssl_engine_config.c
+++ b/modules/ssl/ssl_engine_config.c
@@ -968,7 +968,7 @@ const char *ssl_cmd_SSLSessionCache(cmd_parms *cmd,
 
             mc->nSessionCacheDataSize = atoi(cp);
 
-            if (mc->nSessionCacheDataSize <= 8192) {
+            if (mc->nSessionCacheDataSize < 8192) {
                 return "SSLSessionCache: Invalid argument: "
                        "size has to be >= 8192 bytes";
             }
@@ -1010,7 +1010,7 @@ const char *ssl_cmd_SSLSessionCache(cmd_parms *cmd,
 
             mc->nSessionCacheDataSize = atoi(cp);
 
-            if (mc->nSessionCacheDataSize <= 8192) {
+            if (mc->nSessionCacheDataSize < 8192) {
                 return "SSLSessionCache: Invalid argument: "
                        "size has to be >= 8192 bytes";
 
@@ -1024,9 +1024,9 @@ const char *ssl_cmd_SSLSessionCache(cmd_parms *cmd,
 
             }
         }
-       else {
-            return "SSLSessionCache: Invalid argument";
-        }
+    }
+    else {
+        return "SSLSessionCache: Invalid argument";
     }
 
     return NULL;