Re: [PATCH 1/2] scsi: dpt_i2o: use after free in adpt_release()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 1/2] scsi: dpt_i2o: use after free in adpt_release()
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Wed, 21 Mar 2018 18:37:01 -0400
Cc: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Dan Carpenter <dan.carpenter@xxxxxxxxxx>, Adaptec OEM Raid Solutions <aacraid@xxxxxxxxxxx>, "James E.J. Bottomley" <jejb@xxxxxxxxxxxxxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx
In-reply-to: <20180320084208.GA16215@lst.de> (Christoph Hellwig's message of "Tue, 20 Mar 2018 09:42:08 +0100")
Organization: Oracle Corporation
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Christoph,

> This fix is broken!  adpt_i2o_delete_hba references pHba->host as well.
>
> Instead we need a local variable for the host.

Thanks for spotting this! Fixed it up.

-- 
Martin K. Petersen	Oracle Linux Engineering

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

References:
- [PATCH 1/2] scsi: dpt_i2o: use after free in adpt_release()
  - From: Dan Carpenter
- Re: [PATCH 1/2] scsi: dpt_i2o: use after free in adpt_release()
  - From: Martin K. Petersen
- Re: [PATCH 1/2] scsi: dpt_i2o: use after free in adpt_release()
  - From: Christoph Hellwig

Prev by Date: [trivial PATCH V2] treewide: Align function definition open/close braces
Next by Date: Re: [PATCH v4 00/13] qla2xxx: Fixes for FC-NVMe
Previous by thread: Re: [PATCH 1/2] scsi: dpt_i2o: use after free in adpt_release()
Next by thread: [PATCH 2/2] scsi: dpt_i2o: use after free in __adpt_reset()
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]