From: Michael Altizer (mialtize) <mialtize@cisco.com>
Date: Fri, 29 Jun 2018 14:41:00 +0000 (-0400)
Subject: Merge pull request #1297 in SNORT/snort3 from reload_crash to master
X-Git-Tag: 3.0.0-246~35
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5fa5d4fa97b0749c2b460a262cd0269f174655ea;p=thirdparty%2Fsnort3.git

Merge pull request #1297 in SNORT/snort3 from reload_crash to master

Squashed commit of the following:

commit 9b2fad5be6056526525edab816181c3bbea7582c
Author: Carter Waxman <cwaxman@cisco.com>
Date:   Fri Jun 29 09:21:27 2018 -0400

    FrameworkPolicy: fixed adding probe to wrong SnortConfig
---

diff --git a/src/managers/inspector_manager.cc b/src/managers/inspector_manager.cc
index fb13e026c..ffcadd96d 100644
--- a/src/managers/inspector_manager.cc
+++ b/src/managers/inspector_manager.cc
@@ -228,10 +228,10 @@ struct FrameworkPolicy
 
     bool default_binder;
 
-    void vectorize();
+    void vectorize(SnortConfig*);
 };
 
-void FrameworkPolicy::vectorize()
+void FrameworkPolicy::vectorize(SnortConfig* sc)
 {
     passive.alloc(ilist.size());
     packet.alloc(ilist.size());
@@ -281,7 +281,6 @@ void FrameworkPolicy::vectorize()
         {
             // probes always run
             // add them to default so they can be found on InspectorManager::probe
-            SnortConfig* sc = SnortConfig::get_conf();
             sc->policy_map->get_inspection_policy(0)->framework_policy->probe.add(p);
             break;
         }
@@ -481,6 +480,7 @@ void InspectorManager::delete_policy(InspectionPolicy* pi, bool cloned)
             s_trash2.push_back(p->handler);
         else
             s_trash.push_back(p->handler);
+
         delete p;
     }
     delete pi->framework_policy;
@@ -815,7 +815,7 @@ static bool configure(SnortConfig* sc, FrameworkPolicy* fp, bool cloned)
     }
 
     sort(fp->ilist.begin(), fp->ilist.end(), PHInstance::comp);
-    fp->vectorize();
+    fp->vectorize(sc);
 
     // FIXIT-M checking for wizard here would avoid fatals for
     // can't bind wizard but this exposes other issues that must
@@ -866,6 +866,7 @@ bool InspectorManager::configure(SnortConfig* sc, bool cloned)
     }
 
     set_inspection_policy(sc);
+
     return ok;
 }