diff options
author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2008-08-14 00:36:58 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-08-16 11:22:17 -0400 |
commit | bed475c4cde5d12f537f445bdd0a5a3bb5684481 (patch) | |
tree | e9479adbf180c34ceff3caa942a1fe555e26971c | |
parent | a7a28504bfa61dc6caa495f55c52623d10b42b32 (diff) |
[SCSI] qla2xxx: Don't leak SG-DMA mappings while aborting commands.
Original code inadvertently cleared an SRB's 'flags' while
aborting; causing a follow-on scsi_dma_unmap() to be potentially
missed.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
-rw-r--r-- | drivers/scsi/qla2xxx/qla_os.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index 7c8af7ed2a5d..c2092addb868 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c | |||
@@ -1080,9 +1080,7 @@ qla2x00_abort_all_cmds(scsi_qla_host_t *ha, int res) | |||
1080 | sp = ha->outstanding_cmds[cnt]; | 1080 | sp = ha->outstanding_cmds[cnt]; |
1081 | if (sp) { | 1081 | if (sp) { |
1082 | ha->outstanding_cmds[cnt] = NULL; | 1082 | ha->outstanding_cmds[cnt] = NULL; |
1083 | sp->flags = 0; | ||
1084 | sp->cmd->result = res; | 1083 | sp->cmd->result = res; |
1085 | sp->cmd->host_scribble = (unsigned char *)NULL; | ||
1086 | qla2x00_sp_compl(ha, sp); | 1084 | qla2x00_sp_compl(ha, sp); |
1087 | } | 1085 | } |
1088 | } | 1086 | } |