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>

diff --git a/zebra/if_netlink.c b/zebra/if_netlink.c
index 61a1807fc9adc71f3ec4ab02d77a56f5bed89e9b..98bde4b3c05640545672ef8a551db0c9a3e8abdf 100644 (file)
--- 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])