This is the mail archive of the libc-alpha@sources.redhat.com mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

A patch to reduce the size.


This patch turns nss/digits_dots.c into a function to reduce the size.

-- 
H.J. Lu (hjl@valinux.com)
---
2000-12-09  H.J. Lu  <hjl@gnu.org>

	* nss/Makefile (routines): Add digits_dots.

	* nss/Versions (libc): Add __nss_hostname_digits_dots to
	GLIBC_2.2.1.

	* nss/digits_dots.c (__nss_hostname_digits_dots): Turn template
	into a function.

	* nss/nsswitch.h (__nss_hostname_digits_dots): New internal NSS
	function.

	* nss/getXXbyYY.c (H_ERRNO_VAR_P): New.
	(TYPE_VAR_P): Likewise.
	(FLAGS_VAR): Likewise.
	(AF_VAR_P): Likewise.
	(INTERNAL (REENTRANT_NAME)): Call __nss_hostname_digits_dots ()
	instead of including digits_dots.c.

	* nss/getXXbyYY_r.c (H_ERRNO_VAR_P): New.
	(TYPE_VAR_P): Likewise.
	(FLAGS_VAR): Likewise.
	(AF_VAR_P): Likewise.
	(INTERNAL (REENTRANT_NAME)): Call __nss_hostname_digits_dots ()
	instead of including digits_dots.c.

Index: nss/Makefile
===================================================================
RCS file: /work/cvs/gnu/glibc/nss/Makefile,v
retrieving revision 1.1.1.3
diff -u -p -r1.1.1.3 Makefile
--- nss/Makefile	2000/12/09 02:19:24	1.1.1.3
+++ nss/Makefile	2000/12/09 23:02:21
@@ -27,7 +27,7 @@ distribute		:= nsswitch.h XXX-lookup.c g
 			   nsswitch.conf digits_dots.c function.def
 
 # This is the trivial part which goes into libc itself.
-routines		= nsswitch getnssent getnssent_r \
+routines		= nsswitch getnssent getnssent_r digits_dots \
 			  $(addsuffix -lookup,$(databases))
 
 # These are the databases that go through nss dispatch.
Index: nss/Versions
===================================================================
RCS file: /work/cvs/gnu/glibc/nss/Versions,v
retrieving revision 1.1.1.3
diff -u -p -r1.1.1.3 Versions
--- nss/Versions	2000/11/23 17:38:00	1.1.1.3
+++ nss/Versions	2000/12/09 23:03:33
@@ -5,6 +5,9 @@ libc {
     _nss_files_parse_grent; _nss_files_parse_pwent; _nss_files_parse_spent;
     __nss_database_lookup; __nss_configure_lookup;
   }
