Re: [PATCH] dax: use devm_add_action_or_reset()
Thanks, applied.
On Mon, Jul 4, 2016 at 11:59 PM, Johannes Thumshirn <jthumshirn(a)suse.de> wrote:
On Tue, Jul 05, 2016 at 11:20:07AM +0530, Vikas C Sajjan wrote:
> If devm_add_action() fails, we are explicitly calling the cleanup to free
> the resources allocated. Use the helper devm_add_action_or_reset()
> and return directly in case of error, since the cleanup function
> has been already called by the helper if there was any error.
>
> Reported-by: Sudip Mukherjee <sudipm.mukherjee(a)gmail.com>
> Signed-off-by: Vikas C Sajjan <vikas.cha.sajjan(a)hpe.com>
Reviewed-by: Johannes Thumshirn <jthumshirn(a)suse.de>
--
Johannes Thumshirn Storage
jthumshirn(a)suse.de +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850