From a4ba8cb5d1a040d3dea01a404d49e2bee012a6b5 Mon Sep 17 00:00:00 2001 From: Baptiste Daroussin Date: Fri, 28 Dec 2012 20:21:14 +0000 Subject: malloc() sets errno to ENOMEM already. Submitted by: Christoph Mallon --- libutil/gr_util.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'libutil') diff --git a/libutil/gr_util.c b/libutil/gr_util.c index 759e6e8..6676720 100644 --- a/libutil/gr_util.c +++ b/libutil/gr_util.c @@ -505,17 +505,13 @@ gr_add(struct group *gr, char *newmember) } /* Allocate enough for current pointers + 1 more and NULL marker */ mlen = (num_mem + 2) * sizeof(*gr->gr_mem); - if ((members = malloc(mlen)) == NULL) { + if ((members = malloc(mlen)) == NULL) errno = ENOMEM; - return (NULL); - } memcpy(members, gr->gr_mem, num_mem * sizeof(*gr->gr_mem)); members[num_mem++] = newmember; members[num_mem] = NULL; gr->gr_mem = members; newgr = gr_dup(gr); - if (newgr == NULL) - errno = ENOMEM; free(members); return (newgr); } -- cgit v1.2.3-56-ge451