Re: [RFC v1 2/3] max17042_battery: fix potential user-after-free on module unload
From: Sebastian Reichel
Date: Tue Feb 05 2019 - 12:21:59 EST
On Tue, Feb 05, 2019 at 09:27:49AM -0500, Sven Van Asbroeck wrote:
> On Tue, Feb 5, 2019 at 3:27 AM Dmitry Torokhov
> <dmitry.torokhov@xxxxxxxxx> wrote:
> > Are there many more instances of this?
> Unfortunately I think so.
> A simple grep brings up a couple of candidates, but I'm sure there are more:
> > I am unsure if we need
> > devm_init_work() when we can easily do the same in remove() call.
> The devm_init_work() suggestion only addresses the problem for those modules
> that use devm_. The others will need fixes in remove(). But this is not as
> elegant and error-proof as using devm_init_work().
I would for sure appreciate a devm_init_work(). There are a lot of
devm_ users in power-supply and this helper would definitely
Description: PGP signature