diff options
author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-09-24 18:56:53 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-10-18 14:36:10 -0400 |
commit | 9ef9977cabc1b2c1718ef6eb883caec8dcb80b4c (patch) | |
tree | 54ae4e1c2c42603da00ac8059b94bb6b4e8e38ee /drivers | |
parent | dd5619cb4407e830a8921a93c949be37c81105b5 (diff) |
pciehp: Remove unnecessary check in pciehp_ctrl.c
this was spotted by coverity (cid #819). We dereference p_slot
earlier in the function, and i found no way it could become NULL
anywhere.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/pci/hotplug/pciehp_ctrl.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/hotplug/pciehp_ctrl.c b/drivers/pci/hotplug/pciehp_ctrl.c index c206a3d63b63..372c63e35aa9 100644 --- a/drivers/pci/hotplug/pciehp_ctrl.c +++ b/drivers/pci/hotplug/pciehp_ctrl.c | |||
@@ -797,8 +797,7 @@ int pciehp_enable_slot(struct slot *p_slot) | |||
797 | p_slot->hpc_ops->get_latch_status(p_slot, &getstatus); | 797 | p_slot->hpc_ops->get_latch_status(p_slot, &getstatus); |
798 | } | 798 | } |
799 | 799 | ||
800 | if (p_slot) | 800 | update_slot_info(p_slot); |
801 | update_slot_info(p_slot); | ||
802 | 801 | ||
803 | mutex_unlock(&p_slot->ctrl->crit_sect); | 802 | mutex_unlock(&p_slot->ctrl->crit_sect); |
804 | return rc; | 803 | return rc; |