From: Hannes Reinecke <hare@suse.de>
Date: Wed, 13 Jan 2021 09:04:57 +0000 (+0100)
Subject: scsi: storvsc: Return DID_ERROR for invalid commands
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=ecc751b27a575e2939767b8bf11b05924f442dd2;p=linux.git

scsi: storvsc: Return DID_ERROR for invalid commands

ILLEGAL_COMMAND is a sense code, not a driver byte.

Link: https://lore.kernel.org/r/20210113090500.129644-33-hare@suse.de
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
---

diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
index f0fa42f9374c6..6bc5453cea8a7 100644
--- a/drivers/scsi/storvsc_drv.c
+++ b/drivers/scsi/storvsc_drv.c
@@ -1671,7 +1671,7 @@ static bool storvsc_scsi_cmd_ok(struct scsi_cmnd *scmnd)
 	 * this. So, don't send it.
 	 */
 	case SET_WINDOW:
-		scmnd->result = ILLEGAL_REQUEST << 16;
+		scmnd->result = DID_ERROR << 16;
 		allowed = false;
 		break;
 	default: