diff options
| author |   Florin Coras <fcoras@cisco.com> | 2018-04-18 00:05:21 -0700 |
|---|---|---|
| committer |   Damjan Marion <dmarion.lists@gmail.com> | 2018-04-18 16:18:44 +0000 |
| commit | 6c354941602df78f4c3bd78aa3b21fe447e3173d (patch) | |
| tree | a471b2b643f268e6c25cee80e2d54ddbd87bfd2f | |
| parent | 36ed73acb16c54d556ffd2bba10f0be05cc66ffb (diff) | |
session: coverity warnings
Change-Id: I53bb1e289ba16ee6d9aed7285b18437c996077a4
Signed-off-by: Florin Coras <fcoras@cisco.com>
| -rw-r--r-- | src/vnet/session-apps/echo_client.c | 2 | ||||
| -rwxr-xr-x | src/vnet/session/session_api.c | 4 |
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) { |