aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2018-10-17 10:41:28 -0700
committerFlorin Coras <florin.coras@gmail.com>2018-10-18 01:42:24 +0000
commitdc532e4ff75af463794fe64764da890427a278d5 (patch)
treeb18ab08c622e635782e02acb86e48aacbd07ebc7 /src
parentc10c73ffb3fa45f4f054e9517f1f80bbec489b47 (diff)
tcp: fix cleanup of non established connections (VPP-1463)
- fix delete of connection in syn-received - fix delete of half-open connection Change-Id: I72ff4b60406a2762d998328c52f41adea40d2c1b Signed-off-by: Florin Coras <fcoras@cisco.com>
Diffstat (limited to 'src')
-rw-r--r--src/vnet/session/session.c1
-rw-r--r--src/vnet/tcp/tcp.c5
2 files changed, 5 insertions, 1 deletions
diff --git a/src/vnet/session/session.c b/src/vnet/session/session.c
index 7b50950fe01..23d258ffdea 100644
--- a/src/vnet/session/session.c
+++ b/src/vnet/session/session.c
@@ -820,6 +820,7 @@ stream_session_delete_notify (transport_connection_t * tc)
session_lookup_del_session (s);
break;
case SESSION_STATE_CLOSED:
+ case SESSION_STATE_ACCEPTING:
stream_session_delete (s);
break;
}
diff --git a/src/vnet/tcp/tcp.c b/src/vnet/tcp/tcp.c
index 884602deb6f..5378de1c1da 100644
--- a/src/vnet/tcp/tcp.c
+++ b/src/vnet/tcp/tcp.c
@@ -210,7 +210,8 @@ tcp_connection_cleanup (tcp_connection_t * tc)
/* Try to remove the half-open connection. If this is not the owning
* thread, tc won't be removed. Retransmit or establish timers will
* eventually expire and call again cleanup on the right thread. */
- tcp_half_open_connection_cleanup (tc);
+ if (tcp_half_open_connection_cleanup (tc))
+ tc->flags |= TCP_CONN_HALF_OPEN_DONE;
}
else
{
@@ -322,8 +323,10 @@ tcp_connection_close (tcp_connection_t * tc)
tc->state = TCP_STATE_CLOSED;
break;
case TCP_STATE_SYN_RCVD:
+ tcp_connection_timers_reset (tc);
tcp_send_fin (tc);
tc->state = TCP_STATE_FIN_WAIT_1;
+ tcp_timer_update (tc, TCP_TIMER_WAITCLOSE, TCP_CLEANUP_TIME);
break;
case TCP_STATE_ESTABLISHED:
if (!session_tx_fifo_max_dequeue (&tc->connection))