Commit a67eed57 authored Jul 25, 2014 by Dan Carpenter Committed by David S. Miller Jul 28, 2014

bonding: fix a memory leak in bond_arp_send_all()



This test is reversed so the memory is always leaked.  It's better style
to remove the test anyway.

Fixes: 3e403a77 ('bonding: make it possible to have unlimited nested upper vlans')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Veaceslav Falico <vfalico@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent d87de1f3

drivers/net/bonding/bond_main.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -2287,7 +2287,6 @@ found:
		ip_rt_put(rt);
		bond_arp_send(slave->dev, ARPOP_REQUEST, targets[i],
		addr, tags);
		if (!tags)
		kfree(tags);
		}
		}