diff options
| author | Anirudh Venkataramanan <[email protected]> | 2018-08-09 13:29:00 +0000 |
|---|---|---|
| committer | Jeff Kirsher <[email protected]> | 2018-08-23 18:28:56 +0000 |
| commit | c7f2c42b80ed6009f44e355aefc1e40db9485a9d (patch) | |
| tree | a45b97e5aaadd3db57b82becfbc06294f289d680 /drivers/net/ethernet/intel/ice/ice_common.c | |
| parent | ice: Update to interrupts enabled in OICR (diff) | |
| download | kernel-c7f2c42b80ed6009f44e355aefc1e40db9485a9d.tar.gz kernel-c7f2c42b80ed6009f44e355aefc1e40db9485a9d.zip | |
ice: Fix a few null pointer dereference issues
1) When ice_ena_msix_range() fails to reserve vectors, a devm_kfree()
warning was seen in the error flow path. So check pf->irq_tracker
before use in ice_clear_interrupt_scheme().
2) In ice_vsi_cfg(), check vsi->netdev before use.
3) In ice_get_link_status, check link_up before use.
Signed-off-by: Anirudh Venkataramanan <[email protected]>
Tested-by: Tony Brelinski <[email protected]>
Signed-off-by: Jeff Kirsher <[email protected]>
Diffstat (limited to 'drivers/net/ethernet/intel/ice/ice_common.c')
| -rw-r--r-- | drivers/net/ethernet/intel/ice/ice_common.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ice/ice_common.c b/drivers/net/ethernet/intel/ice/ice_common.c index d5300b606d5a..ebd701ac9428 100644 --- a/drivers/net/ethernet/intel/ice/ice_common.c +++ b/drivers/net/ethernet/intel/ice/ice_common.c @@ -1483,7 +1483,7 @@ enum ice_status ice_get_link_status(struct ice_port_info *pi, bool *link_up) struct ice_phy_info *phy_info; enum ice_status status = 0; - if (!pi) + if (!pi || !link_up) return ICE_ERR_PARAM; phy_info = &pi->phy; |