diff options
| author | Zhu Wang <[email protected]> | 2023-08-01 11:14:21 +0000 |
|---|---|---|
| committer | Martin K. Petersen <[email protected]> | 2023-08-08 01:34:08 +0000 |
| commit | 41320b18a0e0dfb236dba4edb9be12dba1878156 (patch) | |
| tree | 759e7020c2e66081966f0139e841ceddb86d3858 | |
| parent | scsi: core: Fix possible memory leak if device_add() fails (diff) | |
| download | kernel-41320b18a0e0dfb236dba4edb9be12dba1878156.tar.gz kernel-41320b18a0e0dfb236dba4edb9be12dba1878156.zip | |
scsi: snic: Fix possible memory leak if device_add() fails
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 <[email protected]>
Acked-by: Narsimhulu Musini <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Martin K. Petersen <[email protected]>
| -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 3e2e5783924d..e429ad23c396 100644 --- a/drivers/scsi/snic/snic_disc.c +++ b/drivers/scsi/snic/snic_disc.c @@ -303,6 +303,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); |