Message ID | 20220116032211.9728-2-ansuelsmth@gmail.com |
---|---|
State | Accepted |
Headers | show |
Series | [1/2] mtd: parsers: qcom: Fix kernel panic on skipped partition | expand |
On Sun, 2022-01-16 at 03:22:11 UTC, Ansuel Smith wrote: > Mtdpart doesn't free pparts when a cleanup function is declared. > Add missing free for pparts in cleanup function for smem to fix the > leak. > > Fixes: 10f3b4d79958 ("mtd: parsers: qcom: Fix leaking of partition name") > Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Miquel
diff --git a/drivers/mtd/parsers/qcomsmempart.c b/drivers/mtd/parsers/qcomsmempart.c index f4fc7635c1f3..32ddfea70142 100644 --- a/drivers/mtd/parsers/qcomsmempart.c +++ b/drivers/mtd/parsers/qcomsmempart.c @@ -173,6 +173,8 @@ static void parse_qcomsmem_cleanup(const struct mtd_partition *pparts, for (i = 0; i < nr_parts; i++) kfree(pparts[i].name); + + kfree(pparts); } static const struct of_device_id qcomsmem_of_match_table[] = {
Mtdpart doesn't free pparts when a cleanup function is declared. Add missing free for pparts in cleanup function for smem to fix the leak. Fixes: 10f3b4d79958 ("mtd: parsers: qcom: Fix leaking of partition name") Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com> --- drivers/mtd/parsers/qcomsmempart.c | 2 ++ 1 file changed, 2 insertions(+)