]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
detect/dataset: fix space condition in rule lang
authorShivani Bhardwaj <shivanib134@gmail.com>
Fri, 28 Jan 2022 20:17:17 +0000 (01:47 +0530)
committerVictor Julien <vjulien@oisf.net>
Wed, 27 Apr 2022 09:14:27 +0000 (11:14 +0200)
If there is a space following a keyword that does not expect a value,
the rule fails to load due to improper value evaluation.
e.g. Space after "set" command
alert http any any -> any any (http.user_agent; dataset:set  ,ua-seen,type string,save datasets.csv; sid:1;)

gives error
[ERRCODE: SC_ERR_UNKNOWN_VALUE(129)] - dataset action "" is not supported.

Fix this by handling values correctly for such cases.

(cherry picked from commit 6d2a2a073120906304f70c0e120565eae96e36b8)

src/detect-dataset.c

index b29d4eb1df7a1a1420af9258d1aa36c255cd0726..ad579b0b2f4ef8806938ad91649779392378f3fc 100644 (file)
@@ -131,13 +131,13 @@ static int DetectDatasetParse(const char *str,
         }
 
         if (!cmd_set) {
-            if (val) {
+            if (val && strlen(val) != 0) {
                 return -1;
             }
             strlcpy(cmd, key, cmd_len);
             cmd_set = true;
         } else if (!name_set) {
-            if (val) {
+            if (val && strlen(val) != 0) {
                 return -1;
             }
             strlcpy(name, key, name_len);