summaryrefslogtreecommitdiff
path: root/console-client/console.c
diff options
context:
space:
mode:
authorJustus Winter <justus@gnupg.org>2017-08-05 19:28:38 +0200
committerJustus Winter <justus@gnupg.org>2017-08-05 19:34:28 +0200
commit66553fd7a7bcf7c260b45f2e7ad54e98d33f8080 (patch)
tree154a543b44a17302b6618cb9c80d7e6b66703216 /console-client/console.c
parent835b293d35a209d38047126443d41fa7090daa4c (diff)
Simplify deallocations.
free (NULL) is a nop, therefore it is not necessary to check that first. Simplify the code accordingly. This commit is the result of the following semantic patch: @@ identifier X; @@ -if (X) free (X); +free (X); * console-client/console.c: Simplify accordingly. * console-client/driver.c: Likewise. * console-client/vga.c: Likewise. * ftpfs/dir.c: Likewise. * libftpconn/unix.c: Likewise. * libps/fmt.c: Likewise. * libps/proclist.c: Likewise. * libstore/mvol.c: Likewise. * nfs/ops.c: Likewise. * proc/host.c: Likewise. * sutils/fstab.c: Likewise.
Diffstat (limited to 'console-client/console.c')
-rw-r--r--console-client/console.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/console-client/console.c b/console-client/console.c
index 5335156b..3b316115 100644
--- a/console-client/console.c
+++ b/console-client/console.c
@@ -563,8 +563,7 @@ parse_opt (int key, char *arg, struct argp_state *state)
char *s;
char *d;
- if (driver_path)
- free (driver_path);
+ free (driver_path);
driver_path = malloc (strlen (arg) + 2);
if (!driver_path)
{