20 Mar
2013
20 Mar
'13
11:05 a.m.
On Sat, Mar 16, 2013 at 01:45:34PM +0200, Silviu-Mihai Popescu wrote:
The objects allocated by devm_* APIs are managed by devres and are freed when the device is detached. Hence there is no need to use kfree() explicitly.
In this case since we're in error handling paths which might not be fatal (though I suspect they probably should be normally) it looks like the better fix is to use devm_kfree() to free immediately. Otherwise the unused memory will end up hanging around for as long as the device is bound.