diff options
| author |   Steven Luong <sluong@cisco.com> | 2024-08-05 14:58:19 -0700 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2024-08-06 15:52:35 +0000 |
| commit | fc3464dac99c9e3e56616e6c6bc6d10886b2f567 (patch) | |
| tree | b5fd9aaab6960bdd62ac1c2b1b270825b67f32db /src | |
| parent | 26a6b19a0efb0604103e3d4275406fe6c498f878 (diff) | |
session: session_lookup_local_endpoint returns the wrong value
SESSION_INVALID_INDEX is u32. The function is supposed to return u64.
The caller of the function tests the return value to see if it is
SESSION_INVALID_HANDLE in ct_session_connect. If it is not, it
thinks it is a valid handle and calls listen_session_get_from_handle
which causes a crash.
Type: fix
Change-Id: I4ede6b2dfb95259f3a6071cfa3a77ce0d9cd9edd
Signed-off-by: Steven Luong <sluong@cisco.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/vnet/session/session_lookup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/vnet/session/session_lookup.c b/src/vnet/session/session_lookup.c index 1e58e5e7806..3a461a5949f 100644 --- a/src/vnet/session/session_lookup.c +++ b/src/vnet/session/session_lookup.c @@ -595,7 +595,7 @@ session_lookup_local_endpoint (u32 table_index, session_endpoint_t * sep) st = session_table_get (table_index); if (!st) - return SESSION_INVALID_INDEX; + return SESSION_INVALID_HANDLE; ASSERT (st->is_local); if (sep->is_ip4) |