Re: [patch v2] scsi_dh_alua: uninitialized variable in alua_check_vpd()

From: Manoj Kumar
Date: Fri Mar 11 2016 - 09:39:48 EST

On 3/11/2016 5:19 AM, Dan Carpenter wrote:
The pg_updated variable is support to be set to false at the start but
it is uninitialized.

Fixes: cb0a168cb6b8 ('scsi_dh_alua: update 'access_state' field')
Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c b/drivers/scsi/device_handler/scsi_dh_alua.c
index 5bcdf8d..a404a41 100644
--- a/drivers/scsi/device_handler/scsi_dh_alua.c
+++ b/drivers/scsi/device_handler/scsi_dh_alua.c
@@ -332,7 +332,7 @@ static int alua_check_vpd(struct scsi_device *sdev, struct alua_dh_data *h,
int rel_port = -1, group_id;
struct alua_port_group *pg, *old_pg = NULL;
- bool pg_updated;
+ bool pg_updated = false;
unsigned long flags;

Reviewed-by: Manoj Kumar <manoj@xxxxxxxxxxxxxxxxxx>

Manoj Kumar