From: Florin Coras <fcoras@cisco.com>
Date: Tue, 12 Dec 2023 22:01:43 +0000 (-0800)
Subject: session: avoid spurious closed notifications
X-Git-Tag: v24.06-rc0~53
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=5afc13d594bf874092e135a8dc94d30c4537526b;p=vpp.git

session: avoid spurious closed notifications

Type: fix

Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: Ia98556e7ae61547cf153c78ec085cd4248bee74a
---

diff --git a/src/vnet/session/session.c b/src/vnet/session/session.c
index 9ac9229d4db..2afc37a77f8 100644
--- a/src/vnet/session/session.c
+++ b/src/vnet/session/session.c
@@ -1184,6 +1184,9 @@ session_transport_closed_notify (transport_connection_t * tc)
   if (!(s = session_get_if_valid (tc->s_index, tc->thread_index)))
     return;
 
+  if (s->session_state >= SESSION_STATE_TRANSPORT_CLOSED)
+    return;
+
   /* Transport thinks that app requested close but it actually didn't.
    * Can happen for tcp:
    * 1)if fin and rst are received in close succession.