]> git.ipfire.org Git - thirdparty/snort3.git/commitdiff
Merge pull request #1237 in SNORT/snort3 from 245 to master 3.0.0-245
authorRuss Combs (rucombs) <rucombs@cisco.com>
Wed, 23 May 2018 16:35:18 +0000 (12:35 -0400)
committerRuss Combs (rucombs) <rucombs@cisco.com>
Wed, 23 May 2018 16:35:18 +0000 (12:35 -0400)
Squashed commit of the following:

commit 451b9ec52a96fefed5948c4b2d4f79173cd70496
Author: Russ Combs (rucombs) <rucombs@cisco.com>
Date:   Wed May 23 09:06:15 2018 -0400

    file_id: claim gid 146 to avoid legacy issues

commit 1fb59288145a0e883da710f5309ca5e475ca022f
Author: Russ Combs (rucombs) <rucombs@cisco.com>
Date:   Wed May 23 08:59:40 2018 -0400

    events: rename some gid and sid defines for consistency and clarity

commit 1c2156f4cbc7dc50556610b0c01a27b1efe3e662
Author: Russ Combs (rucombs) <rucombs@cisco.com>
Date:   Wed May 23 02:16:33 2018 -0400

    parser: gid value does not indicate rule type

commit 94f77577256b13b1d5edbf0fc21c7613d501f3b3
Author: Russ Combs (rucombs) <rucombs@cisco.com>
Date:   Wed May 23 02:07:07 2018 -0400

    parser: warn about text rules w/o detection options

commit 92969aba5c224525b877c69397b02956fbc7c708
Author: Russ Combs <rucombs@cisco.com>
Date:   Mon May 21 10:25:03 2018 -0400

    build 245

18 files changed:
ChangeLog
src/detection/detect.cc
src/detection/fp_create.cc
src/detection/rules.h
src/detection/service_map.cc
src/detection/signature.h
src/detection/tag.cc
src/file_api/file_module.h
src/filters/rate_filter.cc
src/filters/sfrf.cc
src/main/build.h
src/parser/parse_rule.cc
src/piglet_plugins/pp_event_iface.cc
src/stream/base/stream_module.cc
src/stream/libtcp/tcp_segment_descriptor.cc
src/stream/tcp/tcp_event_logger.cc
src/stream/tcp/tcp_session.cc
src/stream/user/user_session.cc

