[SCSI] libsas: fix ata list corruption issue
authorJames Bottomley <James.Bottomley@suse.de>
Thu, 10 Mar 2011 23:13:18 +0000 (17:13 -0600)
committerJames Bottomley <James.Bottomley@suse.de>
Mon, 14 Mar 2011 23:46:25 +0000 (18:46 -0500)
commita82058a730c2bd01c43beb8a4847526a2998cc1a
tree92608d5ecc6e70976fb320fe0c48074b5e0e5a01
parent941b1cdae83039c99fc5c1884a98d2afd39760e5
[SCSI] libsas: fix ata list corruption issue

I think this stems from a misunderstanding of how the ata error handler
works.  ata_scsi_cmd_error_handler() gets called with a passed in list
of commands to handle.  However, that list may still not be empty when
it exits.  The command ata_scsi_port_error_handler() must be called
(which takes no list) before the list will be completely emptied.  This
bites the sas error handler because the two are called from different
functions and the original list has gone out of scope before
ata_scsi_port_error_handler() is called. leading to some commands
dangling on bare stack, which is a potential memory corruption issue.
Fix this by manually deleting all outstanding commands from the on-stack
list before it goes out of scope.

Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/libsas/sas_ata.c