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

Commit 149d18cc authored by Hannes Reinecke's avatar Hannes Reinecke Committed by Christoph Hellwig
Browse files

scsi: stop decoding if scsi_normalize_sense() fails 



If scsi_normalize_sense() fails we couldn't decode the sense
buffer, and the scsi_sense_hdr fields are invalid.
For those cases we should rather dump the sense buffer
and not try to decode invalid fields.

Signed-off-by: default avatarHannes Reinecke <hare@suse.de>
Reviewed-by: default avatarRobert Elliott <elliott@hp.com>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent 7d170907

drivers/scsi/constants.c

+16 −18
Original line number Diff line number Diff line
@@ -1436,14 +1436,10 @@ scsi_print_sense_hdr(const struct scsi_device *sdev, const char *name, 
EXPORT_SYMBOL(scsi_print_sense_hdr);



static void

scsi_decode_sense_buffer(const unsigned char *sense_buffer, int sense_len,

		       struct scsi_sense_hdr *sshdr)

scsi_dump_sense_buffer(const unsigned char *sense_buffer, int sense_len)

{

	int k, num, res;

	int k, num;



	res = scsi_normalize_sense(sense_buffer, sense_len, sshdr);

	if (0 == res) {

		/* this may be SCSI-1 sense data */

	num = (sense_len < 32) ? sense_len : 32;

	printk("Unrecognized sense data (in hex):");

	for (k = 0; k < num; ++k) {
@@ -1456,7 +1452,6 @@ scsi_decode_sense_buffer(const unsigned char *sense_buffer, int sense_len, 
	printk("\n");

	return;

}

}



static void

scsi_decode_sense_extras(const unsigned char *sense_buffer, int sense_len,
@@ -1525,7 +1520,10 @@ void __scsi_print_sense(const struct scsi_device *sdev, const char *name, 
{

	struct scsi_sense_hdr sshdr;



	scsi_decode_sense_buffer(sense_buffer, sense_len, &sshdr);

	if (!scsi_normalize_sense(sense_buffer, sense_len, &sshdr)) {

		scsi_dump_sense_buffer(sense_buffer, sense_len);

		return;

	}

	scsi_show_sense_hdr(sdev, name, &sshdr);

	scsi_decode_sense_extras(sense_buffer, sense_len, &sshdr);

	scsi_show_extd_sense(sdev, name, sshdr.asc, sshdr.ascq);