Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c021be62 authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman
Browse files

scsi: scsi_dh_alua: Fix signedness bug in alua_rtpg() 

commit 80927822e8b6be46f488524cd7d5fe683de97fc4 upstream.

The "retval" variable needs to be signed for the error handling to work.

Link: https://lore.kernel.org/r/YLjMEAFNxOas1mIp@mwanda


Fixes: 7e26e3ea0287 ("scsi: scsi_dh_alua: Check for negative result value")
Reviewed-by: default avatarMartin Wilck <mwilck@suse.com>
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 2e70bf39

drivers/scsi/device_handler/scsi_dh_alua.c

+2 −1
Original line number Diff line number Diff line
@@ -522,7 +522,8 @@ static int alua_rtpg(struct scsi_device *sdev, struct alua_port_group *pg) 
	struct alua_port_group *tmp_pg;

	int len, k, off, bufflen = ALUA_RTPG_SIZE;

	unsigned char *desc, *buff;

	unsigned err, retval;

	unsigned err;

	int retval;

	unsigned int tpg_desc_tbl_off;

	unsigned char orig_transition_tmo;

	unsigned long flags;