From: Mike Stepanek (mstepane) <mstepane@cisco.com>
Date: Wed, 24 Nov 2021 12:59:44 +0000 (+0000)
Subject: Pull request #3178: Value::get_long(), replacing with platform-independent type
X-Git-Tag: 3.1.18.0~14
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c4a9137a2143196d5f47e8cceb3267f4f8042f95;p=thirdparty%2Fsnort3.git

Pull request #3178: Value::get_long(), replacing with platform-independent type

Merge in SNORT/snort3 from ~OSHUMEIK/snort3:fix_32_64 to master

Squashed commit of the following:

commit 5faafb2d57279064269cb3a58d1b136fd3742d44
Author: Oleksii Shumeiko <oshumeik@cisco.com>
Date:   Fri Nov 19 16:12:50 2021 +0200

    framework: replace Value::get_long() with a platform-independent type
---

diff --git a/src/dump_config/json_config_output.cc b/src/dump_config/json_config_output.cc
index 3469d105c..e1da2db64 100644
--- a/src/dump_config/json_config_output.cc
+++ b/src/dump_config/json_config_output.cc
@@ -41,7 +41,7 @@ static void dump_value(JsonStream& json, const char* node_name, const BaseConfig
             json.put_false(node_name);
         break;
     case Parameter::PT_INT:
-        json.put(node_name, value->get_long());
+        json.put(node_name, value->get_int64());
         break;
     case Parameter::PT_REAL:
     {
diff --git a/src/dump_config/text_config_output.cc b/src/dump_config/text_config_output.cc
index 6d728d32d..c0090e2fb 100644
--- a/src/dump_config/text_config_output.cc
+++ b/src/dump_config/text_config_output.cc
@@ -49,7 +49,7 @@ static void dump_value(const BaseConfigNode* node, const std::string& config_nam
         break;
     }
     case Parameter::PT_INT:
-        std::cout << config_name << "=" << value->get_long() << std::endl;
+        std::cout << config_name << "=" << value->get_int64() << std::endl;
         break;
     case Parameter::PT_REAL:
         std::cout << config_name << "=" << value->get_real() << std::endl;
diff --git a/src/framework/value.h b/src/framework/value.h
index 0837e6b53..a19b6055d 100644
--- a/src/framework/value.h
+++ b/src/framework/value.h
@@ -121,9 +121,6 @@ public:
     bool get_bool() const
     { return num != 0; }
 
-    long get_long() const  // FIXIT-L to be removed
-    { return (long)num; }
-
     size_t get_size() const
     { return (size_t)num; }
 
diff --git a/src/helpers/json_stream.cc b/src/helpers/json_stream.cc
index 28021108d..39d34010e 100644
--- a/src/helpers/json_stream.cc
+++ b/src/helpers/json_stream.cc
@@ -88,7 +88,7 @@ void JsonStream::put(const char* key)
     out << "null";
 }
 
-void JsonStream::put(const char* key, long val)
+void JsonStream::put(const char* key, int64_t val)
 {
     split();
 
diff --git a/src/helpers/json_stream.h b/src/helpers/json_stream.h
index fa7ea2632..b6b85bd1c 100644
--- a/src/helpers/json_stream.h
+++ b/src/helpers/json_stream.h
@@ -40,7 +40,7 @@ public:
     void close_array();
 
     void put(const char* key);    // null
-    void put(const char* key, long val);
+    void put(const char* key, int64_t val);
     void put(const char* key, const char* val);
     void put(const char* key, const std::string& val);
     void put(const char* key, double val, int precision);
diff --git a/src/main/snort_module.cc b/src/main/snort_module.cc
index 9e0a35f12..360c9a47e 100644
--- a/src/main/snort_module.cc
+++ b/src/main/snort_module.cc
@@ -845,7 +845,7 @@ bool SnortModule::set(const char*, Value& v, SnortConfig* sc)
         module_config = sc->daq_config->add_module_config(v.get_string());
 
     else if ( v.is("--daq-batch-size") )
-        sc->daq_config->set_batch_size(v.get_long());
+        sc->daq_config->set_batch_size(v.get_uint32());
 
     else if ( v.is("--daq-dir") )
     {
@@ -859,7 +859,7 @@ bool SnortModule::set(const char*, Value& v, SnortConfig* sc)
     {
         if (!module_config)
             return false;
-        switch ( v.get_long() )
+        switch ( v.get_uint8() )
         {
             case 0:
                 module_config->mode = SFDAQModuleConfig::SFDAQ_MODE_PASSIVE;
diff --git a/src/packet_io/sfdaq_module.cc b/src/packet_io/sfdaq_module.cc
index 87d760bd4..5e875c642 100644
--- a/src/packet_io/sfdaq_module.cc
+++ b/src/packet_io/sfdaq_module.cc
@@ -120,7 +120,7 @@ bool SFDAQModule::set(const char* fqn, Value& v, SnortConfig*)
     }
     else if (!strcmp(fqn, "daq.batch_size"))
     {
-        config->set_batch_size(v.get_long());
+        config->set_batch_size(v.get_uint32());
     }
     else if (!strcmp(fqn, "daq.modules.name"))
     {
@@ -128,7 +128,7 @@ bool SFDAQModule::set(const char* fqn, Value& v, SnortConfig*)
     }
     else if (!strcmp(fqn, "daq.modules.mode"))
     {
-        module_config->mode = (SFDAQModuleConfig::SFDAQMode) (v.get_long() + 1);
+        module_config->mode = (SFDAQModuleConfig::SFDAQMode) (v.get_uint8() + 1);
     }
     else if (!strcmp(fqn, "daq.modules.variables"))
     {