summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2018-04-18 00:05:21 -0700
committerDamjan Marion <dmarion.lists@gmail.com>2018-04-18 16:18:44 +0000
commit6c354941602df78f4c3bd78aa3b21fe447e3173d (patch)
treea471b2b643f268e6c25cee80e2d54ddbd87bfd2f
parent36ed73acb16c54d556ffd2bba10f0be05cc66ffb (diff)
session: coverity warnings
Change-Id: I53bb1e289ba16ee6d9aed7285b18437c996077a4 Signed-off-by: Florin Coras <fcoras@cisco.com>
-rw-r--r--src/vnet/session-apps/echo_client.c2
-rwxr-xr-xsrc/vnet/session/session_api.c4
2 files changed, 3 insertions, 3 deletions
diff --git a/src/vnet/session-apps/echo_client.c b/src/vnet/session-apps/echo_client.c
index 11528ef1899..d47b0d97337 100644
--- a/src/vnet/session-apps/echo_client.c
+++ b/src/vnet/session-apps/echo_client.c
@@ -49,8 +49,8 @@ send_data_chunk (echo_client_main_t * ecm, session_t * s)
int test_buf_len, test_buf_offset, rv;
u32 bytes_this_chunk;
- ASSERT (vec_len (test_data) > 0);
test_buf_len = vec_len (test_data);
+ ASSERT (test_buf_len > 0);
test_buf_offset = s->bytes_sent % test_buf_len;
bytes_this_chunk = clib_min (test_buf_len - test_buf_offset,
s->bytes_to_send);
diff --git a/src/vnet/session/session_api.c b/src/vnet/session/session_api.c
index 67c42faf692..1a41dbd9e08 100755
--- a/src/vnet/session/session_api.c
+++ b/src/vnet/session/session_api.c
@@ -514,12 +514,12 @@ done:
if (!rv)
{
rmp->handle = a->handle;
- rmp->lcl_is_ip4 = tc->is_ip4;
- rmp->lcl_port = tc->lcl_port;
if (app && application_has_global_scope (app))
{
s = listen_session_get_from_handle (a->handle);
tc = listen_session_get_transport (s);
+ rmp->lcl_is_ip4 = tc->is_ip4;
+ rmp->lcl_port = tc->lcl_port;
clib_memcpy (rmp->lcl_ip, &tc->lcl_ip, sizeof(tc->lcl_ip));
if (session_transport_service_type (s) == TRANSPORT_SERVICE_CL)
{