pcmcia: add error handling for pcmcia_enable_device in qlogic_stub
authorZhouyang Jia <jiazhouyang09@gmail.com>
Thu, 14 Jun 2018 23:41:38 +0000 (07:41 +0800)
committerDominik Brodowski <linux@dominikbrodowski.net>
Mon, 1 Oct 2018 10:16:45 +0000 (12:16 +0200)
When pcmcia_enable_device fails, the lack of error-handling code may
cause unexpected results.

This patch adds error-handling code after calling pcmcia_enable_device
in the qlogic_stub scsi driver.

Signed-off-by: Zhouyang Jia <jiazhouyang09@gmail.com>
Cc: James E.J. Bottomley <jejb@linux.vnet.ibm.com>
Cc: Martin K. Petersen <martin.petersen@oracle.com>
Cc: linux-scsi@vger.kernel.org
[linux@dominikbrodowski.net: update commit message]
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
drivers/scsi/pcmcia/qlogic_stub.c

index d4e7e36..173351a 100644 (file)
@@ -254,8 +254,12 @@ static void qlogic_release(struct pcmcia_device *link)
 static int qlogic_resume(struct pcmcia_device *link)
 {
        scsi_info_t *info = link->priv;
+       int ret;
+
+       ret = pcmcia_enable_device(link);
+       if (ret)
+               return ret;
 
-       pcmcia_enable_device(link);
        if ((info->manf_id == MANFID_MACNICA) ||
            (info->manf_id == MANFID_PIONEER) ||
            (info->manf_id == 0x0098)) {