diff options
| author |   Florin Coras <fcoras@cisco.com> | 2018-09-04 19:47:52 -0700 |
|---|---|---|
| committer |   Marco Varlese <marco.varlese@suse.de> | 2018-09-06 08:37:20 +0000 |
| commit | 145987776f57de20e9724d58fe28796d40ffebfa (patch) | |
| tree | b63fb0f62472dc15660c9a1f6a721d33d04e8587 /src/vnet/session | |
| parent | ab2f6dbf9f7b7164a9810f4c80c8abf8463e42ad (diff) | |
vcl/session/svm: fix coverity warnings
Change-Id: I27532b3ab244dc95955e836a42b229a6e4e32818
Signed-off-by: Florin Coras <fcoras@cisco.com>
Diffstat (limited to 'src/vnet/session')
| -rwxr-xr-x | src/vnet/session/session_api.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/vnet/session/session_api.c b/src/vnet/session/session_api.c index aa29090383a..d8531a467dd 100755 --- a/src/vnet/session/session_api.c +++ b/src/vnet/session/session_api.c @@ -1354,8 +1354,8 @@ vl_api_app_worker_add_del_t_handler (vl_api_app_worker_add_del_t * mp) goto done; } - /* Make coverity happy */ - ASSERT (args.evt_q && args.segment); + if (!mp->is_add) + goto done; /* Send fifo segment fd if needed */ if (ssvm_type (args.segment) == SSVM_SEGMENT_MEMFD) @@ -1375,7 +1375,7 @@ vl_api_app_worker_add_del_t_handler (vl_api_app_worker_add_del_t * mp) done: REPLY_MACRO2 (VL_API_APP_WORKER_ADD_DEL_REPLY, ({ rmp->is_add = mp->is_add; - if (!rv) + if (!rv && mp->is_add) { rmp->wrk_index = clib_host_to_net_u32 (args.wrk_index); if (vec_len (args.segment->name)) |