diff options
author | Hannes Reinecke <hare@suse.de> | 2014-10-24 08:26:57 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-11-12 05:16:03 -0500 |
commit | a9a47bf58ac1d5525ae99922e055d8de87eeae78 (patch) | |
tree | b6f3e5f7ed4879e19fb898774dcd2db8e6e3c43c /drivers/scsi/arm | |
parent | 2478a736a7d01e3ef8d273e8fc5b11b6ed9af3ea (diff) |
scsi: repurpose the last argument from print_opcode_name()
print_opcode_name() was only ever called with a '0' argument
from LLDDs and ULDs which were _not_ supporting variable length
CDBs, so the 'if' clause was never triggered.
Instead we should be using the last argument to specify
the cdb length to avoid accidental overflow when reading
the cdb buffer.
Signed-off-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Robert Elliott <elliott@hp.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/arm')
-rw-r--r-- | drivers/scsi/arm/fas216.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/arm/fas216.c b/drivers/scsi/arm/fas216.c index cea34633b90a..d2581cb41ec8 100644 --- a/drivers/scsi/arm/fas216.c +++ b/drivers/scsi/arm/fas216.c | |||
@@ -2424,7 +2424,7 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt) | |||
2424 | info->stats.aborts += 1; | 2424 | info->stats.aborts += 1; |
2425 | 2425 | ||
2426 | printk(KERN_WARNING "scsi%d: abort command ", info->host->host_no); | 2426 | printk(KERN_WARNING "scsi%d: abort command ", info->host->host_no); |
2427 | __scsi_print_command(SCpnt->cmnd); | 2427 | __scsi_print_command(SCpnt->cmnd, SCpnt->cmd_len); |
2428 | 2428 | ||
2429 | print_debug_list(); | 2429 | print_debug_list(); |
2430 | fas216_dumpstate(info); | 2430 | fas216_dumpstate(info); |