aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/arm
diff options
context:
space:
mode:
authorHannes Reinecke <hare@suse.de>2014-10-24 08:26:57 -0400
committerChristoph Hellwig <hch@lst.de>2014-11-12 05:16:03 -0500
commita9a47bf58ac1d5525ae99922e055d8de87eeae78 (patch)
treeb6f3e5f7ed4879e19fb898774dcd2db8e6e3c43c /drivers/scsi/arm
parent2478a736a7d01e3ef8d273e8fc5b11b6ed9af3ea (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.c2
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);