xfsprogs: trivial buffer frees on error paths

Lots of memory leaks on error paths etc, spotted by
coverity.  This patch rolls up the super-straightforward
fixes across xfsprogs.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>

diff --git a/db/addr.c b/db/addr.c
index f74dd628a702f442515fac526b9673e21b065147..9bea1c7a0ba31b73df75c01b5d767e93c9c7fcb4 100644 (file)
--- a/db/addr.c
+++ b/db/addr.c
@@ -85,16 +85,14 @@ addr_f(
        fl = flist_scan(argv[1]);
        if (fl == NULL)
                return 0;
-       if (!flist_parse(fld, fl, iocur_top->data, 0)) {
-               flist_free(fl);
-               return 0;
-       }
+       if (!flist_parse(fld, fl, iocur_top->data, 0))
+               goto out;
+
        flist_print(fl);
        for (tfl = fl; tfl->child != NULL; tfl = tfl->child) {
                if ((tfl->flags & FL_OKLOW) && tfl->low < tfl->high) {
                        dbprintf(_("array not allowed for addr command\n"));
-                       flist_free(fl);
-                       return 0;
+                       goto out;
                }
        }
        fld = tfl->fld;
@@ -103,7 +101,7 @@ addr_f(
                next = inode_next_type();
        if (next == TYP_NONE) {
                dbprintf(_("no next type for field %s\n"), fld->name);
-               return 0;
+               goto out;
        }
        fa = &ftattrtab[fld->ftyp];
        ASSERT(fa->ftyp == fld->ftyp);
@@ -111,9 +109,10 @@ addr_f(
        if (adf == NULL) {
                dbprintf(_("no addr function for field %s (type %s)\n"),
                        fld->name, fa->name);
-               return 0;
+               goto out;
        }
        (*adf)(iocur_top->data, tfl->offset, next);
+out:
        flist_free(fl);
        return 0;
 }

diff --git a/db/check.c b/db/check.c
index 486769866e81896c28d48576f8d97609e94a67d8..baf7f9f654ee2d384c311155484aa92126683e3b 100644 (file)
--- a/db/check.c
+++ b/db/check.c
@@ -1136,7 +1136,7 @@ blocktrash_f(
        }
        if (blocks == 0) {
                dbprintf(_("blocktrash: no matching blocks\n"));
-               return 0;
+               goto out;
        }
        if (!sopt)
                dbprintf(_("blocktrash: seed %u\n"), seed);
@@ -1161,6 +1161,7 @@ blocktrash_f(
                        }
                }
        }
+out:
        xfree(lentab);
        return 0;
 }
@@ -1907,6 +1908,7 @@ ncheck_f(
                        break;
                default:
                        dbprintf(_("bad option -%c for ncheck command\n"), c);
+                       xfree(ilist);
                        return 0;
                }
        }

diff --git a/db/write.c b/db/write.c
index 7b34fc00695c3d5489e85e18816064d0938b1aea..ca8bd0fd1543fe7d516dba9c0e4b0601b9ef0c50 100644 (file)
--- a/db/write.c
+++ b/db/write.c
@@ -233,6 +233,7 @@ bwrite_lrot(
        memcpy(hold_region, base, shift);
        memcpy(base, base+shift, len-shift);
        memcpy(base+(len-shift), hold_region, shift);
+       free(hold_region);
 }
 
 /* ARGSUSED */
@@ -265,6 +266,7 @@ bwrite_rrot(
        memcpy(hold_region, base+(len-shift), shift);
        memmove(base+shift, base, len-shift);
        memcpy(base, hold_region, shift);
+       free(hold_region);
 }
 
 /* ARGSUSED */

diff --git a/io/parent.c b/io/parent.c
index 47faaa00279f69e7d9f681cda3fe7b24675ea710..ca989e9609560c8666e735004c3e65a0c6e23728 100644 (file)
--- a/io/parent.c
+++ b/io/parent.c
@@ -258,6 +258,8 @@ parent_check(void)
        if (!bstatbuf || !parentbuf) {
                fprintf(stderr, _("unable to allocate buffers: %s\n"),
                        strerror(errno));
+               free(bstatbuf);
+               free(parentbuf);
                return 1;
        }
 

diff --git a/mkfs/proto.c b/mkfs/proto.c
index b7e07618fba73562a40a7603440cac93eb0dc057..95583c90bcb245af4933ce0f5b4211706a7d0976 100644 (file)
--- a/mkfs/proto.c
+++ b/mkfs/proto.c
@@ -49,7 +49,7 @@ char *
 setup_proto(
        char    *fname)
 {
-       char            *buf;
+       char            *buf = NULL;
        static char     dflt[] = "d--755 0 0 $";
        int             fd;
        long            size;
@@ -85,6 +85,7 @@ setup_proto(
 
 out_fail:
        close(fd);
+       free(buf);
        exit(1);
 }