scsi: scsi_dh_emc: return success in clariion_std_inquiry()
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 21 Feb 2017 18:46:37 +0000 (21:46 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 23 Feb 2017 00:28:06 +0000 (19:28 -0500)
We accidentally return an uninitialized variable on success.

Fixes: b6ff1b14cdf4 ("[SCSI] scsi_dh: Update EMC handler")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/device_handler/scsi_dh_emc.c

index 4a7679f6c73da04f5f57d20f822bef279a5d2643..f1578832ec7ad91e85a8e3f4434758ce1ca846f6 100644 (file)
@@ -358,7 +358,7 @@ static int clariion_prep_fn(struct scsi_device *sdev, struct request *req)
 static int clariion_std_inquiry(struct scsi_device *sdev,
                                struct clariion_dh_data *csdev)
 {
-       int err;
+       int err = SCSI_DH_OK;
        char *sp_model;
 
        sp_model = parse_sp_model(sdev, sdev->inquiry);