Commit 8b6d5c09 authored Jul 31, 2012 by Jesper Juhl Committed by David S. Miller Aug 02, 2012

bnx2x: fix mem leak when command is unknown



In bnx2x_mcast_enqueue_cmd() we'll leak the memory allocated to
'new_cmd' if we hit the deafault case of the 'switch (cmd)'.
Add a 'kfree(new_cmd)' to that case to avoid the leak.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: Dmitry Kravkov <dmitry@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent e3c0d047

drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c

+1 −0

Original line number	Original line	Diff line number	Diff line
	@@ -2485,6 +2485,7 @@ static int bnx2x_mcast_enqueue_cmd(struct bnx2x *bp,
	break;		break;

	default:		default:
			kfree(new_cmd);
	BNX2X_ERR("Unknown command: %d\n", cmd);		BNX2X_ERR("Unknown command: %d\n", cmd);
	return -EINVAL;		return -EINVAL;
	}		}