diff options
author | Sami Kerola | 2013-11-20 01:57:03 +0100 |
---|---|---|
committer | Karel Zak | 2013-12-02 10:59:20 +0100 |
commit | f614b73c0eb18b554e950df487f66ed6e13ca0fc (patch) | |
tree | d9d9dc6597b759405399d98fdbf12e8ea218820c | |
parent | libfdisk: (dos) warn on type 0 (diff) | |
download | kernel-qcow2-util-linux-f614b73c0eb18b554e950df487f66ed6e13ca0fc.tar.gz kernel-qcow2-util-linux-f614b73c0eb18b554e950df487f66ed6e13ca0fc.tar.xz kernel-qcow2-util-linux-f614b73c0eb18b554e950df487f66ed6e13ca0fc.zip |
losetup, agetty: remove unnecessary if's before free()
Reference: http://git.savannah.gnu.org/cgit/gnulib.git/tree/build-aux/useless-if-before-free
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
-rw-r--r-- | sys-utils/losetup.c | 6 | ||||
-rw-r--r-- | term-utils/agetty.c | 6 |
2 files changed, 4 insertions, 8 deletions
diff --git a/sys-utils/losetup.c b/sys-utils/losetup.c index 5be328848..70d9155c0 100644 --- a/sys-utils/losetup.c +++ b/sys-utils/losetup.c @@ -184,8 +184,7 @@ static int show_all_loops(struct loopdev_cxt *lc, const char *file, printf_loopdev(lc); } loopcxt_deinit_iterator(lc); - if (cn_file) - free(cn_file); + free(cn_file); return 0; } @@ -351,8 +350,7 @@ static int make_table(struct loopdev_cxt *lc, } loopcxt_deinit_iterator(lc); - if (cn_file) - free(cn_file); + free(cn_file); return 0; } diff --git a/term-utils/agetty.c b/term-utils/agetty.c index 113f8e79d..b162e9f75 100644 --- a/term-utils/agetty.c +++ b/term-utils/agetty.c @@ -466,8 +466,7 @@ int main(int argc, char **argv) log_warn(_("%s: can't change process priority: %m"), options.tty); } - if (options.osrelease) - free(options.osrelease); + free(options.osrelease); #ifdef DEBUGGING fprintf(dbf, "read %c\n", ch); if (close_stream(dbf) != 0) @@ -1383,8 +1382,7 @@ static char *read_os_release(struct options *op, const char *varname) } break; } - if (ret) - free(ret); + free(ret); ret = strdup(p); if (!ret) log_err(_("failed to allocate memory: %m")); |