Re: [PATCH] regulator: fix memory leak on error path of regulator_register()

On Fri, 24 Jul 2020 03:50:13 +0300, Vladimir Zapolskiy wrote: > The change corrects registration and deregistration on error path > of a regulator, the problem was manifested by a reported memory > leak on deferred probe: > > as3722-regulator as3722-regulator: regulator 13 register failed -517

Re: [PATCH] regulator: fix memory leak on error path of regulator_register()

> … introduced as a side ef another … Would the following wording variant be more appropriate? … introduced as a side effect of another … How do you think about to replace the wording “…, I believe …” by an imperative description? Regards, Markus

[PATCH] regulator: fix memory leak on error path of regulator_register()

The change corrects registration and deregistration on error path of a regulator, the problem was manifested by a reported memory leak on deferred probe: as3722-regulator as3722-regulator: regulator 13 register failed -517 # cat /sys/kernel/debug/kmemleak unreferenced object 0xecc4374