Merge pull request #2713 in SNORT/snort3 from ~SATHIRKA/snort3:reload_response to...

Squashed commit of the following:

commit ab0f7d9e35572f611a339eb4ff7ddeeeb8b3c547
Author: Sreeja Athirkandathil Narayanan <sathirka@cisco.com>
Date:   Fri Jan 8 11:19:29 2021 -0500

    appid: Send reload detectors and third-party messages to socket immediately if appid is not enabled

diff --git a/src/network_inspectors/appid/appid_module.cc b/src/network_inspectors/appid/appid_module.cc
index 4ce78cd0e1f21b80b850c7aec0b0f62310bee565..37e4489655fe1be0bb68f4cf0e9ea3c5bc780231 100644 (file)
--- a/src/network_inspectors/appid/appid_module.cc
+++ b/src/network_inspectors/appid/appid_module.cc
@@ -312,28 +312,28 @@ static int disable_debug(lua_State*)
 
 static int reload_third_party(lua_State* L)
 {
-    bool from_shell = ( L != nullptr );
     SharedRequest current_request = get_current_request();
     if (Swapper::get_reload_in_progress())
     {
-        current_request->respond("== reload pending; retry\n", from_shell);
+        current_request->respond("== reload pending; retry\n");
         return 0;
     }
-    current_request->respond(".. reloading third-party\n", from_shell);
     AppIdInspector* inspector = (AppIdInspector*) InspectorManager::get_inspector(MOD_NAME);
     if (!inspector)
     {
-        current_request->respond("== reload third-party failed - appid not enabled\n", from_shell);
+        current_request->respond("== reload third-party failed - appid not enabled\n");
         return 0;
     }
     const AppIdContext& ctxt = inspector->get_ctxt();
     ThirdPartyAppIdContext* old_ctxt = ctxt.get_tp_appid_ctxt();
     if (!old_ctxt)
     {
-        current_request->respond("== reload third-party failed - third-party module doesn't exist\n", from_shell);
+        current_request->respond("== reload third-party failed - third-party module doesn't exist\n");
         return 0;
     }
     Swapper::set_reload_in_progress(true);
+
+    bool from_shell = ( L != nullptr );
     current_request->respond("== unloading old third-party configuration\n", from_shell);
     main_broadcast_command(new ACThirdPartyAppIdContextUnload(*inspector, old_ctxt,
         current_request, from_shell), from_shell);
@@ -351,21 +351,21 @@ static void clear_dynamic_host_cache_services()
 
 static int reload_detectors(lua_State* L)
 {
-    bool from_shell = ( L != nullptr );
     SharedRequest current_request = get_current_request();
     if (Swapper::get_reload_in_progress())
     {
-        current_request->respond("== reload pending; retry\n", from_shell);
+        current_request->respond("== reload pending; retry\n");
         return 0;
     }
-    current_request->respond(".. reloading detectors\n", from_shell);
     AppIdInspector* inspector = (AppIdInspector*) InspectorManager::get_inspector(MOD_NAME);
     if (!inspector)
     {
-        current_request->respond("== reload detectors failed - appid not enabled\n", from_shell);
+        current_request->respond("== reload detectors failed - appid not enabled\n");
         return 0;
     }
     Swapper::set_reload_in_progress(true);
+    bool from_shell = ( L != nullptr );
+    current_request->respond(".. reloading detectors\n", from_shell);
 
     AppIdContext& ctxt = inspector->get_ctxt();
     OdpContext& old_odp_ctxt = ctxt.get_odp_ctxt();