index 76f0d4e63f22956be31d23212de66fa71dd90ec1..c0ada4e07f89ce749cbaa29612e86d820bfba798 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,156 @@
+18/05/21 - build 245
+
+-- CodecManager: removed unused code
+-- DataBus: fixed creating DataHandler when one doesn't exist
+-- Debug messages: cleanup for service inspectors.  New traces for detection, stream.
+-- Debug: Final debug messages cleanup, removal of macros from snort_debug
+-- Ipv4Codec: removed random ip id pool and replaced randoms on demand
+-- PacketManager: moved encode storage to heap
+-- PerfMonitor: fixed subscribing to flow events multiple times
+-- ProtoRef: Converge on single name for SnortProtocolId. Fix threading problems.
+-- Reset: Always queue reject and test packet type in RejectAction::exec.
+-- SFDAQModule: moved daq stats here. fixed stats not being output from perfmon.
+-- Snort2lua: Add ftp_data to multiple files when needed, once per file.
+-- Snort2lua: Translate ftp_server relative to default configurations.
+-- Snort: moved s_data to heap
+-- active: Enable when max_responses is enabled
+-- alert: moved alert json. unixsock out from extra to snort3
+-- appid: Add AppID debug command
+-- appid: Enable Third-Party Code for Packet Processing
+-- appid: Fix bug where Service and Application ID's set to port number instead of service appid
+-- appid: Fixing service discovery states
+-- appid: Only import dynamic detector pegcounts once
+-- appid: Refactor debug command
+-- appid: Refactor debug command, use SfIp, and fix non-Linux compilation
+-- appid: Third party integration support
+-- appid: appid session unit test changes
+-- appid: change metadata buffers from std::string to pointers, to avoid extra copying
+-- appid: clean-up code for performance and implement is_tp_processing_done()
+-- appid: create referer object only for non-null string
+-- appid: do not inspect out-of-order flows, ignore zero-payload packets for client/service discovery
+-- appid: fix memory leak in appid_http_event_test and warning in appid_http_session.cc
+-- appid: fix segfault due to dereferencing null host pointer.
+-- appid: fix tabs and indentation
+-- appid: fixed http fields, referer payload and appid debug
+-- appid: make tp_attribute_data more localized, so we only allocate/deallocate it if needed.
+-- appid: moved HttpFieldIds to appid_http_session
+-- appid: peg count / dynamic peg count update.  Split peg counts into the ones known at compile time and dynamic ones.  Update stats , module manager and module to support dumping dynamic stats.
+-- appid: report when third party appid is done inspecting
+-- appid: sip: moved pattern thread local to class instance
+-- base64_decode: moved buffer storage to regular heap
+-- binder: Fix UBSAN invalid value type runtime error
+-- build: 244
+-- build: Add --enable-ub-sanitizer option for undefined behavior sanitizer
+-- build: Add some header includes for FreeBSD
+-- build: Clean up CMake string APPENDing for configure options
+-- build: Clean up HAVE_* definition checks
+-- build: Define NDEBUG if debugging is not enabled
+-- build: Fix building unit tests on FreeBSD
+-- build: Modernize code with =default for special member functions
+-- build: Modernize code with virtual/override/final cleanups
+-- build: Remove bashisms from most shell scripts
+-- build: add cmake configure switches for NO_PROFILER, NO_MEM_MGR and DEEP_PROFILING
+-- build: add disable-docs to disable doc build
+-- build: fix various drops const qualifier cases
+-- build: fix various warnings:
+-- build: propogate snort3 tsc build option to the extra build system
+-- byte_extract: fix cursor update
+-- byte_jump: fix from_beginning
+-- byte_math: allow rvalue == 0 except for division
+-- catch: Update to Catch v2.2.1
+-- clock: Allow use of ARM64 CNTVCT_EL0 register for timing (#46); thanks to j.mcdowell@titan-ic.com for the patch.
+-- clock: use uint64_t with tsc clock instead of std::chrono for performance
+-- cmake: Add --enable-appid-third-party to configure_cmake.sh
+-- cmake: Add support for building with tcmalloc
+-- cmake: Rework FindPCAP logic and ignore SFBPF
+-- cmake: fixed checks for functions
+-- cmake: update for iconv
+-- codecs: add config option to detection to enable check and alert for address anomalies
+-- daq_hext: Make IpAddr() static to fix compiler warning
+-- dce_co_process_ctx_id needs to update its caller's (DCE2_CoCtxReq) frag_ptr as it is called in a loop in order to parse each dce/rpc ctx item, otherwise it ends up parsing the same ctx item over and over.
+-- dce_rpc: fix parsing of dce/rpc ctx items
+-- dce_rpc: pass frag_ptr by reference
+-- debug: Remove debug messages from appid, arp_spoof, and perf_monitor
+-- debug: Remove debug messages from detection and ips_options
+-- debug: Remove debug messages from stream
+-- decompress/file_decomp_pdf.cc: implicit fallthrough
+-- detect: moving thread locals identified to ips context
+-- detection: fixed uninitialized MpseStash
+-- doc: add doc for module trace
+-- encoders: fixed off-by-one error in underlying buffer handling
+-- extra: Port some CMake options from Snort prime
+-- extra: splitted extra out to snort3_extra repo
+-- file_api: combine file cache for file resume and partial file processing
+-- file_connector: Fix address-of-packed-member compiler warnings
+-- file_decomp_pdf.cc: unreachable code return
+-- file_type: Require strings instead of integers for types. Handle versions.
+-- flow: SO_PUBLIC FlowKey
+-- framework: align PktType and proto bits
+-- framework: remove bogus PktType for ARP and just use proto bits instead
+-- ftp_server: Added Flow::set_service and fixed FtpDataFlowData::handled_expected.
+-- ftp_server: Added ability get TCP options length from TcpStreamSession.
+-- ftp_server: Added accessors to Stream so TcpStreamSession can be private.
+-- ftp_server: Base last_seg_size off of MSS.
+-- ftp_server: Provide FLOW_SERVICE_CHANGE pub/sub event.
+-- ftp_server: ftp_server requires that ftp_client and ftp_data be configured.
+-- hashfcn: Fix UBSAN integer overflow runtime error
+-- hashfcn: Fix UBSAN left shift of negative value runtime error
+-- http_inspect: broken chunk performance improvement
+-- http_inspect: bugfix and new alert for gzip underrun
+-- http_inspect: embedded white space in Content-Length
+-- http_inspect: handling of run-to-connection-close bodies beyond depth
+-- http_inspect: know more Content-Encodings by name
+-- http_inspect: patch around regression failures until a permanent solution is implemented
+-- http_inspect: performance enhancements for file processing beyond detection depth
+-- ip: replaced REG_TEST with -H option for ipv4 codec fixed seed
+-- ips_byte_jump: Fix UBSAN left shift of negative value runtime error
+-- ips_byte_math: Fix UBSAN left shift of negative value runtime error
+-- ips_flags: remove dead code
+-- javascript: moved decode buffer to stack
+-- memory: disable with -DNO_MEM_MGR
+-- memory_manager.cc: dangling references
+-- packet_capture, cmake: Remove SFBPF dependencies
+-- packet_capture: adding analyzer command to initialize dump file
+-- packet_tracer: Fix compiler warning when compiling with NDEBUG
+-- packet_tracer: Modularize and add constraint-based shell enablement
+-- parameter: Fix UBSAN shift exponent is too large for 32-bit type runtime error
+-- parser: allow arbitrary rule gids
+-- pop, imap, and smtp: changes to MIME configuration parameters
+-- port_scan: include open ports with alerts instead of separate
+-- profile: disable with -DNO_PROFILER
+-- profiler: add deep profiler option
+-- reload: enabled reloading ips_actions; added parse error check for reloading
+-- repuation: remove the limit for zone id
+-- reputation: add zone support
+-- search_engine: revert default detect_raw_tcp to false
+-- service inspectors: debug cleanup
+-- sfip: A version of set() which automatically determines the family
+-- sfip: removed ntoa. use ntop(SfIpString) instead.
+-- snort2lua: Add reject action when active responses is enabled
+-- snort2lua: conversion of gid 120 to 119
+-- snort2lua: enable reject action when firewall is enabled
+-- snort: -r- will read packets from stdin
+-- spell check: fix memeory and indicies typos
+-- steam_tcp: change singleton names from linux to new_linux to avoid spurious collisions with defines
+-- stream ip: refactored to use MemoryManager allocators
+-- stream: assume gid 135 so those rules are handled as standard builtins
+-- stream: be selective about flow creation for scans
+-- stream: refactor flow control for new PktTypes
+-- stream: remove usused ignore_any_rules from tcp and udp
+-- stream: respect tcp require_3whs
+-- stream: warning: potential memory leaks
+-- stream_tcp: refactor tcp normalizer and reassembler to eliminate dynamic heap allocations per flow
+-- stream_tcp: switch to splitter max
+-- stream_tcp: tweak seglist cursor handling
+-- target_based: 100% coverage on snort_protocols.cc
+-- target_based: unit tests for ProtocolReference class
+-- tcp codec: count bad ip6 checksums correctly; thanks to j.mcdowell@titan-ic.com for reporting the issue
+-- tcp: allow data  handlding for packet with invalid ack
+-- time: initialize Stopwatch::start_time member variable to 0 ticks when TSC clock is enabled
+-- trace: add traces for deleted debug messages
+-- wizard: Fix UBSAN out-of-bounds access runtime error
+-- zhash: cleanup cruftiness
+
 18/03/15 - build 244
 
 -- appid: unit-tests for http detector plugins
index 10be0ef272ecdea4dde00b72fdf3215b096a1b69..2f3e4900bc3695a313dcb1f80409de506880aad7 100644 (file)
@@ -105,7 +105,7 @@ void CallAlertFuncs(Packet* p, const OptTreeNode* otn, ListHead* head)
 
 #if 0
     // FIXIT-M this should be a generic feature of otn
-    if ( otn->sigInfo.gid != GENERATOR_SPP_REPUTATION )
+    if ( otn->sigInfo.gid != GID_REPUTATION )
     {
         /* Don't include IP Reputation events in count */
         pc.alert_pkts++;
index 796d2d858c0cff7f721af3508ba22e03a313b770..1ad01c0152b4585634ed27ea415f5a2e45544c4c 100644 (file)
@@ -506,7 +506,7 @@ static int fpAddPortGroupRule(
     PatternMatchVector pmv;
 
     // skip builtin rules, continue for text and so rules
-    if ( !otn->sigInfo.text_rule )
+    if ( otn->sigInfo.builtin )
         return -1;
 
     /* Rule not enabled */
index 21927580bcad9107109fc3f1bc9ceeafaf9235b4..07cc6bf990d5893671fd8ee955463ed4752ca3cb 100644 (file)
 #define ANY_SRC_IP      0x0100
 #define ANY_DST_IP      0x0200
 
-#define GENERATOR_SNORT_ENGINE        1
-#define GENERATOR_SNORT_SHARED        3
+#define GID_DEFAULT          1
+#define GID_SESSION        135
 
-// FIXIT-L gid 135 defines should be renamed and moved to stream
-#define GENERATOR_INTERNAL          135
-#define INTERNAL_EVENT_SYN_RECEIVED   1
-#define INTERNAL_EVENT_SESSION_ADD    2
-#define INTERNAL_EVENT_SESSION_DEL    3
+#define SESSION_EVENT_SYN_RX 1
+#define SESSION_EVENT_SETUP  2
+#define SESSION_EVENT_CLEAR  3
 
-#define EventIsInternal(gid) ((gid) == GENERATOR_INTERNAL)
+#define EventIsInternal(gid) ((gid) == GID_SESSION)
 
 namespace snort
 {
index b447ffeca58342585369c6cbae827ec64b01dca9..fdd9e4c04cc62fe573f097a2dcbcc4a40d12bdaf 100644 (file)
@@ -254,7 +254,7 @@ int fpCreateServiceMaps(SnortConfig* sc)
             if ( rtn )
             {
                 // skip builtin rules
-                if ( !otn->sigInfo.text_rule )
+                if ( otn->sigInfo.builtin )
                     continue;
 
                 /* Not enabled, don't do the FP content */
index d6d307080352da3f81f5cad2306e2c8a28a313be..afe453f2d757387a80bda53a8d86a0f40b0e3d54 100644 (file)
@@ -103,7 +103,7 @@ struct SigInfo
     uint32_t priority;
     uint32_t num_services;
 
-    bool text_rule;
+    bool builtin;
     Target target;
 };
 
index 96b2407474149ec3a9d20a7c5853140a36abec02..f08f34d0dee50e510fa3ce52b9b888521eb45a88 100644 (file)
@@ -51,7 +51,7 @@ using namespace snort;
 #define TAG_PRUNE_QUANTUM   300
 #define TAG_MEMCAP          4194304  /* 4MB */
 
-#define GENERATOR_TAG       2
+#define GID_TAG             2
 #define TAG_LOG_PKT         1
 
 /*  D A T A   S T R U C T U R E S  **********************************/
@@ -549,7 +549,7 @@ int CheckTagList(Packet* p, Event& event, void** log_list)
         if ( create_event )
         {
             /* set the event info */
-            SetEvent(event, GENERATOR_TAG, TAG_LOG_PKT, 1, 1, 1, returned->event_id);
+            SetEvent(event, GID_TAG, TAG_LOG_PKT, 1, 1, 1, returned->event_id);
 
             /* set event reference details */
             event.ref_time.tv_sec = returned->event_time.tv_sec;
index ae860930df3bcd7586495f3701c86d3350fcd699..fca12fe42391279f6d4934d24ec20f891fc38140 100644 (file)
@@ -54,6 +54,11 @@ public:
 
     void show_dynamic_stats() override;
 
+    // FIXIT-L delete file_id gid when bogus rules are eliminated
+    // (this ensures those rules don't fire on every packet)
+    unsigned get_gid() const override
+    { return 146; }
+
 private:
     FileMagicRule rule;
     FileMagicData magic;
index 92d9b0830f72136b62d03bebf89bc7361aba1457..10ba4311b4dcd45b1af6676b67ab5ea747e8eefe 100644 (file)
@@ -91,8 +91,8 @@ int RateFilter_Create(
     {
         enable_internal_event(rf_config, thdx->sid);
 
-        if ( thdx->sid == INTERNAL_EVENT_SESSION_ADD )
-            enable_internal_event(rf_config, INTERNAL_EVENT_SESSION_DEL);
+        if ( thdx->sid == SESSION_EVENT_SETUP )
+            enable_internal_event(rf_config, SESSION_EVENT_CLEAR);
     }
     return error;
 }
index 5654cae34133d55e8830591ca9dd361c0c20bb43..50bce1e1544811ef06fe327654e1e2e73b757fb2 100644 (file)
@@ -755,8 +755,8 @@ static void _updateDependentThresholds(
     time_t curTime
     )
 {
-    if ( gid == GENERATOR_INTERNAL &&
-        sid == INTERNAL_EVENT_SESSION_DEL )
+    if ( gid == GID_SESSION &&
+        sid == SESSION_EVENT_CLEAR )
     {
         // decrementing counters - this results in the following sequence:
         // 1. sfdos_thd_test_threshold(gid internal, sid DEL)
@@ -765,7 +765,7 @@ static void _updateDependentThresholds(
         // 4.    |       _updateDependentThresholds(gid internal, sid ADD)
         // 5.    continue with regularly scheduled programming (ie step 1)
 
-        SFRF_TestThreshold(config, gid, INTERNAL_EVENT_SESSION_ADD,
+        SFRF_TestThreshold(config, gid, SESSION_EVENT_SETUP,
             sip, dip, curTime, SFRF_COUNT_DECREMENT);
         return;
     }
index b0c404d7a9000343456dbd2218515af75b5a10fc..98f94e429d12d0d928a8a09e267e42ce55af6329 100644 (file)
@@ -12,7 +12,7 @@
 //                                               //
 //-----------------------------------------------//
 
-#define BUILD_NUMBER 244
+#define BUILD_NUMBER 245
 
 #ifndef EXTRABUILD
 #define BUILD STRINGIFY_MX(BUILD_NUMBER)
index 841372278fb3baf2fe1eca576b4f40d66c84bfb5..7fe0487ff56af846488a5caf0e919485197dc24b 100644 (file)
@@ -1137,7 +1137,7 @@ OptTreeNode* parse_rule_open(SnortConfig* sc, RuleTreeNode& rtn, bool stub)
     otn->state = (OtnState*)snort_calloc(ThreadConfig::get_instance_max(), sizeof(OtnState));
 
     if ( !stub )
-        otn->sigInfo.gid = GENERATOR_SNORT_ENGINE;
+        otn->sigInfo.gid = GID_DEFAULT;
 
     otn->chain_node_number = otn_count;
     otn->snort_protocol_id = rtn.snort_protocol_id;
@@ -1173,8 +1173,6 @@ const char* parse_rule_close(SnortConfig* sc, RuleTreeNode& rtn, OptTreeNode* ot
             ParseError("SO rule %s not loaded.", otn->soid);
         else
         {
-            // FIXIT-L gid may be overwritten when set to 3 upon close
-            otn->sigInfo.gid = GENERATOR_SNORT_SHARED;
             entered = true;
             return so_opts;
         }
@@ -1204,13 +1202,9 @@ const char* parse_rule_close(SnortConfig* sc, RuleTreeNode& rtn, OptTreeNode* ot
     otn_count++;
     rule_count++;
 
-    // FIXIT-L need more reliable way of knowing type of rule instead of
-    // hard coding these gids do GIDs actually matter anymore (w/o conflict
-    // with builtins)?
-
-    if ( otn->sigInfo.gid == GENERATOR_SNORT_SHARED )
+    if ( otn->soid )
     {
-        otn->sigInfo.text_rule = true;
+        otn->sigInfo.builtin = false;
         so_rule_count++;
     }
     else if ( ModuleManager::gid_in_use(otn->sigInfo.gid) )
@@ -1219,12 +1213,16 @@ const char* parse_rule_close(SnortConfig* sc, RuleTreeNode& rtn, OptTreeNode* ot
             ParseError("%u:%u builtin rules do not support detection options",
                 otn->sigInfo.gid, otn->sigInfo.sid);
 
-        otn->sigInfo.text_rule = false;
+        otn->sigInfo.builtin = true;
         builtin_rule_count++;
     }
     else
     {
-        otn->sigInfo.text_rule = true;
+        if ( !otn->num_detection_opts )
+            ParseWarning(WARN_RULES, "%u:%u does not have any detection options",
+                otn->sigInfo.gid, otn->sigInfo.sid);
+
+        otn->sigInfo.builtin = false;
         detect_rule_count++;
     }
 
index eebbc94ae52c691088646f934c544cc5474a91e7..5b0b564329a1a6e48401644985b9d461da4bafb6 100644 (file)
@@ -106,7 +106,7 @@ static const luaL_Reg methods[] =
                 si_table.set_field("rev", si->rev);
                 si_table.set_field("class_id", si->class_id);
                 si_table.set_field("priority", si->priority);
-                si_table.set_field("text_rule", si->text_rule);
+                si_table.set_field("builtin", si->builtin);
                 si_table.set_field("num_services", si->num_services);
 
                 Lua::Table(L, 2).set_field_from_stack("sig_info", si_table.index);
@@ -138,7 +138,7 @@ static const luaL_Reg methods[] =
                 si_table.get_field("rev", si->rev);
                 si_table.get_field("class_id", si->class_id);
                 si_table.get_field("priority", si->priority);
-                si_table.get_field("text_rule", si->text_rule);
+                si_table.get_field("builtin", si->builtin);
                 si_table.get_field("num_services", si->num_services);
             }
 
index acdbe3c3fa02ba149bbd7b9779b29b2c346b05e0..1d9ef6390933619b774b99ea654855a2c2ca1ce7 100644 (file)
@@ -79,11 +79,12 @@ static const Parameter s_params[] =
     { nullptr, Parameter::PT_MAX, nullptr, nullptr, nullptr }
 };
 
+// FIXIT-L setup and clear should extend to non-tcp flows as well
 static const RuleMap stream_rules[] =
 {
-    { INTERNAL_EVENT_SYN_RECEIVED, "TCP SYN received" },
-    { INTERNAL_EVENT_SESSION_ADD, "TCP session established" },
-    { INTERNAL_EVENT_SESSION_DEL, "TCP session cleared" },
+    { SESSION_EVENT_SYN_RX, "TCP SYN received" },
+    { SESSION_EVENT_SETUP,  "TCP session established" },
+    { SESSION_EVENT_CLEAR,  "TCP session cleared" },
 
     { 0, nullptr }
 };
@@ -102,7 +103,7 @@ ProfileStats* StreamModule::get_profile() const
 { return &s5PerfStats; }
 
 unsigned StreamModule::get_gid() const
-{ return GENERATOR_INTERNAL; }
+{ return GID_SESSION; }
 
 const RuleMap* StreamModule::get_rules() const
 { return stream_rules; }
index 88c5fcc2304b91b85a1aa4ff8fffc68f2f78f4f1..045441a5533369a5f9d51b40d695918d4ada71c4 100644 (file)
@@ -48,7 +48,7 @@ TcpSegmentDescriptor::TcpSegmentDescriptor(Flow* flow, Packet* pkt, TcpEventLogg
     {
         end_seq++;
         if ( !tcph->is_ack() )
-            tel.log_internal_event(INTERNAL_EVENT_SYN_RECEIVED);
+            tel.log_internal_event(SESSION_EVENT_SYN_RX);
     }
 }
 
index 87c546eb92d241dc366d8a7a91b2e15c566d8934..464be07ab8ee8a6de5c32189d90f0a3f8ec641f1 100644 (file)
@@ -93,7 +93,7 @@ void TcpEventLogger::log_internal_event(uint32_t eventSid)
     if (is_internal_event_enabled(snort::SnortConfig::get_conf()->rate_filter_config, eventSid))
     {
         tcpStats.internalEvents++;
-        snort::DetectionEngine::queue_event(GENERATOR_INTERNAL, eventSid);
+        snort::DetectionEngine::queue_event(GID_SESSION, eventSid);
     }
 }
 
index 389fe507bacfa54b3c5b755bf3349fa7eb3ceee1..feca899734b67908fac8b2ad26d806791d85110b 100644 (file)
@@ -176,7 +176,7 @@ void TcpSession::clear_session(bool free_flow_data, bool flush_segments, bool re
     set_splitter(true, nullptr);
     set_splitter(false, nullptr);
 
-    tel.log_internal_event(INTERNAL_EVENT_SESSION_DEL);
+    tel.log_internal_event(SESSION_EVENT_CLEAR);
 
     lws_init = false;
     tcp_init = false;
@@ -204,7 +204,7 @@ void TcpSession::update_perf_base_state(char newState)
             session_flags |= SSNFLAG_COUNTED_ESTABLISH;
             fire_event = true;
 
-            tel.log_internal_event(INTERNAL_EVENT_SESSION_ADD);
+            tel.log_internal_event(SESSION_EVENT_SETUP);
             if ( ( session_flags & SSNFLAG_COUNTED_INITIALIZE )
                 && !( session_flags & SSNFLAG_COUNTED_CLOSING ) )
             {
index 9670c224c8f8fbef393e8862d0ba4d14514af964..b18274c1452ea5adce50d2e8a5f39734bdd236e1 100644 (file)
@@ -365,7 +365,7 @@ void UserSession::start(Packet* p, Flow* flow)
 
         StreamUpdatePerfBaseState(&sfBase, tmp->flow, TCP_STATE_SYN_SENT);
 
-        EventInternal(INTERNAL_EVENT_SESSION_ADD);
+        EventInternal(SESSION_EVENT_SETUP);
 #endif
     }
 }