+  GLIBC_2.2.1 {
+    __nss_hostname_digits_dots;
+  }
 }
 
 libnss_files {
Index: nss/digits_dots.c
===================================================================
RCS file: /work/cvs/gnu/glibc/nss/digits_dots.c,v
retrieving revision 1.1.1.4
diff -u -p -r1.1.1.4 digits_dots.c
--- nss/digits_dots.c	2000/11/07 01:28:56	1.1.1.4
+++ nss/digits_dots.c	2000/12/09 23:03:33
@@ -17,10 +17,41 @@
    write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
    Boston, MA 02111-1307, USA.  */
 
+#include <assert.h>
+#include <errno.h>
+#include <string.h>
+#include <stdlib.h>
+#include <ctype.h>
+#include <wctype.h>
+#include <resolv.h>
+#include <netdb.h>
+#include <arpa/inet.h>
+#include "nsswitch.h"
+
 #ifdef USE_NSCD
 # define inet_aton __inet_aton
+# include <nscd/nscd_proto.h>
 #endif
 
+int
+__nss_hostname_digits_dots (const char *name, struct hostent *resbuf,
+			    char **buffer, size_t *buffer_size,
+			    size_t buflen, struct hostent **result,
+			    enum nss_status *status, int *typep,
+			    int flags, int *afp, int *h_errnop)
+{
+  int save;
+
+  /* We have to test for the use of IPv6 which can only be done by 
+     examining `_res'.  */
+  if ((_res.options & RES_INIT) == 0 && __res_ninit (&_res) == -1)
+    {
+      if (h_errnop)
+	*h_errnop = NETDB_INTERNAL;
+      *result = NULL; 
+      return -1;
+    }
+
   /*
    * disallow names consisting only of digits/dots, unless
    * they end in a dot.
@@ -36,14 +67,15 @@
       char **h_alias_ptr;
       size_t size_needed;
       int addr_size;
-#ifdef HAVE_TYPE
-      int af = type;
-#else
-# ifndef HAVE_AF
-      int af = -1;
-# endif
-#endif
+      int af;
 
+      if (typep)
+	af = *typep;
+      else if (afp)
+	af = *afp;
+      else
+	af = -1;
+
       switch (af)
 	{
 	case AF_INET:
@@ -55,56 +87,57 @@
 	  break;
 
 	default:
-#ifdef HAVE_TYPE
-	  /* This must not happen.  */
-	  *h_errnop = HOST_NOT_FOUND;
-	  goto done;
-#else
-	  af = (_res.options & RES_USE_INET6) ? AF_INET6 : AF_INET;
-	  addr_size = af == AF_INET6 ? IN6ADDRSZ : INADDRSZ;
+	  if (typep)
+	    {
+	      /* This must not happen.  */
+	      *h_errnop = HOST_NOT_FOUND;
+	      goto done;
+	    }
+	  else
+	    {
+	      af = (_res.options & RES_USE_INET6) ? AF_INET6 : AF_INET;
+	      addr_size = af == AF_INET6 ? IN6ADDRSZ : INADDRSZ;
+	    }
 	  break;
-#endif
 	}
 
       size_needed = (sizeof (*host_addr)
 		     + sizeof (*h_addr_ptrs) + strlen (name) + 1);
 
-#ifdef HAVE_LOOKUP_BUFFER
-      if (buflen < size_needed)
-	{
-# ifdef NEED_H_ERRNO
-	  *h_errnop = TRY_AGAIN;
-# endif
-	  __set_errno (ERANGE);
-	  goto done;
+      if (buffer_size == NULL)
+        {
+	  if (buflen < size_needed)
+	    {
+	      if (h_errnop)
+		*h_errnop = TRY_AGAIN;
+	      __set_errno (ERANGE);
+	      goto done;
+	    }
 	}
-#else
-      if (buffer_size < size_needed)
+      else if (buffer_size != NULL && *buffer_size < size_needed)
 	{
 	  char *new_buf;
-	  buffer_size = size_needed;
-	  new_buf = realloc (buffer, buffer_size);
+	  *buffer_size = size_needed;
+	  new_buf = realloc (*buffer, *buffer_size);
 
 	  if (new_buf == NULL)
 	    {
 	      save = errno;
-	      free (buffer);
-	      buffer = NULL;
-	      buffer_size = 0;
+	      free (*buffer);
+	      *buffer = NULL;
+	      *buffer_size = 0;
 	      __set_errno (save);
-# ifdef NEED_H_ERRNO
-	      *h_errnop = TRY_AGAIN;
-# endif
-	      result = (struct hostent *) NULL;
+	      if (h_errnop)
+		*h_errnop = TRY_AGAIN;
+	      *result = (struct hostent *) NULL;
 	      goto done;
 	    }
-	  buffer = new_buf;
+	  *buffer = new_buf;
 	}
-#endif /* HAVE_LOOKUP_BUFFER */
 
-      memset (buffer, 0, size_needed);
+      memset (*buffer, 0, size_needed);
 
-      host_addr = (host_addr_t *) buffer;
+      host_addr = (host_addr_t *) *buffer;
       h_addr_ptrs = (host_addr_list_t *)
 	((char *) host_addr + sizeof (*host_addr));
       h_alias_ptr = (char **) ((char *) h_addr_ptrs + sizeof (*h_addr_ptrs));
@@ -135,37 +168,30 @@
 		  if (! ok)
 		    {
 		      *h_errnop = HOST_NOT_FOUND;
-#ifndef HAVE_LOOKUP_BUFFER
-		      result = (struct hostent *) NULL;
-#endif
+		      if (buffer_size)
+			*result = (struct hostent *) NULL;
 		      goto done;
 		    }
 
-		  resbuf.h_name = strcpy (hostname, name);
+		  resbuf->h_name = strcpy (hostname, name);
 		  h_alias_ptr[0] = NULL;
-		  resbuf.h_aliases = h_alias_ptr;
+		  resbuf->h_aliases = h_alias_ptr;
 		  (*h_addr_ptrs)[0] = (char *)host_addr;
 		  (*h_addr_ptrs)[1] = (char *)0;
-		  resbuf.h_addr_list = *h_addr_ptrs;
-		  if (
-#ifdef HAVE_TYPE
-		      type == AF_INET6
-#else
-		      af == AF_INET && (_res.options & RES_USE_INET6)
-#endif
-		      )
+		  resbuf->h_addr_list = *h_addr_ptrs;
+		  if ((typep != NULL && *typep == AF_INET6)
+		      || (af == AF_INET
+			  && (_res.options & RES_USE_INET6)))
 		    {
-#ifdef HAVE_TYPE
-		      if ((flags & AI_V4MAPPED) == 0)
+		      if (typep != NULL && (flags & AI_V4MAPPED) == 0)
 			{
 			  /* That's bad.  The user hasn't specified that she
 			     allows IPv4 numeric addresses.  */
-			  result = NULL;
+			  *result = NULL;
 			  *h_errnop = HOST_NOT_FOUND;
 			  goto done;
 			}
 		      else
-#endif
 			{
 			  /* We need to change the IP v4 address into the
 			     IP v6 address.  */
@@ -181,21 +207,20 @@
 			  *p++ = 0xff;
 			  /* Copy the IP v4 address. */
 			  memcpy (p, tmp, INADDRSZ);
-			  resbuf.h_addrtype = AF_INET6;
-			  resbuf.h_length = IN6ADDRSZ;
+			  resbuf->h_addrtype = AF_INET6;
+			  resbuf->h_length = IN6ADDRSZ;
 			}
 		    }
 		  else
 		    {
-		      resbuf.h_addrtype = af;
-		      resbuf.h_length = addr_size;
+		      resbuf->h_addrtype = af;
+		      resbuf->h_length = addr_size;
 		    }
 		  *h_errnop = NETDB_SUCCESS;
-#ifdef HAVE_LOOKUP_BUFFER
-		  status = NSS_STATUS_SUCCESS;
-#else
-		  result = &resbuf;
-#endif
+		  if (buffer_size == NULL)
+		    *status = NSS_STATUS_SUCCESS;
+		  else
+		   *result = resbuf;
 		  goto done;
 		}
 
@@ -214,13 +239,14 @@
 	  host_addr_list_t *h_addr_ptrs;
 	  size_t size_needed;
 	  int addr_size;
-#ifdef HAVE_TYPE
-	  int af = type;
-#else
-# ifndef HAVE_AF
-	  int af = -1;
-# endif
-#endif
+	  int af;
+
+	  if (typep)
+	    af = *typep;
+	  else if (afp)
+	    af = *afp;
+	  else
+	    af = -1;
 
 	  switch (af)
 	    {
@@ -237,7 +263,7 @@
 	      /* This is not possible.  We cannot represent an IPv6 address
 		 in an `struct in_addr' variable.  */
 	      *h_errnop = HOST_NOT_FOUND;
-	      result = NULL;
+	      *result = NULL;
 	      goto done;
 
 	    case AF_INET6:
@@ -248,39 +274,35 @@
 	  size_needed = (sizeof (*host_addr)
 			 + sizeof (*h_addr_ptrs) + strlen (name) + 1);
 
-#ifdef HAVE_LOOKUP_BUFFER
-	  if (buflen < size_needed)
+	  if (buffer_size == NULL && buflen < size_needed)
 	    {
-# ifdef NEED_H_ERRNO
-	      *h_errnop = TRY_AGAIN;
-# endif
+	      if (h_errnop)
+		*h_errnop = TRY_AGAIN;
 	      __set_errno (ERANGE);
 	      goto done;
 	    }
-#else
-	  if (buffer_size < size_needed)
+	  else if (buffer_size != NULL && *buffer_size < size_needed)
 	    {
 	      char *new_buf;
-	      buffer_size = size_needed;
-	      new_buf = realloc (buffer, buffer_size);
+	      *buffer_size = size_needed;
+	      new_buf = realloc (*buffer, *buffer_size);
 
 	      if (new_buf == NULL)
 		{
 		  save = errno;
-		  free (buffer);
+		  free (*buffer);
 		  __set_errno (save);
-		  buffer = NULL;
-		  buffer_size = 0;
-		  result = (struct hostent *) NULL;
+		  *buffer = NULL;
+		  *buffer_size = 0;
+		  *result = (struct hostent *) NULL;
 		  goto done;
 		}
-	      buffer = new_buf;
+	      *buffer = new_buf;
 	    }
-#endif /* HAVE_LOOKUP_BUFFER */
 
-	  memset (buffer, 0, size_needed);
+	  memset (*buffer, 0, size_needed);
 
-	  host_addr = (host_addr_t *) buffer;
+	  host_addr = (host_addr_t *) *buffer;
 	  h_addr_ptrs = (host_addr_list_t *)
 	    ((char *) host_addr + sizeof (*host_addr));
 	  hostname = (char *) h_addr_ptrs + sizeof (*h_addr_ptrs);
@@ -297,26 +319,24 @@
 		  if (inet_pton (AF_INET6, name, host_addr) <= 0)
 		    {
 		      *h_errnop = HOST_NOT_FOUND;
-#ifndef HAVE_LOOKUP_BUFFER
-		      result = (struct hostent *) NULL;
-#endif
+		      if (buffer_size)
+			*result = (struct hostent *) NULL;
 		      goto done;
 		    }
 
-		  resbuf.h_name = strcpy (hostname, name);
+		  resbuf->h_name = strcpy (hostname, name);
 		  h_alias_ptr[0] = NULL;
-		  resbuf.h_aliases = h_alias_ptr;
+		  resbuf->h_aliases = h_alias_ptr;
 		  (*h_addr_ptrs)[0] = (char *) host_addr;
 		  (*h_addr_ptrs)[1] = (char *) 0;
-		  resbuf.h_addr_list = *h_addr_ptrs;
-		  resbuf.h_addrtype = AF_INET6;
-		  resbuf.h_length = addr_size;
+		  resbuf->h_addr_list = *h_addr_ptrs;
+		  resbuf->h_addrtype = AF_INET6;
+		  resbuf->h_length = addr_size;
 		  *h_errnop = NETDB_SUCCESS;
-#ifdef HAVE_LOOKUP_BUFFER
-		  status = NSS_STATUS_SUCCESS;
-#else
-		  result = &resbuf;
-#endif
+		  if (buffer_size == NULL)
+		    *status = NSS_STATUS_SUCCESS;
+		  else
+		    *result = resbuf;
 		  goto done;
 		}
 
@@ -324,3 +344,9 @@
 	    }
 	}
     }
