]> git.ipfire.org Git - thirdparty/snort3.git/commitdiff
Pull request #4068: http_inspect: add correct handling of configuration error
authorYehor Velykozhon -X (yvelykoz - SOFTSERVE INC at Cisco) <yvelykoz@cisco.com>
Thu, 26 Oct 2023 15:11:05 +0000 (15:11 +0000)
committerOleksii Shumeiko -X (oshumeik - SOFTSERVE INC at Cisco) <oshumeik@cisco.com>
Thu, 26 Oct 2023 15:11:05 +0000 (15:11 +0000)
Merge in SNORT/snort3 from ~YVELYKOZ/snort3:http_param_fix to master

Squashed commit of the following:

commit 2a15f0c5742bd014c9152620b68158db81237637
Author: Yehor Velykozhon <yvelykoz@cisco.com>
Date:   Wed Oct 25 14:02:35 2023 +0300

    http_inspect: add correct handling of configuration error

src/service_inspectors/http_inspect/ips_http_param.cc

index 55c2f7185009dbc4b98dbf4287aacb131a5c3f22..1404419420d199b97bd8b1a362c88a4500419aab 100644 (file)
@@ -80,7 +80,11 @@ uint32_t HttpParamIpsOption::hash() const
 bool HttpParamRuleOptModule::end(const char*, int, SnortConfig*)
 {
     if (param.length() == 0)
+    {
         ParseError("Specify parameter name");
+        return false;
+    }
+
     return true;
 }