scsi: hpsa: fix an uninitialized read and dereference of pointer dev
authorColin Ian King <colin.king@canonical.com>
Wed, 22 May 2019 08:39:03 +0000 (09:39 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 18 Jun 2019 23:46:23 +0000 (19:46 -0400)
Currently the check for a lockup_detected failure exits via the label
return_reset_status that reads and dereferences an uninitialized pointer
dev.  Fix this by ensuring dev is inintialized to null.

Addresses-Coverity: ("Uninitialized pointer read")
Fixes: 14991a5bade5 ("scsi: hpsa: correct device resets")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Don Brace <don.brace@microsemi.com>
Reviewed-by: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/hpsa.c

index c560a45..ac8338b 100644 (file)
@@ -5947,7 +5947,7 @@ static int hpsa_eh_device_reset_handler(struct scsi_cmnd *scsicmd)
        int rc = SUCCESS;
        int i;
        struct ctlr_info *h;
-       struct hpsa_scsi_dev_t *dev;
+       struct hpsa_scsi_dev_t *dev = NULL;
        u8 reset_type;
        char msg[48];
        unsigned long flags;