diff options
| author | Anuradha Karuppiah <anuradhak@nvidia.com> | 2021-02-19 02:49:20 +0100 |
|---|---|---|
| committer | Anuradha Karuppiah <anuradhak@nvidia.com> | 2021-02-24 17:11:26 +0100 |
| commit | 3f589fa8ecd0eafadef6938637aa8f442702ae91 (patch) | |
| tree | 00911cf7256e1321bd344d69b682c8a8b4c37b06 /zebra/if_netlink.c | |
| parent | tests: increase ping attempts in EVPN MH to fix failures on ARM (diff) | |
| download | frr-3f589fa8ecd0eafadef6938637aa8f442702ae91.tar.xz frr-3f589fa8ecd0eafadef6938637aa8f442702ae91.zip | |
zebra: fix problem with bypass getting set accidentally on all ESs
This was caused because of uninitialized netlint attrs in the bond-member
netlink parse API.
PS: It was caught by the upstream topotests on ARM8 (passed everywhere
else).
Signed-off-by: Anuradha Karuppiah <anuradhak@nvidia.com>
Diffstat (limited to '')
| -rw-r--r-- | zebra/if_netlink.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/zebra/if_netlink.c b/zebra/if_netlink.c index 61a1807fc..98bde4b3c 100644 --- a/zebra/if_netlink.c +++ b/zebra/if_netlink.c @@ -724,6 +724,7 @@ static uint8_t netlink_parse_lacp_bypass(struct rtattr **linkinfo) uint8_t bypass = 0; struct rtattr *mbrinfo[IFLA_BOND_SLAVE_MAX + 1]; + memset(mbrinfo, 0, sizeof(mbrinfo)); parse_rtattr_nested(mbrinfo, IFLA_BOND_SLAVE_MAX, linkinfo[IFLA_INFO_SLAVE_DATA]); if (mbrinfo[IFLA_BOND_SLAVE_AD_RX_BYPASS]) |