From: Wei Yongjun <yongjun_...@trendmicro.com.cn>

Add the missing unlock before return from function bfad_iocmd_cfg_trunk()
in the error handling case.

Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>
---
 drivers/scsi/bfa/bfad_bsg.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/scsi/bfa/bfad_bsg.c b/drivers/scsi/bfa/bfad_bsg.c
index 157f604..8994fb8 100644
--- a/drivers/scsi/bfa/bfad_bsg.c
+++ b/drivers/scsi/bfa/bfad_bsg.c
@@ -2304,8 +2304,10 @@ bfad_iocmd_cfg_trunk(struct bfad_s *bfad, void *cmd, 
unsigned int v_cmd)
 
        spin_lock_irqsave(&bfad->bfad_lock, flags);
 
-       if (bfa_fcport_is_dport(&bfad->bfa))
+       if (bfa_fcport_is_dport(&bfad->bfa)) {
+               spin_unlock_irqrestore(&bfad->bfad_lock, flags);
                return BFA_STATUS_DPORT_ERR;
+       }
 
        if ((fcport->cfg.topology == BFA_PORT_TOPOLOGY_LOOP) ||
                (fcport->topology == BFA_PORT_TOPOLOGY_LOOP))

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to