diff options
| author | Zhu Wang <wangzhu9@huawei.com> | 2023-08-01 19:14:21 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-08-16 18:10:55 +0200 |
| commit | 789275f7c0544374d40bc8d9c81f96751a41df45 (patch) | |
| tree | dd954191cdb51241f09f13a5e6c3f35330a6e855 | |
| parent | d76618133815999ff5bcf1ac166803f91c39df57 (diff) | |
| download | linux-stable-789275f7c0544374d40bc8d9c81f96751a41df45.tar.gz linux-stable-789275f7c0544374d40bc8d9c81f96751a41df45.tar.bz2 linux-stable-789275f7c0544374d40bc8d9c81f96751a41df45.zip | |
scsi: snic: Fix possible memory leak if device_add() fails
commit 41320b18a0e0dfb236dba4edb9be12dba1878156 upstream.
If device_add() returns error, the name allocated by dev_set_name() needs
be freed. As the comment of device_add() says, put_device() should be used
to give up the reference in the error path. So fix this by calling
put_device(), then the name can be freed in kobject_cleanp().
Fixes: c8806b6c9e82 ("snic: driver for Cisco SCSI HBA")
Signed-off-by: Zhu Wang <wangzhu9@huawei.com>
Acked-by: Narsimhulu Musini <nmusini@cisco.com>
Link: https://lore.kernel.org/r/20230801111421.63651-1-wangzhu9@huawei.com
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/scsi/snic/snic_disc.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/scsi/snic/snic_disc.c b/drivers/scsi/snic/snic_disc.c index 69c5e26a9d5b..388ba2ebcce5 100644 --- a/drivers/scsi/snic/snic_disc.c +++ b/drivers/scsi/snic/snic_disc.c @@ -316,6 +316,7 @@ snic_tgt_create(struct snic *snic, struct snic_tgt_id *tgtid) "Snic Tgt: device_add, with err = %d\n", ret); + put_device(&tgt->dev); put_device(&snic->shost->shost_gendev); spin_lock_irqsave(snic->shost->host_lock, flags); list_del(&tgt->list); |