mpt2sas: fix oops when firmware sends large sense buffer size

PATCH New Eric Moore 1 participant 1 message Apr 9, 2009 SCSI

Share & tools

Eric Moore

author

2009-04-09 23:39 UTC #0 lore raw

PATCH 3/4 mpt2sas: fix oops when firmware sends large sense buffer size

There is a bug in firmware where the reply message frame says there is a 16kb sense buffer, when in reality its only 20 bytes. This fix insures the memcpy action doesn't corrupte the memory beyond the 90 bytes allocated in the scsi command for sense buffer.

Signed-off-by Eric Moore <[email protected]>

drivers/scsi/mpt2sas/mpt2sas_scsih.c +2 -1

diff -uaprN a/drivers/scsi/mpt2sas/mpt2sas_scsih.c b/drivers/scsi/mpt2sas/mpt2sas_scsih.c
@@ -2863,8 +2863,9 @@ scsih_io_done(struct MPT2SAS_ADAPTER *io
 		struct sense_info data;
 		const void *sense_data = mpt2sas_base_get_sense_buffer(ioc,
 		    smid);
-		memcpy(scmd->sense_buffer, sense_data,
+		u32 sz = min_t(u32, SCSI_SENSE_BUFFERSIZE,
 		    le32_to_cpu(mpi_reply->SenseCount));
+		memcpy(scmd->sense_buffer, sense_data, sz);
 		_scsih_normalize_sense(scmd->sense_buffer, &data);
 		/* failure prediction threshold exceeded */
 		if (data.asc == 0x5D)