From: Martin v. Löwis <martin@v.loewis.de>
Date: Fri, 3 Aug 2001 10:02:29 +0000 (+0000)
Subject: Do not use the system getaddrinfo on Mac OS X. Fixes bug #445928.
X-Git-Tag: v2.2a3~856
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ae26dc23a9c91510244c774bf34c011d19c4cdfa;p=python

Do not use the system getaddrinfo on Mac OS X. Fixes bug #445928.
Since getnameinfo is not implemented, remove __APPLE__ check here.
---

diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c
index 706ca78bcd..e6a9ec82ef 100644
--- a/Modules/socketmodule.c
+++ b/Modules/socketmodule.c
@@ -218,15 +218,25 @@ int inet_pton (int af, const char *src, void *dst);
 const char *inet_ntop(int af, const void *src, char *dst, socklen_t size);
 #endif
 
+#ifdef __APPLE__
+/* On OS X, getaddrinfo returns no error indication of lookup
+   failure, so we must use the emulation instead of the libinfo
+   implementation. Unfortunately, performing an autoconf test
+   for this bug would require DNS access for the machine performing
+   the configuration, which is not acceptable. Therefore, we
+   determine the bug just by checking for __APPLE__. If this bug
+   gets ever fixed, perhaps checking for sys/version.h would be
+   appropriate, which is 10/0 on the system with the bug. */
+#undef HAVE_GETADDRINFO
+/* avoid clashes with the C library definition of the symbol. */
+#define getaddrinfo fake_getaddrinfo
+#endif
+
 /* I know this is a bad practice, but it is the easiest... */
-/* XXX Temporarily work around bug #445928:
-   getaddrinfo on Darwin seems to return an empty result list, with
-   no error, even if host lookup ought to work fine. So use the
-   emulation code for now. */
-#if !defined(HAVE_GETADDRINFO) || defined(__APPLE__)
+#if !defined(HAVE_GETADDRINFO)
 #include "getaddrinfo.c"
 #endif
-#if !defined(HAVE_GETNAMEINFO) || defined(__APPLE__)
+#if !defined(HAVE_GETNAMEINFO)
 #include "getnameinfo.c"
 #endif