From 87dd569b8846707ecbbc131e1b7997053bf7c76c Mon Sep 17 00:00:00 2001 From: Manish Katiyar Date: Wed, 3 Sep 2008 11:23:53 +0530 Subject: [PATCH] debugfs: Fix memory leak in do_ncheck()'s error path Also the progname printed as "ncheck" instead of "do_ncheck" to be consistent with the other error messages in debugfs. Signed-off-by: "Manish Katiyar" Signed-off-by: "Theodore Ts'o" --- debugfs/ncheck.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/debugfs/ncheck.c b/debugfs/ncheck.c index 3381dd9..22529c4 100644 --- a/debugfs/ncheck.c +++ b/debugfs/ncheck.c @@ -76,7 +76,7 @@ void do_ncheck(int argc, char **argv) iw.iarray = malloc(sizeof(struct inode_info) * argc); if (!iw.iarray) { - com_err("do_ncheck", ENOMEM, + com_err("ncheck", ENOMEM, "while allocating inode info array"); return; } @@ -86,7 +86,7 @@ void do_ncheck(int argc, char **argv) iw.iarray[i-1].ino = strtol(argv[i], &tmp, 0); if (*tmp) { com_err(argv[0], 0, "Bad inode - %s", argv[i]); - return; + goto error_out; } } -- 1.8.3.1