+
+  return 0;
+
+done:
+  return 1;
+}
Index: nss/getXXbyYY.c
===================================================================
RCS file: /work/cvs/gnu/glibc/nss/getXXbyYY.c,v
retrieving revision 1.1.1.2
diff -u -p -r1.1.1.2 getXXbyYY.c
--- nss/getXXbyYY.c	2000/07/14 23:42:39	1.1.1.2
+++ nss/getXXbyYY.c	2000/12/09 23:03:33
@@ -59,11 +59,23 @@
 #ifdef NEED_H_ERRNO
 # define H_ERRNO_PARM , int *h_errnop
 # define H_ERRNO_VAR , &h_errno_tmp
+# define H_ERRNO_VAR_P &h_errno_tmp
 #else
 # define H_ERRNO_PARM
 # define H_ERRNO_VAR
+# define H_ERRNO_VAR_P NULL
 #endif
 
+#ifndef HAVE_TYPE
+#define TYPE_VAR_P NULL
+#define FLAGS_VAR 0
+#endif
+
+#ifdef HAVE_AF
+#define AF_VAR_P &af
+#else
+#define AF_VAR_P NULL
+#endif
 
 /* Prototype for reentrant version we use here.  */
 extern int INTERNAL (REENTRANT_NAME) (ADD_PARAMS, LOOKUP_TYPE *resbuf,
@@ -86,9 +98,6 @@ FUNCTION_NAME (ADD_PARAMS)
   int save;
 #ifdef NEED_H_ERRNO
   int h_errno_tmp = 0;
-# ifdef HANDLE_DIGITS_DOTS
-  int *const h_errnop = &h_errno_tmp;;
-# endif
 #endif
 
   /* Get lock.  */
@@ -100,22 +109,17 @@ FUNCTION_NAME (ADD_PARAMS)
       buffer = malloc (buffer_size);
     }
 
