From: Mike Stepanek (mstepane) <mstepane@cisco.com>
Date: Tue, 14 Jun 2022 14:13:44 +0000 (+0000)
Subject: Pull request #3463: Fixit handling
X-Git-Tag: 3.1.32.0~3
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5fe5f3316ec24273bf5fe1b5e1437804c0a1e42a;p=thirdparty%2Fsnort3.git

Pull request #3463: Fixit handling

Merge in SNORT/snort3 from ~DKYRYLOV/snort3:fixit_handling to master

Squashed commit of the following:

commit 0941456ee529069d60f8edd1725ccbda3dbfb015
Author: dkyrylov <dkyrylov@cisco.com>
Date:   Tue Jun 7 12:58:02 2022 +0300

    utils, parser: remove redundant fixits
       Remove FIXIT-L@js_normalizer_test.cc:4249
       US created to handle the fixit

       Remove FIXIT-M@parser.cc:702
       The issue was handled in 2414d8b9d22,
       but the FIXIT comment was left

       Reword FIXIT-M@parse-conf.cc:210
---

diff --git a/src/parser/parse_conf.cc b/src/parser/parse_conf.cc
index e5e95f647..5efc9a83b 100644
--- a/src/parser/parse_conf.cc
+++ b/src/parser/parse_conf.cc
@@ -207,8 +207,9 @@ void parse_include(SnortConfig* sc, const char* arg)
 void ParseIpVar(const char* var, const char* value)
 {
     int ret;
-    IpsPolicy* p = get_ips_policy();  // FIXIT-M double check, see below
-    DisallowCrossTableDuplicateVars(var, VAR_TYPE__IPVAR);
+    IpsPolicy* p = get_ips_policy();
+    DisallowCrossTableDuplicateVars(var, VAR_TYPE__IPVAR); 
+    // FIXIT-M: ip checked for duplicates twice: in the function above and in sfvt_add_str
 
     if ((ret = sfvt_define(p->ip_vartable, var, value)) != SFIP_SUCCESS)
     {
diff --git a/src/parser/parser.cc b/src/parser/parser.cc
index 84620e28e..0ae4e5899 100644
--- a/src/parser/parser.cc
+++ b/src/parser/parser.cc
@@ -698,7 +698,7 @@ void OrderRuleLists(SnortConfig* sc)
     const char* order = sc->rule_order.c_str();
     if ( !*order )
     {
-        default_priorities = Actions::get_default_priorities();  // FIXIT-M apply builtin module defaults
+        default_priorities = Actions::get_default_priorities();
         order = default_priorities.c_str();
     }
 
diff --git a/src/utils/test/js_normalizer_test.cc b/src/utils/test/js_normalizer_test.cc
index 5449a3748..906657e66 100644
--- a/src/utils/test/js_normalizer_test.cc
+++ b/src/utils/test/js_normalizer_test.cc
@@ -4204,9 +4204,6 @@ TEST_CASE("ignored properties", "[JSNormalizer]")
         const char dat19[] = "foo . + unwatch() . bar ;";
         const char dat20[] = "foo . + ['unwatch']() . bar ;";
 
-        // FIXIT-L: add support for proper tracking of bracket accessors.
-        // Current behavior: foo['watch'].bar -> var_0000['watch'].var_0001
-        // Expected behavior: foo['watch'].bar -> var_0000['watch'].bar
         const char exp1[] = "var_0000.watch.bar;";
         const char exp2[] = "var_0000['watch'].var_0001;";
         const char exp3[] = "var_0000.var_0001.watch.bar;";