Merge pull request #1351 in SNORT/snort3 from valgrind_reputation_init to master

Squashed commit of the following:

commit 0944be2fb6ec5eed346db9881102dd0396b460e4
Author: Masud Hasan <mashasan@cisco.com>
Date:   Fri Aug 31 15:40:51 2018 -0400

    reputation: Early return on parsing error causing uninitialized id

diff --git a/src/network_inspectors/reputation/reputation_inspect.cc b/src/network_inspectors/reputation/reputation_inspect.cc
index 5a80035b4bd7986d5d3ceed5ca7188e5f1dac71b..270a2741d47d7846965b22e2088655647e75633c 100644 (file)
--- a/src/network_inspectors/reputation/reputation_inspect.cc
+++ b/src/network_inspectors/reputation/reputation_inspect.cc
@@ -329,6 +329,7 @@ private:
 
 Reputation::Reputation(ReputationConfig* pc)
 {
+    reputation_id = create_reputation_id();
     config = *pc;
     ReputationConfig* conf = &config;
     if (!config.list_dir.empty())
@@ -345,7 +346,6 @@ Reputation::Reputation(ReputationConfig* pc)
 
     ip_list_init(conf->num_entries + 1, conf);
     reputationstats.memory_allocated = sfrt_flat_usage(conf->ip_list);
-    reputation_id = create_reputation_id();
 }
 
 bool Reputation::is_reputation_disabled(Flow* flow)