+#ifdef HANDLE_DIGITS_DOTS
   if (buffer != NULL)
     {
-#ifdef HANDLE_DIGITS_DOTS
-      /* We have to test for the use of IPv6 which can only be done by
-	 examining `_res'.  */
-      if ((_res.options & RES_INIT) == 0 && __res_ninit (&_res) == -1)
-	{
-# ifdef NEED_H_ERRNO
-	  h_errno_tmp = NETDB_INTERNAL;
-# endif
-	  result = NULL;
-	  goto done;
-	}
-# include "digits_dots.c"
-#endif
+      if (__nss_hostname_digits_dots (name, &resbuf, &buffer,
+				      &buffer_size,
+				      0, &result, NULL, TYPE_VAR_P,
+				      FLAGS_VAR, AF_VAR_P,
+				      H_ERRNO_VAR_P))
+	goto done;
     }
+#endif
 
   while (buffer != NULL
 	 && (INTERNAL (REENTRANT_NAME) (ADD_VARIABLES, &resbuf, buffer,
Index: nss/getXXbyYY_r.c
===================================================================
RCS file: /work/cvs/gnu/glibc/nss/getXXbyYY_r.c,v
retrieving revision 1.1.1.3
diff -u -p -r1.1.1.3 getXXbyYY_r.c
--- nss/getXXbyYY_r.c	2000/12/04 18:36:01	1.1.1.3
+++ nss/getXXbyYY_r.c	2000/12/09 23:03:33
@@ -87,11 +87,23 @@
 #ifdef NEED_H_ERRNO
 # define H_ERRNO_PARM , int *h_errnop
 # define H_ERRNO_VAR , h_errnop
+# define H_ERRNO_VAR_P h_errnop
 #else
 # define H_ERRNO_PARM
 # define H_ERRNO_VAR
+# define H_ERRNO_VAR_P NULL
 #endif
 
+#ifndef HAVE_TYPE
+#define TYPE_VAR_P NULL
+#define FLAGS_VAR 0
+#endif
+
+#ifdef HAVE_AF 
+#define AF_VAR_P &af
+#else
+#define AF_VAR_P NULL
+#endif
 
 /* Type of the lookup function we need here.  */
 typedef enum nss_status (*lookup_function) (ADD_PARAMS, LOOKUP_TYPE *, char *,
@@ -123,19 +135,16 @@ INTERNAL (REENTRANT_NAME) (ADD_PARAMS, L
 #endif
 
 #ifdef HANDLE_DIGITS_DOTS
-  /* We have to test for the use of IPv6 which can only be done by
-     examining `_res'.  */
-  if ((_res.options & RES_INIT) == 0 && __res_ninit (&_res) == -1)
+  switch (__nss_hostname_digits_dots (name, resbuf, &buffer, NULL,
+				      buflen, result, &status,
+				      TYPE_VAR_P, FLAGS_VAR, AF_VAR_P,
+				      H_ERRNO_VAR_P))
     {
-      *h_errnop = NETDB_INTERNAL;
-      *result = NULL;
+    case -1:
       return errno;
+    case 1:
+      goto done;
     }
-# define resbuf (*resbuf)
-# define result *result
-# include "digits_dots.c"
-# undef resbuf
-# undef result
 #endif
 
 #ifdef USE_NSCD
Index: nss/nsswitch.h
===================================================================
RCS file: /work/cvs/gnu/glibc/nss/nsswitch.h,v
retrieving revision 1.1.1.2
diff -u -p -r1.1.1.2 nsswitch.h
--- nss/nsswitch.h	2000/12/09 02:19:24	1.1.1.2
+++ nss/nsswitch.h	2000/12/09 23:03:33
@@ -160,5 +160,15 @@ extern int __nss_getent_r (const char *g
 extern void *__nss_getent (getent_r_function func,
 			   void **resbuf, char **buffer, size_t buflen,
 			   size_t *buffer_size, int *h_errnop);
+struct hostent;
+extern int __nss_hostname_digits_dots (const char *name,
+				       struct hostent *resbuf,
+				       char **buffer,
+				       size_t *buffer_size,
+				       size_t buflen,
+				       struct hostent **result,
+				       enum nss_status *status,
+				       int *typep, int flags,
+				       int *afp, int *h_errnop);
 
 #endif	/* nsswitch.h */

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]