diff options
| author | James Smart <[email protected]> | 2017-03-04 09:30:24 -0800 |
|---|---|---|
| committer | Martin K. Petersen <[email protected]> | 2017-03-06 23:04:22 -0500 |
| commit | d11f54b7d1d40463024be3b40ab7be2c8a84fa43 (patch) | |
| tree | 3ca62489bbc6e48313bdd480206ca581c65d9a0c | |
| parent | 332ba3b5d6d27a60d445704ed7c88c7e9f958a30 (diff) | |
scsi: lpfc: fix missing spin_unlock on sql_list_lock
From: Colin Ian King <[email protected]>
In the case where sglq is null, the current code just returns without
unlocking the spinlock sql_list_lock. Fix this by breaking out of the
while loop and the exit path will then unlock and return NULL as was
the original intention.
Detected by CoverityScan, CID#1411635 ("Missing unlock")
Signed-off-by: Colin Ian King <[email protected]>
Signed-off-by: James Smart <[email protected]>
Signed-off-by: Martin K. Petersen <[email protected]>
| -rw-r--r-- | drivers/scsi/lpfc/lpfc_sli.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c index 562a5286bc47..7389c130ffcc 100644 --- a/drivers/scsi/lpfc/lpfc_sli.c +++ b/drivers/scsi/lpfc/lpfc_sli.c @@ -952,7 +952,7 @@ __lpfc_sli_get_els_sglq(struct lpfc_hba *phba, struct lpfc_iocbq *piocbq) start_sglq = sglq; while (!found) { if (!sglq) - return NULL; + break; if (ndlp && ndlp->active_rrqs_xri_bitmap && test_bit(sglq->sli4_lxritag, ndlp->active_rrqs_xri_bitmap)) { |