Merge pull request #2834 from martinhsv/v2/master

Fix: handle error with SecConnReadStateLimit configuration

Author: martinhsv

CHANGES

@@ -1,6 +1,8 @@
 DD mmm YYYY - 2.9.x (to be released)
 -------------------
 
+ * Fix: handle error with SecConnReadStateLimit configuration
+   [Issue #2815, #2834 - @marcstern, @martinhsv]
  * Only check for pcre2 install if required
    [Issue #2833 - @martinhsv]
  * Adjustment of previous fix for log messages

apache2/apache2_config.c

@@ -1732,6 +1732,12 @@ char *parser_conn_limits_operator(apr_pool_t *mp, const char *p2,
     char *param = strchr(p2, ' ');
     char *file = NULL;
     char *error_msg = NULL;
+
+    if (param == NULL) {
+        return apr_psprintf(mp, "ModSecurity: Space character between operator " \
+           "and parameter not found with ConnReadStateLimit: %s", p2);
+    }
+
     param++;
 
     config_orig_path = apr_pstrndup(mp, filename,