diff options
| author |   Florin Coras <fcoras@cisco.com> | 2020-04-09 20:59:20 +0000 |
|---|---|---|
| committer |   Dave Barach <openvpp@barachs.net> | 2020-04-09 22:47:02 +0000 |
| commit | 12813d5e4a7b858304093ce5f76026f456c79b98 (patch) | |
| tree | 6173df413ea7e1b2f7df3e27181a5324e9720105 /src/vnet/session/application_local.c | |
| parent | 573f44c2f5c883753bc9880eead1bd662586521d (diff) | |
session: fix leak on accept fifo alloc failure
Type: fix
Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: Ia928d6ea05ff7cb9a15e1ddc58234d000ebfd7fb
Diffstat (limited to 'src/vnet/session/application_local.c')
| -rw-r--r-- | src/vnet/session/application_local.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/vnet/session/application_local.c b/src/vnet/session/application_local.c index 0e1f6db19fb..927bf91a040 100644 --- a/src/vnet/session/application_local.c +++ b/src/vnet/session/application_local.c @@ -294,7 +294,6 @@ ct_connect (app_worker_t * client_wrk, session_t * ll, if (ct_init_local_session (client_wrk, server_wrk, sct, ss, ll)) { - clib_warning ("failed"); ct_connection_free (sct); session_free (ss); return -1; @@ -303,9 +302,9 @@ ct_connect (app_worker_t * client_wrk, session_t * ll, ss->session_state = SESSION_STATE_ACCEPTING; if (app_worker_accept_notify (server_wrk, ss)) { - clib_warning ("failed"); ct_connection_free (sct); - session_free_w_fifos (ss); + segment_manager_dealloc_fifos (ss->rx_fifo, ss->tx_fifo); + session_free (ss); return -1; } |