aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFilip Tehlar <ftehlar@cisco.com>2016-06-30 09:46:08 +0200
committerFilip Tehlar <ftehlar@cisco.com>2016-06-30 10:01:02 +0200
commitc4770ecc622e815bd6f355aea0bedfe38820c757 (patch)
tree30cf261f5fc4568cb0bed4dcf94634806ec60f5a
parentec175104cf5b09a91f2f9c7a36c6ca0703979eeb (diff)
Fix wrong interger type when unformating value in VAT
Since format string "%d" expects a 32 bit value using u8 type in unformat() results in buffer overflow. Change-Id: Idee5b588049d7573ddbde56c28030f9db3758493 Signed-off-by: Filip Tehlar <ftehlar@cisco.com>
-rw-r--r--vpp-api-test/vat/api_format.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/vpp-api-test/vat/api_format.c b/vpp-api-test/vat/api_format.c
index ad9986984ec..504c3b40854 100644
--- a/vpp-api-test/vat/api_format.c
+++ b/vpp-api-test/vat/api_format.c
@@ -10013,9 +10013,9 @@ api_lisp_add_del_locator(vat_main_t * vam)
u32 sw_if_index = ~0;
u8 sw_if_index_set = 0;
u8 sw_if_index_if_name_set = 0;
- u8 priority = ~0;
+ u32 priority = ~0;
u8 priority_set = 0;
- u8 weight = ~0;
+ u32 weight = ~0;
u8 weight_set = 0;
u8 is_add = 1;
u8 *locator_set_name = NULL;
@@ -10112,7 +10112,7 @@ api_lisp_add_del_local_eid(vat_main_t * vam)
ip4_address_t eidv4;
ip6_address_t eidv6;
u8 mac[6] = {0};
- u8 tmp_eid_lenght = ~0;
+ u32 tmp_eid_lenght = ~0;
u8 eid_lenght = ~0;
u8 *locator_set_name = NULL;
u8 locator_set_name_set = 0;
@@ -10221,7 +10221,7 @@ api_lisp_gpe_add_del_fwd_entry(vat_main_t * vam)
u8 eidv6_set = 0, slocv6_set = 0, dlocv6_set = 0;
ip4_address_t eidv4, slocv4, dlocv4;
ip6_address_t eidv6, slocv6, dlocv6;
- u8 tmp_eid_lenght = ~0;
+ u32 tmp_eid_lenght = ~0;
u8 eid_lenght = ~0;
/* Parse args required to build the message */