In both cases FILE *f was leaked. Signed-off-by: Sami Kerola <kerolasa@xxxxxx> --- misc-utils/findmnt-verify.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/misc-utils/findmnt-verify.c b/misc-utils/findmnt-verify.c index b2a88c5f6..e3fb391d4 100644 --- a/misc-utils/findmnt-verify.c +++ b/misc-utils/findmnt-verify.c @@ -328,14 +328,15 @@ static int read_proc_filesystems(struct verify_context *vfy) rc = add_filesystem(vfy, cp); if (rc) - return rc; + break; } fclose(f); - return 0; + return rc; } static int read_kernel_filesystems(struct verify_context *vfy) { + int rc = 0; #ifdef __linux__ struct utsname uts; FILE *f; @@ -351,7 +352,6 @@ static int read_kernel_filesystems(struct verify_context *vfy) while (!feof(f)) { char *p, *name; - int rc; if (!fgets(buf, sizeof(buf), f)) break; @@ -377,11 +377,11 @@ static int read_kernel_filesystems(struct verify_context *vfy) rc = add_filesystem(vfy, name); if (rc) - return rc; + break; } fclose(f); #endif /* __linux__ */ - return 0; + return rc; } static int verify_fstype(struct verify_context *vfy) -- 2.11.1 -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html