linux_dsm_epyc7002/drivers/net/ethernet/intel
Alex Williamson d0f63acc2f igb: Fix null pointer dereference
The max_vfs= option has always been self limiting to the number of VFs
supported by the device.  fa44f2f1 added SR-IOV configuration via
sysfs, but in the process broke this self correction factor.  The
failing path is:

igb_probe
  igb_sw_init
    if (max_vfs > 7) {
        adapter->vfs_allocated_count = 7;
    ...
    igb_probe_vfs
    igb_enable_sriov(, max_vfs)
      if (num_vfs > 7) {
        err = -EPERM;
        ...

This leaves vfs_allocated_count = 7 and vf_data = NULL, so we bomb out
when igb_probe finally calls igb_reset.  It seems like a really bad
idea, and somewhat pointless, to set vfs_allocated_count separate from
vf_data, but limiting max_vfs is enough to avoid the null pointer.

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Acked-by: Greg Rose <gregory.v.rose@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
2013-03-26 03:00:32 -07:00
..
e1000 e1000: fix whitespace issues and multi-line comments 2013-02-15 21:46:37 -08:00
e1000e e1000e: fix accessing to suspended device 2013-03-07 02:55:30 -08:00
igb igb: Fix null pointer dereference 2013-03-26 03:00:32 -07:00
igbvf igbvf: Make next_to_watch a pointer and adjust memory barriers to avoid races 2013-02-15 21:46:51 -08:00
ixgb drivers: net: Remove remaining alloc/OOM messages 2013-02-08 17:44:39 -05:00
ixgbe hlist: drop the node parameter from iterators 2013-02-27 19:10:24 -08:00
ixgbevf ixgbevf: don't release the soft entries 2013-03-26 02:31:48 -07:00
e100.c remove init of dev->perm_addr in drivers 2013-01-08 18:00:48 -08:00
Kconfig Driver core patches for 3.9-rc1 2013-02-21 12:05:51 -08:00
Makefile