From: Dan Carpenter <dan.carpen...@oracle.com>

commit b9223da41794030a5dfd5106c34ed1b98255e2ae upstream.

We accidentally free "micro_ac_power" which is an error pointer and it
leads to an oops.  We intended to free "micro_batt_power".

Fixes: a2c1d531854c ('power_supply: ipaq_micro_battery: Check return values in 
probe')
Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
Signed-off-by: Sebastian Reichel <s...@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>

---
 drivers/power/ipaq_micro_battery.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/power/ipaq_micro_battery.c
+++ b/drivers/power/ipaq_micro_battery.c
@@ -261,7 +261,7 @@ static int micro_batt_probe(struct platf
        return 0;
 
 ac_err:
-       power_supply_unregister(micro_ac_power);
+       power_supply_unregister(micro_batt_power);
 batt_err:
        cancel_delayed_work_sync(&mb->update);
        destroy_workqueue(mb->wq);


Reply via email to