summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2018-07-05 23:44:23 -0700
committerDamjan Marion <dmarion@me.com>2018-07-08 11:21:23 +0000
commit95e0ce05470554e403ade1db322800e489165b1b (patch)
tree47264bc99f04bce9a5f7e747c2b5df08a0ed2644
parente7cde313e1df14ea118386d678c3e47219090a0d (diff)
session/svm: fix coverity warnings
Change-Id: Ic5daed1f6c23d9561a04e235dcbf257f190d066a Signed-off-by: Florin Coras <fcoras@cisco.com>
-rw-r--r--src/svm/message_queue.c7
-rw-r--r--src/vnet/session/session.c6
2 files changed, 10 insertions, 3 deletions
diff --git a/src/svm/message_queue.c b/src/svm/message_queue.c
index dc0f6251120..4f3e7642740 100644
--- a/src/svm/message_queue.c
+++ b/src/svm/message_queue.c
@@ -21,6 +21,7 @@ svm_msg_q_alloc (svm_msg_q_cfg_t * cfg)
{
svm_msg_q_ring_t *ring;
svm_msg_q_t *mq;
+ uword size;
int i;
if (!cfg)
@@ -39,8 +40,10 @@ svm_msg_q_alloc (svm_msg_q_cfg_t * cfg)
if (cfg->ring_cfgs[i].data)
ring->data = cfg->ring_cfgs[i].data;
else
- ring->data = clib_mem_alloc_aligned (ring->nitems * ring->elsize,
- CLIB_CACHE_LINE_BYTES);
+ {
+ size = (uword) ring->nitems * ring->elsize;
+ ring->data = clib_mem_alloc_aligned (size, CLIB_CACHE_LINE_BYTES);
+ }
}
return mq;
diff --git a/src/vnet/session/session.c b/src/vnet/session/session.c
index 7131f51cdda..26bc70e6fd7 100644
--- a/src/vnet/session/session.c
+++ b/src/vnet/session/session.c
@@ -1217,7 +1217,11 @@ session_vpp_event_queues_allocate (session_manager_main_t * smm)
eqs->name = format (0, "%s%c", "evt-qs-segment", 0);
eqs->requested_va = smm->session_baseva;
- ssvm_master_init (eqs, SSVM_SEGMENT_MEMFD);
+ if (ssvm_master_init (eqs, SSVM_SEGMENT_MEMFD))
+ {
+ clib_warning ("failed to initialize queue segment");
+ return;
+ }
}
if (smm->evt_qs_use_memfd_seg)