mirror of
https://github.com/torvalds/linux.git
synced 2024-10-30 16:51:45 +00:00
RDMA/nes: Remove redundant NULL check in nes_unregister_ofa_device()
nes_unregister_ofa_device() dereferences the nesibdev pointer before testing if it's NULL. Also, the test is doubly redundant because the only caller of nes_unregister_ofa_device() is nes_destroy_ofa_device(), which already tests if nesibdev is NULL. Remove the unnecessary test. This was spotted by the Coverity checker (CID 2190). Signed-off-by: Roland Dreier <rolandd@cisco.com>
This commit is contained in:
parent
a7dab9e887
commit
5bd8341ce2
@ -3900,9 +3900,6 @@ void nes_unregister_ofa_device(struct nes_ib_device *nesibdev)
|
||||
struct nes_vnic *nesvnic = nesibdev->nesvnic;
|
||||
int i;
|
||||
|
||||
if (nesibdev == NULL)
|
||||
return;
|
||||
|
||||
for (i = 0; i < ARRAY_SIZE(nes_class_attributes); ++i) {
|
||||
class_device_remove_file(&nesibdev->ibdev.class_dev, nes_class_attributes[i]);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user