igb: check memory allocation failure
authorChristophe JAILLET <[email protected]>
Sun, 27 Aug 2017 06:39:51 +0000 (08:39 +0200)
committerJeff Kirsher <[email protected]>
Tue, 10 Oct 2017 16:01:11 +0000 (09:01 -0700)
Check memory allocation failures and return -ENOMEM in such cases, as
already done for other memory allocations in this function.

This avoids NULL pointers dereference.

Signed-off-by: Christophe JAILLET <[email protected]>
Tested-by: Aaron Brown <[email protected]
Acked-by: PJ Waskiewicz <[email protected]>
Signed-off-by: Jeff Kirsher <[email protected]>
drivers/net/ethernet/intel/igb/igb_main.c

index fd4a46b03cc8317f44f0c4d3ee5141dd05858244..837d9b46a390526711a1ebcb3b43c1a5b13a708b 100644 (file)
@@ -3162,6 +3162,8 @@ static int igb_sw_init(struct igb_adapter *adapter)
        /* Setup and initialize a copy of the hw vlan table array */
        adapter->shadow_vfta = kcalloc(E1000_VLAN_FILTER_TBL_SIZE, sizeof(u32),
                                       GFP_ATOMIC);
+       if (!adapter->shadow_vfta)
+               return -ENOMEM;
 
        /* This call may decrease the number of queues */
        if (igb_init_interrupt_scheme(adapter, true)) {