Prev: tree/tiny rcu: Add debug RCU head objects (v5)
Next: tracing: Dump either the oops's cpu source or all cpus buffers
From: Axel Lin on 18 Apr 2010 22:00
This patch fixes a memory leak by freeing priv in mc13783_regulator_remove

Signed-off-by: Axel Lin <axel.lin(a)gmail.com>
Cc: Sascha Hauer <s.hauer(a)pengutronix.de>
Cc: Liam Girdwood <lrg(a)slimlogic.co.uk>
Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com>
Cc: Samuel Ortiz <sameo(a)linux.intel.com>
---
drivers/regulator/mc13783-regulator.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/drivers/regulator/mc13783-regulator.c
b/drivers/regulator/mc13783-regulator.c
index a681f5e..ad036dd 100644
--- a/drivers/regulator/mc13783-regulator.c
+++ b/drivers/regulator/mc13783-regulator.c
@@ -618,9 +618,12 @@ static int __devexit
mc13783_regulator_remove(struct platform_device *pdev)
dev_get_platdata(&pdev->dev);
int i;

+ platform_set_drvdata(pdev, NULL);
+
for (i = 0; i < pdata->num_regulators; i++)
regulator_unregister(priv->regulators[i]);

+ kfree(priv);
return 0;
}

--
1.5.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: tree/tiny rcu: Add debug RCU head objects (v5)
Next: tracing: Dump either the oops's cpu source or all cpus buffers