tree 591115b3f18593cf52f358ac52960a6e48a50182
parent 8feffd109977c045669913f4e80e8811f2212cd9
author Julia Lawall <julia.lawall@lip6.fr> 1357505183 -0800
committer Anton Vorontsov <anton@enomsg.org> 1357505183 -0800

88pm860x_battery: Eliminate possible references to released resources

devm_kzalloc should not be followed by kfree, as this results in a double
free.  The problem was found using the following semantic match
(http://coccinelle.lip6.fr/):

// <smpl>
@@
expression x,e;
@@
x = devm_kzalloc(...)
... when != x = e
?-kfree(x,...);
// </smpl>

Furthermore, in the remove function, the calls to free_irq are moved up to
prevent a possible reference in the interrupt handler to resources freed by
power_supply_unregister.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Anton Vorontsov <anton@enomsg.org>