diff options
author | Sami Kerola | 2013-04-08 21:32:51 +0200 |
---|---|---|
committer | Karel Zak | 2013-04-09 12:26:42 +0200 |
commit | 9e01635e326c787059e5c1b83b5fbb9dddd67275 (patch) | |
tree | 7ea0cd1a052db858538079c01f2647acfce2acd0 | |
parent | various: fix shadow declarations [smatch scan] (diff) | |
download | kernel-qcow2-util-linux-9e01635e326c787059e5c1b83b5fbb9dddd67275.tar.gz kernel-qcow2-util-linux-9e01635e326c787059e5c1b83b5fbb9dddd67275.tar.xz kernel-qcow2-util-linux-9e01635e326c787059e5c1b83b5fbb9dddd67275.zip |
libmount, col: remove redundant null checks [smatch scan]
libmount/src/context_umount.c:174 mnt_context_find_umount_fs() info: redundant null check on loopdev calling free()
libmount/src/context_umount.c:179 mnt_context_find_umount_fs() info: redundant null check on loopdev calling free()
text-utils/col.c:406 flush_lines() info: redundant null check on l->l_line calling free()
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
-rw-r--r-- | libmount/src/context_umount.c | 6 | ||||
-rw-r--r-- | text-utils/col.c | 3 |
2 files changed, 3 insertions, 6 deletions
diff --git a/libmount/src/context_umount.c b/libmount/src/context_umount.c index 96ae87ae1..b9484594b 100644 --- a/libmount/src/context_umount.c +++ b/libmount/src/context_umount.c @@ -170,13 +170,11 @@ try_loopdev: } *pfs = fs; - if (loopdev) - free(loopdev); + free(loopdev); return fs ? 0 : 1; err: - if (loopdev) - free(loopdev); + free(loopdev); return rc; } diff --git a/text-utils/col.c b/text-utils/col.c index 474b33fca..9aa6a414b 100644 --- a/text-utils/col.c +++ b/text-utils/col.c @@ -402,8 +402,7 @@ void flush_lines(int nflush) flush_line(l); } nblank_lines++; - if (l->l_line) - free((void *)l->l_line); + free((void *)l->l_line); free_line(l); } if (lines) |