diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2020-02-28 10:25:54 +0100 |
|---|---|---|
| committer | Miquel Raynal <miquel.raynal@bootlin.com> | 2020-03-11 14:49:30 +0100 |
| commit | 4da0ea71ea934af18db4c63396ba2af1a679ef02 (patch) | |
| tree | 8a186d9293e335cd6f25240e30a46d96e37aa0c2 | |
| parent | MAINTAINERS: Add staging branch for HyperBus (diff) | |
| download | linux-4da0ea71ea934af18db4c63396ba2af1a679ef02.tar.xz linux-4da0ea71ea934af18db4c63396ba2af1a679ef02.zip | |
mtd: lpddr: Fix a double free in probe()
This function is only called from lpddr_probe(). We free "lpddr" both
here and in the caller, so it's a double free. The best place to free
"lpddr" is in lpddr_probe() so let's delete this one.
Fixes: 8dc004395d5e ("[MTD] LPDDR qinfo probing.")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20200228092554.o57igp3nqhyvf66t@kili.mountain
| -rw-r--r-- | drivers/mtd/lpddr/lpddr_cmds.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/mtd/lpddr/lpddr_cmds.c b/drivers/mtd/lpddr/lpddr_cmds.c index 1efc643c9871..9341a8a592e8 100644 --- a/drivers/mtd/lpddr/lpddr_cmds.c +++ b/drivers/mtd/lpddr/lpddr_cmds.c @@ -68,7 +68,6 @@ struct mtd_info *lpddr_cmdset(struct map_info *map) shared = kmalloc_array(lpddr->numchips, sizeof(struct flchip_shared), GFP_KERNEL); if (!shared) { - kfree(lpddr); kfree(mtd); return NULL; } |