From 66553fd7a7bcf7c260b45f2e7ad54e98d33f8080 Mon Sep 17 00:00:00 2001 From: Justus Winter Date: Sat, 5 Aug 2017 19:28:38 +0200 Subject: 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. --- proc/host.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'proc') diff --git a/proc/host.c b/proc/host.c index 25066888..7d76f7cb 100644 --- a/proc/host.c +++ b/proc/host.c @@ -113,8 +113,7 @@ S_proc_setexecdata (struct proc *p, mach_port_deallocate (mach_task_self (), std_port_array[i]); free (std_port_array); } - if (std_int_array) - free (std_int_array); + free (std_int_array); std_port_array = std_port_array_new; n_std_ports = nports; -- cgit v1.2.3