diff options
| author | Sami Kerola | 2017-02-12 21:42:49 +0100 |
|---|---|---|
| committer | Karel Zak | 2017-02-20 13:00:58 +0100 |
| commit | 79b915e8df2eaa060e157ed51b8c70fed177d34d (patch) | |
| tree | 31683601e8ea903432a069b0598897d06032ef8c /misc-utils/findmnt-verify.c | |
| parent | libblkid: declare across file variables in blkidP.h [smatch scan] (diff) | |
| download | kernel-qcow2-util-linux-79b915e8df2eaa060e157ed51b8c70fed177d34d.tar.gz kernel-qcow2-util-linux-79b915e8df2eaa060e157ed51b8c70fed177d34d.tar.xz kernel-qcow2-util-linux-79b915e8df2eaa060e157ed51b8c70fed177d34d.zip | |
findmnt: fix couple memory leaks [cppcheck]
In both cases FILE *f was leaked.
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Diffstat (limited to 'misc-utils/findmnt-verify.c')
| -rw-r--r-- | misc-utils/findmnt-verify.c | 10 |
1 files 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) |