From: Tom Peters (thopeter) <thopeter@cisco.com>
Date: Tue, 6 Jun 2017 16:08:32 +0000 (-0400)
Subject: Merge pull request #911 in SNORT/snort3 from appid_stats to master
X-Git-Tag: 3.0.0-239~56
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=012ab77c779b4524f6cca1aec0055d945d06eff4;p=thirdparty%2Fsnort3.git

Merge pull request #911 in SNORT/snort3 from appid_stats to master

Squashed commit of the following:

commit 9e6310eba1bf2f4388b0d882e6de33e1ae2c5b67
Author: Steven Baigal <sbaigal@cisco.com>
Date:   Wed May 24 17:05:33 2017 -0400

    removed empty stats printing that was executed from main thread
---

diff --git a/src/network_inspectors/appid/appid_inspector.cc b/src/network_inspectors/appid/appid_inspector.cc
index a9c9fb54d..0a7d1ccd7 100644
--- a/src/network_inspectors/appid/appid_inspector.cc
+++ b/src/network_inspectors/appid/appid_inspector.cc
@@ -52,17 +52,6 @@
 
 static THREAD_LOCAL AppIdStatistics* appid_stats_manager = nullptr;
 
-static void dump_appid_stats()
-{
-    LogMessage("Application Identification Preprocessor:\n");
-    LogMessage("   Total packets received : %" PRIu64 "\n", appid_stats.packets);
-    LogMessage("  Total packets processed : %" PRIu64 "\n", appid_stats.processed_packets);
-    if (thirdparty_appid_module)
-        thirdparty_appid_module->print_stats();
-    LogMessage("    Total packets ignored : %" PRIu64 "\n", appid_stats.ignored_packets);
-    AppIdServiceState::dump_stats();
-}
-
 // FIXIT-L - appid cleans up openssl now as it is the primary (only) user... eventually this
 //           should probably be done outside of appid
 static void openssl_cleanup()
@@ -78,9 +67,6 @@ AppIdInspector::AppIdInspector(const AppIdModuleConfig* pc)
 
 AppIdInspector::~AppIdInspector()
 {
-    if (config->debug)
-        dump_appid_stats();
-
     delete active_config;
     delete config;
 }