diff options
| author |   Yulong Pei <yulong.pei@intel.com> | 2019-08-23 17:24:13 +0800 |
|---|---|---|
| committer |   Andrew Yourtchenko <ayourtch@gmail.com> | 2019-09-25 06:48:00 +0000 |
| commit | 7cd964ee45e845e0f02d9dbcecc3c1cc9088e447 (patch) | |
| tree | 27aeba75f9087c4fe341402b74de11e16f3ea2ea /src/plugins/lb | |
| parent | 54ff3fc834b9e97b77313501437260699e617e7c (diff) | |
lb: fix memcpy error issue
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>
(cherry picked from commit 71f36067ca923fa4eb10060d6f6878f5fa864673)
Diffstat (limited to 'src/plugins/lb')
| -rw-r--r-- | src/plugins/lb/api.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/plugins/lb/api.c b/src/plugins/lb/api.c index 86e2ce08ed7..53541799f49 100644 --- a/src/plugins/lb/api.c +++ b/src/plugins/lb/api.c @@ -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))) |