From b569ae384410aeff8bf3252eca64df1686bcd70a Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Mon, 29 Mar 2004 21:46:58 +0000 Subject: Update. * nis/nss_nis/nis-rpc.c (saveit): Improve memory. No need to allocate multiple blocks. (internal_nis_endrpcent): Adjust accordingly. * nis/nss_nis/nis-proto.c (saveit): Improve memory. No need to allocate multiple blocks. (internal_nis_endprotoent): Adjust accordingly. * nis/nss_nis/nis-initgroups.c (saveit): Improve memory. No need to allocate multiple blocks. (_nss_nis_initgroups_dyn): Adjust accordingly. * nis/nss_nis/nis-ethers.c (saveit): Improve memory. No need to allocate multiple blocks. (internal_nis_endetherent): Adjust accordingly. * nis/nss_nis/nis-service.c (saveit): Improve memory. No need to allocate multiple blocks. (internal_nis_endservent): Adjust accordingly. --- nis/nss_nis/nis-ethers.c | 34 ++++++++++++---------------------- nis/nss_nis/nis-initgroups.c | 35 +++++++++++++---------------------- nis/nss_nis/nis-proto.c | 34 ++++++++++++---------------------- nis/nss_nis/nis-rpc.c | 35 +++++++++++++---------------------- nis/nss_nis/nis-service.c | 36 +++++++++++++----------------------- 5 files changed, 63 insertions(+), 111 deletions(-) (limited to 'nis') diff --git a/nis/nss_nis/nis-ethers.c b/nis/nss_nis/nis-ethers.c index 713fe31..328d61f 100644 --- a/nis/nss_nis/nis-ethers.c +++ b/nis/nss_nis/nis-ethers.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1996-2000, 2001, 2002, 2003 Free Software Foundation, Inc. +/* Copyright (C) 1996-2003, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Thorsten Kukuk , 1996. @@ -40,8 +40,8 @@ __libc_lock_define_initialized (static, lock) struct response { - char *val; struct response *next; + char val[0]; }; static struct response *start; @@ -56,26 +56,18 @@ saveit (int instatus, char *inkey, int inkeylen, char *inval, if (inkey && inkeylen > 0 && inval && invallen > 0) { + struct response *newp = malloc (sizeof (struct response) + invallen + 1); + if (newp == NULL) + return YP_FALSE; /* We have no error code for out of memory */ + if (start == NULL) - { - start = malloc (sizeof (struct response)); - if (start == NULL) - return YP_FALSE; - next = start; - } + start = newp; else - { - next->next = malloc (sizeof (struct response)); - if (next->next == NULL) - return YP_FALSE; - next = next->next; - } - next->next = NULL; - next->val = malloc (invallen + 1); - if (next->val == NULL) - return YP_FALSE; - strncpy (next->val, inval, invallen); - next->val[invallen] = '\0'; + next->next = newp; + next = newp; + + newp->next = NULL; + *((char *) mempcpy (newp->val, inval, invallen)) = '\0'; } return 0; @@ -86,8 +78,6 @@ internal_nis_endetherent (void) { while (start != NULL) { - if (start->val != NULL) - free (start->val); next = start; start = start->next; free (next); diff --git a/nis/nss_nis/nis-initgroups.c b/nis/nss_nis/nis-initgroups.c index e3a8f57..4c97988 100644 --- a/nis/nss_nis/nis-initgroups.c +++ b/nis/nss_nis/nis-initgroups.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1998, 1999, 2000, 2002, 2003 Free Software Foundation, Inc. +/* Copyright (C) 1998-2000, 2002, 2003, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Thorsten Kukuk , 1998. @@ -38,8 +38,8 @@ struct response_t { - char *val; struct response_t *next; + char val[0]; }; struct intern_t @@ -60,26 +60,19 @@ saveit (int instatus, char *inkey, int inkeylen, char *inval, if (inkey && inkeylen > 0 && inval && invallen > 0) { + struct response_t *newp = malloc (sizeof (struct response_t) + + invallen + 1); + if (newp == NULL) + return YP_FALSE; /* We have no error code for out of memory */ + if (intern->start == NULL) - { - intern->start = malloc (sizeof (struct response_t)); - if (intern->start == NULL) - return YP_FALSE; - intern->next = intern->start; - } + intern->start = newp; else - { - intern->next->next = malloc (sizeof (struct response_t)); - if (intern->next->next == NULL) - return YP_FALSE; - intern->next = intern->next->next; - } - intern->next->next = NULL; - intern->next->val = malloc (invallen + 1); - if (intern->next->val == NULL) - return YP_FALSE; - strncpy (intern->next->val, inval, invallen); - intern->next->val[invallen] = '\0'; + intern->next->next = newp; + intern->next = newp; + + newp->next = NULL; + *((char *) mempcpy (newp->val, inval, invallen)) = '\0'; } return 0; @@ -209,8 +202,6 @@ _nss_nis_initgroups_dyn (const char *user, gid_t group, long int *start, done: while (intern.start != NULL) { - if (intern.start->val != NULL) - free (intern.start->val); intern.next = intern.start; intern.start = intern.start->next; free (intern.next); diff --git a/nis/nss_nis/nis-proto.c b/nis/nss_nis/nis-proto.c index b8c234e..7c1ce32 100644 --- a/nis/nss_nis/nis-proto.c +++ b/nis/nss_nis/nis-proto.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1996-1998, 2000-2002, 2003 Free Software Foundation, Inc. +/* Copyright (C) 1996-1998, 2000-2003, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Thorsten Kukuk , 1996. @@ -37,8 +37,8 @@ __libc_lock_define_initialized (static, lock) struct response { - char *val; struct response *next; + char val[0]; }; static struct response *start; @@ -53,26 +53,18 @@ saveit (int instatus, char *inkey, int inkeylen, char *inval, if (inkey && inkeylen > 0 && inval && invallen > 0) { + struct response *newp = malloc (sizeof (struct response) + invallen + 1); + if (newp == NULL) + return YP_FALSE; /* We have no error code for out of memory */ + if (start == NULL) - { - start = malloc (sizeof (struct response)); - if (start == NULL) - return YP_FALSE; - next = start; - } + start = newp; else - { - next->next = malloc (sizeof (struct response)); - if (next->next == NULL) - return YP_FALSE; - next = next->next; - } - next->next = NULL; - next->val = malloc (invallen + 1); - if (next->val == NULL) - return YP_FALSE; - strncpy (next->val, inval, invallen); - next->val[invallen] = '\0'; + next->next = newp; + next = newp; + + newp->next = NULL; + *((char *) mempcpy (newp->val, inval, invallen)) = '\0'; } return 0; @@ -83,8 +75,6 @@ internal_nis_endprotoent (void) { while (start != NULL) { - if (start->val != NULL) - free (start->val); next = start; start = start->next; free (next); diff --git a/nis/nss_nis/nis-rpc.c b/nis/nss_nis/nis-rpc.c index 6e707b9..e4a3954 100644 --- a/nis/nss_nis/nis-rpc.c +++ b/nis/nss_nis/nis-rpc.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1996-1998, 2000, 2002, 2003 Free Software Foundation, Inc. +/* Copyright (C) 1996-1998,2000,2002,2003,2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Thorsten Kukuk , 1996. @@ -37,8 +37,8 @@ __libc_lock_define_initialized (static, lock) struct response_t { - char *val; struct response_t *next; + char val[0]; }; struct intern_t @@ -61,26 +61,19 @@ saveit (int instatus, char *inkey, int inkeylen, char *inval, if (inkey && inkeylen > 0 && inval && invallen > 0) { + struct response_t *newp = malloc (sizeof (struct response_t) + + invallen + 1); + if (newp == NULL) + return YP_FALSE; /* We have no error code for out of memory */ + if (intern->start == NULL) - { - intern->start = malloc (sizeof (struct response_t)); - if (intern->start == NULL) - return YP_FALSE; - intern->next = intern->start; - } + intern->start = newp; else - { - intern->next->next = malloc (sizeof (struct response_t)); - if (intern->next->next == NULL) - return YP_FALSE; - intern->next = intern->next->next; - } - intern->next->next = NULL; - intern->next->val = malloc (invallen + 1); - if (intern->next->val == NULL) - return YP_FALSE; - strncpy (intern->next->val, inval, invallen); - intern->next->val[invallen] = '\0'; + intern->next->next = newp; + intern->next = newp; + + newp->next = NULL; + *((char *) mempcpy (newp->val, inval, invallen)) = '\0'; } return 0; @@ -91,8 +84,6 @@ internal_nis_endrpcent (intern_t *intern) { while (intern->start != NULL) { - if (intern->start->val != NULL) - free (intern->start->val); intern->next = intern->start; intern->start = intern->start->next; free (intern->next); diff --git a/nis/nss_nis/nis-service.c b/nis/nss_nis/nis-service.c index c25422f..9af9fd2 100644 --- a/nis/nss_nis/nis-service.c +++ b/nis/nss_nis/nis-service.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1996-2001, 2002, 2003 Free Software Foundation, Inc. +/* Copyright (C) 1996-2001, 2002, 2003, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Thorsten Kukuk , 1996. @@ -38,8 +38,8 @@ __libc_lock_define_initialized (static, lock) struct response_t { - char *val; struct response_t *next; + char val[0]; }; struct intern_t @@ -62,26 +62,19 @@ saveit (int instatus, char *inkey, int inkeylen, char *inval, if (inkey && inkeylen > 0 && inval && invallen > 0) { + struct response_t *newp = malloc (sizeof (struct response_t) + + invallen + 1); + if (newp == NULL) + return YP_FALSE; /* We have no error code for out of memory */ + if (intern->start == NULL) - { - intern->start = malloc (sizeof (struct response_t)); - if (intern->start == NULL) - return YP_FALSE; /* We have no error code for out of memory */ - intern->next = intern->start; - } + intern->start = newp; else - { - intern->next->next = malloc (sizeof (struct response_t)); - if (intern->next->next == NULL) - return YP_FALSE; /* We have no error code for out of memory */ - intern->next = intern->next->next; - } - intern->next->next = NULL; - intern->next->val = malloc (invallen + 1); - if (intern->next->val == NULL) - return YP_FALSE; /* We have no error code for out of memory */ - strncpy (intern->next->val, inval, invallen); - intern->next->val[invallen] = '\0'; + intern->next->next = newp; + intern->next = newp; + + newp->next = NULL; + *((char *) mempcpy (newp->val, inval, invallen)) = '\0'; } return 0; @@ -92,13 +85,10 @@ internal_nis_endservent (intern_t * intern) { while (intern->start != NULL) { - if (intern->start->val != NULL) - free (intern->start->val); intern->next = intern->start; intern->start = intern->start->next; free (intern->next); } - intern->start = NULL; return NSS_STATUS_SUCCESS; } -- cgit v1.1