scsi: stop decoding if scsi_normalize_sense() fails
authorHannes Reinecke <hare@suse.de>
Fri, 24 Oct 2014 12:26:50 +0000 (14:26 +0200)
committerChristoph Hellwig <hch@lst.de>
Wed, 12 Nov 2014 10:16:00 +0000 (11:16 +0100)
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: Hannes Reinecke <hare@suse.de>
Reviewed-by: Robert Elliott <elliott@hp.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/constants.c

index 9065b6f8f51b421d5e805d0b3c6fd620dbb2442c..d7b6e4bfa55c13a46a4ecaa40d639dea7377b509 100644 (file)
@@ -1436,26 +1436,21 @@ 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;
-
-       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) {
-                       if (0 == (k % 16)) {
-                               printk("\n");
-                               printk(KERN_INFO "        ");
-                       }
-                       printk("%02x ", sense_buffer[k]);
+       int k, num;
+
+       num = (sense_len < 32) ? sense_len : 32;
+       printk("Unrecognized sense data (in hex):");
+       for (k = 0; k < num; ++k) {
+               if (0 == (k % 16)) {
+                       printk("\n");
+                       printk(KERN_INFO "        ");
                }
-               printk("\n");
-               return;
+               printk("%02x ", sense_buffer[k]);
        }
+       printk("\n");
+       return;
 }
 
 static void
@@ -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);