Commit bb132083 authored Jul 21, 2019 by Navid Emamdoost Committed by David S. Miller Jul 21, 2019

allocate_flower_entry: should check for null deref

allocate_flower_entry does not check for allocation success, but tries
to deref the result. I only moved the spin_lock under null check, because
the caller is checking allocation's status at line 652.

Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 4803d010

drivers/net/ethernet/chelsio/cxgb4/cxgb4_tc_flower.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -67,6 +67,7 @@ static struct ch_tc_pedit_fields pedits[] = {
		static struct ch_tc_flower_entry *allocate_flower_entry(void)
		{
		struct ch_tc_flower_entry new = kzalloc(sizeof(new), GFP_KERNEL);
		if (new)
		spin_lock_init(&new->lock);
		return new;
		}

Admin message