diff options
author | Ulrich Drepper <drepper@redhat.com> | 1999-05-16 10:14:25 +0000 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 1999-05-16 10:14:25 +0000 |
commit | 32abdb710c9335481da780eab5ad790318d2fadf (patch) | |
tree | 70c94809f5ec43a895c614a445a32f4d18d925c5 /nis/nis_removemember.c | |
parent | 7d853c902b9f79a32c8c1921b16107648c15f0e3 (diff) | |
download | glibc-32abdb710c9335481da780eab5ad790318d2fadf.tar glibc-32abdb710c9335481da780eab5ad790318d2fadf.tar.gz glibc-32abdb710c9335481da780eab5ad790318d2fadf.tar.bz2 glibc-32abdb710c9335481da780eab5ad790318d2fadf.zip |
Update.
1999-05-16 Thorsten Kukuk <kukuk@suse.de>
* sunrpc/etc.rpc: Add portmapper, rstat_svc and pcnfs.
* sunrpc/key_call.c: Fix memory leak, close file handle.
* nis/nis_creategroup.c: Use malloc instead of calloc,
set ctime and mtime.
* nis/nis_subr.c: Check for realloc result.
* nis/nis_file.c: Fix memory leak.
* nis/nis_table.c: Pretty print.
* nis/nis_getservlist.c: Likewise.
* nis/nis_ismember.c: Likewise.
* nis/nis_lookup.c: Likewise.
* nis/nis_ping.c: Likewise.
* nis/nis_removemember.c: Likewise.
* nis/nis_util.c: Likewise, check calloc result.
Diffstat (limited to 'nis/nis_removemember.c')
-rw-r--r-- | nis/nis_removemember.c | 22 |
1 files changed, 16 insertions, 6 deletions
diff --git a/nis/nis_removemember.c b/nis/nis_removemember.c index 4fa8893a84..2efa4cf060 100644 --- a/nis/nis_removemember.c +++ b/nis/nis_removemember.c @@ -1,4 +1,4 @@ -/* Copyright (c) 1997, 1998 Free Software Foundation, Inc. +/* Copyright (c) 1997, 1998, 1999 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1997. @@ -44,22 +44,31 @@ nis_removemember (const_nis_name member, const_nis_name group) stpcpy (cp, cp2); } res = nis_lookup (buf, FOLLOW_LINKS|EXPAND_NAME); - if (NIS_RES_STATUS (res) != NIS_SUCCESS) + if (res == NULL || NIS_RES_STATUS (res) != NIS_SUCCESS) { - status = NIS_RES_STATUS (res); - nis_freeresult (res); + if (res) + { + status = NIS_RES_STATUS (res); + nis_freeresult (res); + } + else + return NIS_NOMEMORY; return status; } if ((res->objects.objects_len != 1) || (__type_of (NIS_RES_OBJECT (res)) != NIS_GROUP_OBJ)) - return NIS_INVALIDOBJ; + { + nis_freeresult (res); + return NIS_INVALIDOBJ; + } newmem = - calloc (1, NIS_RES_OBJECT(res)->GR_data.gr_members.gr_members_len * + calloc (NIS_RES_OBJECT(res)->GR_data.gr_members.gr_members_len, sizeof (char *)); if (newmem == NULL) return NIS_NOMEMORY; + k = NIS_RES_OBJECT (res)[0].GR_data.gr_members.gr_members_len; j = 0; for (i = 0; i < NIS_RES_OBJECT(res)->GR_data.gr_members.gr_members_len; @@ -81,6 +90,7 @@ nis_removemember (const_nis_name member, const_nis_name group) newmem = realloc (newmem, k * sizeof (char*)); if (newmem == NULL) return NIS_NOMEMORY; + NIS_RES_OBJECT (res)->GR_data.gr_members.gr_members_val = newmem; NIS_RES_OBJECT (res)->GR_data.gr_members.gr_members_len = k; |