On Fri, Feb 24, 2017 at 01:37:30PM +0100, Stefan Wahren wrote:

Hi Christophe,

Am 19.02.2017 um 11:34 schrieb Christophe JAILLET:

quoted

If 'kzalloc()' fails, we should release resources allocated so far, just as
done in all other cases in this function.

Signed-off-by: Christophe JAILLET <redacted>
---
Not sure that the error handling path is correct.
Is 'gdev[0]' freed? Should it be?

Yes, but I already sent a patch to fix this and your leak as well and
Greg merged it.

sorry, didn't checked your patch yet.

It takes like 30 seconds to review this patch.  Do you use mutt?  I have
a macro that applies patches and loads vim at the right line.

regards,
dan carpenter