From 88917b0f143e2eac468aea14338b4d9eddba389d Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Tue, 20 May 2008 20:39:38 +0000 Subject: replace "if (p) free(p)" by just "free(p)" - free(NULL) is safe. --- libc/sysdeps/linux/common/getgroups.c | 3 +-- libc/sysdeps/linux/common/setgroups.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) (limited to 'libc/sysdeps/linux/common') diff --git a/libc/sysdeps/linux/common/getgroups.c b/libc/sysdeps/linux/common/getgroups.c index b2f34ed2f..10da03ddd 100644 --- a/libc/sysdeps/linux/common/getgroups.c +++ b/libc/sysdeps/linux/common/getgroups.c @@ -53,8 +53,7 @@ ret_error: } } - if (kernel_groups) - free(kernel_groups); + free(kernel_groups); return ngids; } } diff --git a/libc/sysdeps/linux/common/setgroups.c b/libc/sysdeps/linux/common/setgroups.c index 7bfcc6c20..eb5245d59 100644 --- a/libc/sysdeps/linux/common/setgroups.c +++ b/libc/sysdeps/linux/common/setgroups.c @@ -56,8 +56,7 @@ ret_error: } i = __syscall_setgroups(size, kernel_groups); - if (kernel_groups) - free(kernel_groups); + free(kernel_groups); return i; } } -- cgit v1.2.3