Commit db7157d4 authored by Joe Lawrence's avatar Joe Lawrence Committed by Christoph Hellwig

qla2xxx: Fix shost use-after-free on device removal

Once calling scsi_host_put, be careful to not access qla_hw_data through
the Scsi_Host private data (ie, scsi_qla_host base_vha).

Fixes: fe1b806f ("qla2xxx: Refactor shutdown code so some functionality can be reused")
Cc: stable@vger.kernel.org # 3.14, 3.15, 3.16
Signed-off-by: default avatarJoe Lawrence <joe.lawrence@stratus.com>
Acked-by: default avatarChad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent c01a8bc0
...@@ -3119,10 +3119,8 @@ qla2x00_unmap_iobases(struct qla_hw_data *ha) ...@@ -3119,10 +3119,8 @@ qla2x00_unmap_iobases(struct qla_hw_data *ha)
} }
static void static void
qla2x00_clear_drv_active(scsi_qla_host_t *vha) qla2x00_clear_drv_active(struct qla_hw_data *ha)
{ {
struct qla_hw_data *ha = vha->hw;
if (IS_QLA8044(ha)) { if (IS_QLA8044(ha)) {
qla8044_idc_lock(ha); qla8044_idc_lock(ha);
qla8044_clear_drv_active(ha); qla8044_clear_drv_active(ha);
...@@ -3193,7 +3191,7 @@ qla2x00_remove_one(struct pci_dev *pdev) ...@@ -3193,7 +3191,7 @@ qla2x00_remove_one(struct pci_dev *pdev)
scsi_host_put(base_vha->host); scsi_host_put(base_vha->host);
qla2x00_clear_drv_active(base_vha); qla2x00_clear_drv_active(ha);
qla2x00_unmap_iobases(ha); qla2x00_unmap_iobases(ha);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment