diff options
author | Johannes Thumshirn <johannes.thumshirn@wdc.com> | 2020-07-01 15:14:54 +0200 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2020-07-08 05:46:38 +0200 |
commit | 0fd181456aa0826057adbfb6c79c40f4083cfd75 (patch) | |
tree | a32ea34880208057a617949b59f4d0c1762b19f5 /.cocciconfig | |
parent | scsi: mpt3sas: Fix unlock imbalance (diff) | |
download | linux-0fd181456aa0826057adbfb6c79c40f4083cfd75.tar.xz linux-0fd181456aa0826057adbfb6c79c40f4083cfd75.zip |
scsi: mpt3sas: Fix error returns in BRM_status_show
BRM_status_show() has several error branches, but none of them record the
error in the error return.
Also while at it remove the manual mutex_unlock() of the pci_access_mutex
in case of an ongoing pci error recovery or host removal and jump to the
cleanup label instead.
Note: We can safely jump to out from here as io_unit_pg3 is initialized to
NULL and if it hasn't been allocated, kfree() skips the NULL pointer.
[mkp: compilation warning]
Link: https://lore.kernel.org/r/20200701131454.5255-1-johannes.thumshirn@wdc.com
Reviewed-by: Damien Le Moal <damien.lemoal@wdc.com>
Acked-by: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions