lb: fix memcpy error issue 70/21470/4
authorYulong Pei <yulong.pei@intel.com>
Fri, 23 Aug 2019 09:24:13 +0000 (17:24 +0800)
committerDamjan Marion <dmarion@me.com>
Thu, 29 Aug 2019 12:34:16 +0000 (12:34 +0000)
vl_api_prefix_t size is 21 byte, ip46_address_t size is 16 byte, only copy
16 byte of vl_api_prefix_t to ip46_address_t is not correct.

Type:fix

Change-Id: I35ede7836a0c878d39388b29b15d91bb08f87a07
Signed-off-by: Yulong Pei <yulong.pei@intel.com>
src/plugins/lb/api.c

index 86e2ce0..5354179 100644 (file)
@@ -120,7 +120,7 @@ vl_api_lb_add_del_vip_t_handler
       mp->protocol = ~0;
     }
 
-  memcpy (&(args.prefix), &mp->pfx, sizeof(args.prefix));
+  ip_address_decode (&mp->pfx.address, &(args.prefix));
 
   if (mp->is_del) {
     u32 vip_index;
@@ -210,14 +210,10 @@ vl_api_lb_add_del_as_t_handler
   int rv = 0;
   u32 vip_index;
   ip46_address_t vip_ip_prefix;
-
-  memcpy(&vip_ip_prefix, &mp->pfx,
-              sizeof(vip_ip_prefix));
-
   ip46_address_t as_address;
 
-  memcpy(&as_address, &mp->as_address,
-         sizeof(as_address));
+  ip_address_decode (&mp->pfx.address, &vip_ip_prefix);
+  ip_address_decode (&mp->as_address, &as_address);
 
   if ((rv = lb_vip_find_index(&vip_ip_prefix, mp->pfx.len,
                               mp->protocol, ntohs(mp->port), &vip_index)))