[Seq-cvs-commit] SF.net SVN: seq:[1222] showeq/branches/cn187_devel/src/packetstream.cpp

[<cn...@us...>]

Revision: 1222
          http://sourceforge.net/p/seq/svn/1222
Author:   cn187
Date:     2021-12-07 13:41:51 +0000 (Tue, 07 Dec 2021)
Log Message:
-----------
Only check disconnected sessionId when session tracking is enabled

Modified Paths:
--------------
    showeq/branches/cn187_devel/src/packetstream.cpp

Modified: showeq/branches/cn187_devel/src/packetstream.cpp
===================================================================
--- showeq/branches/cn187_devel/src/packetstream.cpp	2021-11-30 22:38:49 UTC (rev 1221)
+++ showeq/branches/cn187_devel/src/packetstream.cpp	2021-12-07 13:41:51 UTC (rev 1222)
@@ -1144,17 +1144,16 @@
     break;
     case OP_SessionDisconnect:
     {
-
-      // When session tracking isn't enabled, random UDP traffic on the LAN can
-      // cause unexpected session disconnects if the packet payload happens
-      // to start with the same bytes as the OP_SessionDisconnect netOp.
-      //
-      // So check the sessionId in the disconnect packet, and only process
-      // it if it matches the current sessionId.
-      SessionDisconnectStruct* disconnect = (SessionDisconnectStruct*) packet.payload();
-      uint32_t  disconnectedSessionId = eqntohuint32((uint8_t*)&(disconnect->sessionId));
-      if (m_sessionId != disconnectedSessionId) {
-          break;
+      if (m_session_tracking_enabled)
+      {
+        // Check the sessionId in the disconnect packet, and only process
+        // it if it matches the current sessionId.
+        SessionDisconnectStruct* disconnect = (SessionDisconnectStruct*) packet.payload();
+        uint32_t  disconnectedSessionId = eqntohuint32((uint8_t*)&(disconnect->sessionId));
+        if (m_sessionId != disconnectedSessionId)
+        {
+            break;
+        }
       }
 
 #if defined(PACKET_PROCESS_DIAG) || defined(PACKET_SESSION_DIAG)

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.