On Mon, Jan 19, 2015 at 11:02:00PM +0100, Christophe Jaillet wrote: > Free allocated page in case of error returned by hmcdrv_ftp_startup. > > Signed-off-by: Christophe Jaillet <[email protected]>
Thanks, I applied the version below. >From e38409df9fdbe8b4667cc2ee0e44ff5825ed7049 Mon Sep 17 00:00:00 2001 From: Christophe Jaillet <[email protected]> Date: Mon, 19 Jan 2015 23:02:00 +0100 Subject: [PATCH] s390/hmcdrv: free memory on error path Free allocated page in case of error returned by hmcdrv_ftp_startup. [[email protected]]: slightly changed Christophe's patch Signed-off-by: Christophe Jaillet <[email protected]> Signed-off-by: Heiko Carstens <[email protected]> --- drivers/s390/char/hmcdrv_ftp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/s390/char/hmcdrv_ftp.c b/drivers/s390/char/hmcdrv_ftp.c index 4bd63322fc29..d4b61d9088fb 100644 --- a/drivers/s390/char/hmcdrv_ftp.c +++ b/drivers/s390/char/hmcdrv_ftp.c @@ -200,10 +200,9 @@ int hmcdrv_ftp_probe(void) rc = hmcdrv_ftp_startup(); if (rc) - return rc; + goto out; rc = hmcdrv_ftp_do(&ftp); - free_page((unsigned long) ftp.buf); hmcdrv_ftp_shutdown(); switch (rc) { @@ -216,7 +215,8 @@ int hmcdrv_ftp_probe(void) rc = 0; /* clear length (success) */ break; } /* switch */ - +out: + free_page((unsigned long) ftp.buf); return rc; } EXPORT_SYMBOL(hmcdrv_ftp_probe); -- 2.1.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
