about summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog10
-rw-r--r--debug/execinfo.h4
-rw-r--r--sysdeps/posix/getaddrinfo.c22
-rw-r--r--sysdeps/unix/sysv/linux/nscd_setup_thread.c2
4 files changed, 30 insertions, 8 deletions
diff --git a/ChangeLog b/ChangeLog
index d06511f628..768cc8b37b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2004-09-18  Jakub Jelinek  <jakub@redhat.com>
+
+	* sysdeps/unix/sysv/linux/nscd_setup_thread.c (setup_thread):
+	Do nothing if __NR_set_tid_address is not defined.  [BZ #390]
+
+2004-09-17  Ulrich Drepper  <drepper@redhat.com>
+
+	* sysdeps/posix/getaddrinfo.c: Use gethostbyname3_r NSS function
+	in case it is available.
+
 2004-09-17  Jakub Jelinek  <jakub@redhat.com>
 
 	* nscd/nscd.c (parse_opt): Write arg string instead of (void *)
diff --git a/debug/execinfo.h b/debug/execinfo.h
index 0ea5f6777e..ff11c0c314 100644
--- a/debug/execinfo.h
+++ b/debug/execinfo.h
@@ -37,8 +37,8 @@ extern char **backtrace_symbols (void *__const *__array, int __size)
 /* This function is similar to backtrace_symbols() but it writes the result
    immediately to a file and can therefore also be used in situations where
    malloc() is not usable anymore.  */
-extern void backtrace_symbols_fd (void *__const *__array, int __size,
-				  int __fd) __THROW __nonnull ((1));
+extern void backtrace_symbols_fd (void *__const *__array, int __size, int __fd)
+     __THROW __nonnull ((1));
 
 __END_DECLS
 
diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
index 9bf2537235..c8cb752b18 100644
--- a/sysdeps/posix/getaddrinfo.c
+++ b/sysdeps/posix/getaddrinfo.c
@@ -289,11 +289,12 @@ gaih_inet_serv (const char *servicename, const struct gaih_typeproto *tp,
   int herrno;								      \
   struct hostent th;							      \
   struct hostent *h;							      \
+  char *localcanon = NULL;						      \
   no_data = 0;								      \
   while (1) {								      \
     rc = 0;								      \
     status = DL_CALL_FCT (fct, (name, _family, &th, tmpbuf,		      \
-           tmpbuflen, &rc, &herrno));					      \
+			  tmpbuflen, &rc, &herrno, NULL, &localcanon));	      \
     if (rc != ERANGE || herrno != NETDB_INTERNAL)			      \
       break;								      \
     tmpbuf = extend_alloca (tmpbuf, tmpbuflen, 2 * tmpbuflen);		      \
@@ -342,16 +343,19 @@ gaih_inet_serv (const char *servicename, const struct gaih_typeproto *tp,
 	  pat = &((*pat)->next);					      \
 	}								      \
 									      \
+      if (localcanon !=	NULL)						      \
+	canon = strdupa (localcanon);					      \
+									      \
       if (_family == AF_INET6 && i > 0)					      \
 	got_ipv6 = true;						      \
     }									      \
  }
 
 
-typedef enum nss_status (*nss_gethostbyname2_r)
+typedef enum nss_status (*nss_gethostbyname3_r)
   (const char *name, int af, struct hostent *host,
    char *buffer, size_t buflen, int *errnop,
-   int *h_errnop);
+   int *h_errnop, int32_t *ttlp, char **canonp);
 typedef enum nss_status (*nss_getcanonname_r)
   (const char *name, char *buffer, size_t buflen, char **result,
    int *errnop, int *h_errnop);
@@ -771,8 +775,14 @@ gaih_inet (const char *name, const struct gaih_service *service,
 
 	  while (!no_more)
 	    {
-	      nss_gethostbyname2_r fct
-		= __nss_lookup_function (nip, "gethostbyname2_r");
+	      nss_gethostbyname3_r fct
+		= __nss_lookup_function (nip, "gethostbyname3_r");
+	      if (fct == NULL)
+		/* We are cheating here.  The gethostbyname2_r function does
+		   not have the same interface as gethostbyname3_r but the
+		   extra arguments the latter takes are added at the end.
+		   So the gethostbyname2_r code will just ignore them.  */
+		fct = __nss_lookup_function (nip, "gethostbyname2_r");
 
 	      if (fct != NULL)
 		{
@@ -805,7 +815,7 @@ gaih_inet (const char *name, const struct gaih_service *service,
 		  if (inet6_status == NSS_STATUS_SUCCESS
 		      || status == NSS_STATUS_SUCCESS)
 		    {
-		      if ((req->ai_flags & AI_CANONNAME) != 0)
+		      if ((req->ai_flags & AI_CANONNAME) != 0 && canon == NULL)
 			{
 			  /* If we need the canonical name, get it
 			     from the same service as the result.  */
diff --git a/sysdeps/unix/sysv/linux/nscd_setup_thread.c b/sysdeps/unix/sysv/linux/nscd_setup_thread.c
index 3496f02dca..7d1bfd8dd1 100644
--- a/sysdeps/unix/sysv/linux/nscd_setup_thread.c
+++ b/sysdeps/unix/sysv/linux/nscd_setup_thread.c
@@ -27,6 +27,7 @@
 void
 setup_thread (struct database_dyn *db)
 {
+#ifdef __NR_set_tid_address
   /* Only supported when NPTL is used.  */
   char buf[100];
   if (confstr (_CS_GNU_LIBPTHREAD_VERSION, buf, sizeof (buf)) >= sizeof (buf)
@@ -44,4 +45,5 @@ setup_thread (struct database_dyn *db)
        So, set the field to a nonzero value which indicates that nscd
        is certainly running and clients can skip the test.  */
     db->head->nscd_certainly_running = 1;
+#endif
 }