summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2023-03-13 16:31:52 -0700
committerDave Wallace <dwallacelf@gmail.com>2023-03-14 16:09:50 +0000
commit02aa2ca0934dbbed7440bd8225f39dde95610206 (patch)
tree120ead93929dceae6387c619894ea55569d5dc48 /src
parent1552228350bb73db281aee43d90bcb068189d5ba (diff)
session: cleanup lcl endpt freelist before all alloc
Make sure endpoint freelist is drained before alloc of fixed local source port is tried. Type: fix Signed-off-by: Florin Coras <fcoras@cisco.com> Change-Id: I302deee5609a463af8135185af71722ac8c55a27
Diffstat (limited to 'src')
-rw-r--r--src/vnet/session/transport.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/src/vnet/session/transport.c b/src/vnet/session/transport.c
index 0020d743d9b..8b3dacfe309 100644
--- a/src/vnet/session/transport.c
+++ b/src/vnet/session/transport.c
@@ -578,10 +578,6 @@ transport_alloc_local_port (u8 proto, ip46_address_t *lcl_addr,
/* Only support active opens from one of ctrl threads */
ASSERT (vlib_get_thread_index () <= transport_cl_thread ());
- /* Cleanup freelist if need be */
- if (vec_len (tm->lcl_endpts_freelist))
- transport_cleanup_freelist ();
-
/* Search for first free slot */
for (tries = 0; tries < limit; tries++)
{
@@ -669,6 +665,7 @@ transport_alloc_local_endpoint (u8 proto, transport_endpoint_cfg_t * rmt_cfg,
ip46_address_t * lcl_addr, u16 * lcl_port)
{
transport_endpoint_t *rmt = (transport_endpoint_t *) rmt_cfg;
+ transport_main_t *tm = &tp_main;
session_error_t error;
int port;
@@ -689,6 +686,10 @@ transport_alloc_local_endpoint (u8 proto, transport_endpoint_cfg_t * rmt_cfg,
sizeof (rmt_cfg->peer.ip));
}
+ /* Cleanup freelist if need be */
+ if (vec_len (tm->lcl_endpts_freelist))
+ transport_cleanup_freelist ();
+
/*
* Allocate source port
*/