diff options
author | Dan Carpenter <error27@gmail.com> | 2009-11-16 11:05:03 +0200 |
---|---|---|
committer | Liam Girdwood <lrg@slimlogic.co.uk> | 2009-11-16 12:40:25 +0000 |
commit | 80099c707e0915ac3159ddc70fb714adbb3d53e3 (patch) | |
tree | 621aefafce62e64ab79a999b9cd4bebe9ef47e6d /drivers | |
parent | 9a767d43fbddb3319dca568df49c48e0d1bb6bd8 (diff) | |
download | op-kernel-dev-80099c707e0915ac3159ddc70fb714adbb3d53e3.zip op-kernel-dev-80099c707e0915ac3159ddc70fb714adbb3d53e3.tar.gz |
dereferencing freed memory regulator_fixed_voltage_remove()
Don't dereference drvdata after it has been freed.
regards,
dan carpenter
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/regulator/fixed.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c index f8b2957..f9f516a 100644 --- a/drivers/regulator/fixed.c +++ b/drivers/regulator/fixed.c @@ -196,11 +196,10 @@ static int regulator_fixed_voltage_remove(struct platform_device *pdev) struct fixed_voltage_data *drvdata = platform_get_drvdata(pdev); regulator_unregister(drvdata->dev); - kfree(drvdata->desc.name); - kfree(drvdata); - if (gpio_is_valid(drvdata->gpio)) gpio_free(drvdata->gpio); + kfree(drvdata->desc.name); + kfree(drvdata); return 0; } |