summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2016-08-05 16:57:33 +0200
committerDave Barach <openvpp@barachs.net>2016-08-08 19:06:19 +0000
commit7fbfad3ee30af92539c4095c41daec5686d6735d (patch)
treeb774d87e4ccf78d9f7e464e42436f92931cc8727
parent6f6f34f620a18638eb820a9ab3d8a93e350d722a (diff)
VPP-189 Fix new Coverity warnings
Change-Id: I2a580320118d9ab638d9ac0986544ebad8d50e67 Signed-off-by: Florin Coras <fcoras@cisco.com>
-rw-r--r--vpp-api-test/vat/api_format.c52
1 files changed, 38 insertions, 14 deletions
diff --git a/vpp-api-test/vat/api_format.c b/vpp-api-test/vat/api_format.c
index 93007fb45f3..03919aea9b5 100644
--- a/vpp-api-test/vat/api_format.c
+++ b/vpp-api-test/vat/api_format.c
@@ -11848,6 +11848,8 @@ api_lisp_gpe_add_del_fwd_entry (vat_main_t * vam)
ip6_address_t rmt_rloc6, lcl_rloc6;
rloc_t *rmt_locs = 0, *lcl_locs = 0, rloc, *curr_rloc = 0;
+ memset(&rloc, 0, sizeof(rloc));
+
/* Parse args required to build the message */
while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT)
{
@@ -11877,20 +11879,27 @@ api_lisp_gpe_add_del_fwd_entry (vat_main_t * vam)
&lcl_rloc4, unformat_ip4_address, &rmt_rloc4))
{
rloc.is_ip4 = 1;
+
clib_memcpy (&rloc.addr, &lcl_rloc4, sizeof (lcl_rloc4));
+ rloc.priority = rloc.weight = 0;
vec_add1 (lcl_locs, rloc);
+
clib_memcpy (&rloc.addr, &rmt_rloc4, sizeof (rmt_rloc4));
vec_add1 (rmt_locs, rloc);
+ /* priority and weight saved in rmt loc */
curr_rloc = &rmt_locs[vec_len (rmt_locs) - 1];
}
- else if (unformat (input, "loc-pair %U", unformat_ip6_address,
+ else if (unformat (input, "loc-pair %U %U", unformat_ip6_address,
&lcl_rloc6, unformat_ip6_address, &rmt_rloc6))
{
rloc.is_ip4 = 0;
clib_memcpy (&rloc.addr, &lcl_rloc6, sizeof (lcl_rloc6));
+ rloc.priority = rloc.weight = 0;
vec_add1 (lcl_locs, rloc);
+
clib_memcpy (&rloc.addr, &rmt_rloc6, sizeof (rmt_rloc6));
vec_add1 (rmt_locs, rloc);
+ /* priority and weight saved in rmt loc */
curr_rloc = &rmt_locs[vec_len (rmt_locs) - 1];
}
else if (unformat (input, "action %d", &action))
@@ -11916,6 +11925,12 @@ api_lisp_gpe_add_del_fwd_entry (vat_main_t * vam)
return -99;
}
+ if (0 == rmt_locs && (u32) ~0 == action)
+ {
+ errmsg ("action not set for negative mapping\n");
+ return -99;
+ }
+
/* Construct the API message */
M (LISP_GPE_ADD_DEL_FWD_ENTRY, lisp_gpe_add_del_fwd_entry);
@@ -11927,11 +11942,14 @@ api_lisp_gpe_add_del_fwd_entry (vat_main_t * vam)
mp->lcl_len = lcl_eid->len;
mp->action = action;
- mp->loc_num = vec_len (rmt_locs);
- clib_memcpy (mp->lcl_locs, lcl_locs,
- (sizeof (rloc_t) * vec_len (lcl_locs)));
- clib_memcpy (mp->rmt_locs, rmt_locs,
- (sizeof (rloc_t) * vec_len (rmt_locs)));
+ if (0 != rmt_locs && 0 != lcl_locs)
+ {
+ mp->loc_num = vec_len(rmt_locs);
+ clib_memcpy (mp->lcl_locs, lcl_locs,
+ (sizeof(rloc_t) * vec_len(lcl_locs)));
+ clib_memcpy (mp->rmt_locs, rmt_locs,
+ (sizeof(rloc_t) * vec_len(rmt_locs)));
+ }
vec_free (lcl_locs);
vec_free (rmt_locs);
@@ -12483,41 +12501,47 @@ api_lisp_gpe_add_del_iface (vat_main_t * vam)
unformat_input_t *input = vam->input;
vl_api_lisp_gpe_add_del_iface_t *mp;
f64 timeout = ~0;
- u8 is_set = 0, is_add = 1, is_l2 = 0;
- u32 dp_table, vni;
+ u8 action_set = 0, is_add = 1, is_l2 = 0, dp_table_set = 0, vni_set = 0;
+ u32 dp_table=0, vni=0;
/* Parse args required to build the message */
while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT)
{
if (unformat (input, "up"))
{
- is_set = 1;
+ action_set = 1;
is_add = 1;
}
else if (unformat (input, "down"))
{
- is_set = 1;
+ action_set = 1;
is_add = 0;
}
else if (unformat (input, "table_id %d", &dp_table))
{
- ;
+ dp_table_set = 1;
}
else if (unformat (input, "bd_id %d", &dp_table))
{
+ dp_table_set = 1;
is_l2 = 1;
}
else if (unformat (input, "vni %d", &vni))
{
- ;
+ vni_set = 1;
}
else
break;
}
- if (is_set == 0)
+ if (action_set == 0)
{
- errmsg ("Value not set\n");
+ errmsg ("Action not set\n");
+ return -99;
+ }
+ if (dp_table_set == 0 || vni_set == 0)
+ {
+ errmsg ("vni and dp_table must be set\n");
return -99;
}