From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752227AbdAaMpL (ORCPT ); Tue, 31 Jan 2017 07:45:11 -0500 Received: from mx02-sz.bfs.de ([194.94.69.103]:14356 "EHLO mx02-sz.bfs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752185AbdAaMov (ORCPT ); Tue, 31 Jan 2017 07:44:51 -0500 Message-ID: <5890846C.50406@bfs.de> Date: Tue, 31 Jan 2017 13:34:52 +0100 From: walter harms Reply-To: wharms@bfs.de User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; de; rv:1.9.1.16) Gecko/20101125 SUSE/3.0.11 Thunderbird/3.0.11 MIME-Version: 1.0 To: Colin King CC: "James E . J . Bottomley" , "Martin K . Petersen" , Jens Axboe , Hannes Reinecke , Shaun Tancheff , Johannes Thumshirn , linux-scsi@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] scsi_dh_emc: fix uninitialized return value ret References: <20170131122205.24482-1-colin.king@canonical.com> In-Reply-To: <20170131122205.24482-1-colin.king@canonical.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am 31.01.2017 13:22, schrieb Colin King: > From: Colin Ian King > > ret is uninitialized on a successful execution of clarrion_std_inquiry > and a garbage return value is being returted. Fix this by setting ret > to zero on the success exit path. > > Found by CoverityScan, CID#1398889 ("Uninitialized scaler variable") > > Fixes: b78205c93297b8219 ("scsi_dh_emc: switch to scsi_execute_req_flags()") > Signed-off-by: Colin Ian King > --- > drivers/scsi/device_handler/scsi_dh_emc.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/scsi/device_handler/scsi_dh_emc.c b/drivers/scsi/device_handler/scsi_dh_emc.c > index 4a7679f..6647da2 100644 > --- a/drivers/scsi/device_handler/scsi_dh_emc.c > +++ b/drivers/scsi/device_handler/scsi_dh_emc.c > @@ -376,6 +376,7 @@ static int clariion_std_inquiry(struct scsi_device *sdev, > sdev_printk(KERN_INFO, sdev, > "%s: detected Clariion %s, flags %x\n", > CLARIION_NAME, sp_model, csdev->flags); > + err = 0; > out: > return err; > } I am not sure IMHO err is set in the first line err = send_inquiry_cmd(sdev, 0, csdev); but this looks strange: if (!strlen(sp_model) || !strncmp(sp_model, "FC",2)) IMHO we can drop strlen() and the compare should be agaist: #define emc_default_str "FC (Legacy)" just my 2 cents, re, wh