bonding: do not allow rlb updates to invalid mac

dbavatar · gregkh · commit 2eca993ddc83 · 2018-05-19T10:20:26.000+02:00
[ Upstream commit 4fa8667 ] Make sure multicast, broadcast, and zero mac's cannot be the output of rlb updates, which should all be directed arps. Receive load balancing will be collapsed if any of these happen, as the switch will broadcast. Signed-off-by: Debabrata Banerjee <dbanerje@akamai.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff --git a/drivers/net/bonding/bond_alb.c b/drivers/net/bonding/bond_alb.c
@@ -450,7 +450,7 @@ static void rlb_update_client(struct rlb_client_info *client_info)
 {
 	int i;
 
-	if (!client_info->slave)
+	if (!client_info->slave || !is_valid_ether_addr(client_info->mac_dst))
 		return;
 
 	for (i = 0; i < RLB_ARP_BURST_SIZE; i++) {

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ static void rlb_update_client(struct rlb_client_info *client_info)`
`450`	`450`	`{`
`451`	`451`	`int i;`
`452`	`452`
`453`		`- if (!client_info->slave)`
	`453`	`+ if (!client_info->slave \|\| !is_valid_ether_addr(client_info->mac_dst))`
`454`	`454`	`return;`
`455`	`455`
`456`	`456`	`for (i = 0; i < RLB_ARP_BURST_SIZE; i++) {`