aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2015-06-12 22:35:07 +0000
committerJoseph Myers <joseph@codesourcery.com>2015-06-12 22:35:07 +0000
commit8737be806407be51f046b358d8824c4c968912de (patch)
tree11c6eeb4637ff4e67ca80b9a0105a334f1373f0f
parentb400fdefc71ccc2a118ddfa78d3658d3ea3af7a2 (diff)
downloadglibc-8737be806407be51f046b358d8824c4c968912de.zip
glibc-8737be806407be51f046b358d8824c4c968912de.tar.gz
glibc-8737be806407be51f046b358d8824c4c968912de.tar.bz2
Fix netdb.h addrinfo namespace (bug 18529).
netdb.h declares interfaces such as getaddrinfo if __USE_POSIX, i.e. POSIX.1:1990 or later. However, these interfaces were new in the 2001 edition of POSIX, although the header was in XPG4 and UNIX98, so they should not be declared for XPG4 or UNIX98. (This produces spurious linknamespace test failures, although there are other failures for this header as well for the same standards so this patch doesn't remove any XFAILs.) This patch corrects the condition, and the conform/ test expectations which were similarly wrong. Tested for x86_64 and x86 (testsuite, and that installed stripped shared libraries are unchanged by the patch). [BZ #18529] * resolv/netdb.h [__USE_POSIX]: Change condition to [__USE_XOPEN2K]. * conform/data/netdb.h-data [XPG4 || UNIX98] (struct addrinfo): Do not expect. [XPG4 || UNIX98] (AI_PASSIVE): Likewise. [XPG4 || UNIX98] (AI_CANONNAME): Likewise. [XPG4 || UNIX98] (AI_NUMERICHOST): Likewise. [XPG4 || UNIX98] (AI_V4MAPPED): Likewise. [XPG4 || UNIX98] (AI_ALL): Likewise. [XPG4 || UNIX98] (AI_ADDRCONFIG): Likewise. [XPG4 || UNIX98] (AI_NUMERICSERV): Likewise. [XPG4 || UNIX98] (NI_NOFQDN): Likewise. [XPG4 || UNIX98] (NI_NUMERICHOST): Likewise. [XPG4 || UNIX98] (NI_NAMEREQD): Likewise. [XPG4 || UNIX98] (NI_NUMERICSERV): Likewise. [XPG4 || UNIX98] (NI_DGRAM): Likewise. [XPG4 || UNIX98] (EAI_AGAIN): Likewise. [XPG4 || UNIX98] (EAI_BADFLAGS): Likewise. [XPG4 || UNIX98] (EAI_FAIL): Likewise. [XPG4 || UNIX98] (EAI_FAMILY): Likewise. [XPG4 || UNIX98] (EAI_MEMORY): Likewise. [XPG4 || UNIX98] (EAI_NONAME): Likewise. [XPG4 || UNIX98] (EAI_SERVICE): Likewise. [XPG4 || UNIX98] (EAI_SOCKTYPE): Likewise. [XPG4 || UNIX98] (EAI_SYSTEM): Likewise. [XPG4 || UNIX98] (EAI_SYSTEM): Likewise. [XPG4 || UNIX98] (freeaddrinfo): Likewise. [XPG4 || UNIX98] (gai_strerror): Likewise. [XPG4 || UNIX98] (getaddrinfo): Likewise. [XPG4 || UNIX98] (getnameinfo): Likewise.
-rw-r--r--ChangeLog32
-rw-r--r--NEWS2
-rw-r--r--conform/data/netdb.h-data6
-rw-r--r--resolv/netdb.h4
4 files changed, 41 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index fa59616..a16069d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,37 @@
2015-06-12 Joseph Myers <joseph@codesourcery.com>
+ [BZ #18529]
+ * resolv/netdb.h [__USE_POSIX]: Change condition to
+ [__USE_XOPEN2K].
+ * conform/data/netdb.h-data [XPG4 || UNIX98] (struct addrinfo): Do
+ not expect.
+ [XPG4 || UNIX98] (AI_PASSIVE): Likewise.
+ [XPG4 || UNIX98] (AI_CANONNAME): Likewise.
+ [XPG4 || UNIX98] (AI_NUMERICHOST): Likewise.
+ [XPG4 || UNIX98] (AI_V4MAPPED): Likewise.
+ [XPG4 || UNIX98] (AI_ALL): Likewise.
+ [XPG4 || UNIX98] (AI_ADDRCONFIG): Likewise.
+ [XPG4 || UNIX98] (AI_NUMERICSERV): Likewise.
+ [XPG4 || UNIX98] (NI_NOFQDN): Likewise.
+ [XPG4 || UNIX98] (NI_NUMERICHOST): Likewise.
+ [XPG4 || UNIX98] (NI_NAMEREQD): Likewise.
+ [XPG4 || UNIX98] (NI_NUMERICSERV): Likewise.
+ [XPG4 || UNIX98] (NI_DGRAM): Likewise.
+ [XPG4 || UNIX98] (EAI_AGAIN): Likewise.
+ [XPG4 || UNIX98] (EAI_BADFLAGS): Likewise.
+ [XPG4 || UNIX98] (EAI_FAIL): Likewise.
+ [XPG4 || UNIX98] (EAI_FAMILY): Likewise.
+ [XPG4 || UNIX98] (EAI_MEMORY): Likewise.
+ [XPG4 || UNIX98] (EAI_NONAME): Likewise.
+ [XPG4 || UNIX98] (EAI_SERVICE): Likewise.
+ [XPG4 || UNIX98] (EAI_SOCKTYPE): Likewise.
+ [XPG4 || UNIX98] (EAI_SYSTEM): Likewise.
+ [XPG4 || UNIX98] (EAI_SYSTEM): Likewise.
+ [XPG4 || UNIX98] (freeaddrinfo): Likewise.
+ [XPG4 || UNIX98] (gai_strerror): Likewise.
+ [XPG4 || UNIX98] (getaddrinfo): Likewise.
+ [XPG4 || UNIX98] (getnameinfo): Likewise.
+
[BZ #18528]
* grp/grp.h (endgrent): Condition on [__USE_MISC ||
__USE_XOPEN_EXTENDED], not [__USE_XOPEN_EXTENDED ||
diff --git a/NEWS b/NEWS
index 75bddc1..592fa45 100644
--- a/NEWS
+++ b/NEWS
@@ -21,7 +21,7 @@ Version 2.22
18211, 18217, 18220, 18221, 18234, 18244, 18247, 18287, 18319, 18324,
18333, 18346, 18397, 18409, 18410, 18412, 18418, 18422, 18434, 18444,
18468, 18469, 18470, 18479, 18483, 18495, 18496, 18497, 18498, 18507,
- 18512, 18519, 18520, 18522, 18527, 18528.
+ 18512, 18519, 18520, 18522, 18527, 18528, 18529.
* Cache information can be queried via sysconf() function on s390 e.g. with
_SC_LEVEL1_ICACHE_SIZE as argument.
diff --git a/conform/data/netdb.h-data b/conform/data/netdb.h-data
index c5fd257..63a42ae 100644
--- a/conform/data/netdb.h-data
+++ b/conform/data/netdb.h-data
@@ -44,6 +44,7 @@ macro NO_RECOVERY
macro TRY_AGAIN
#endif
+#if !defined XPG4 && !defined UNIX98
type {struct addrinfo}
element {struct addrinfo} int ai_flags
element {struct addrinfo} int ai_family
@@ -78,18 +79,23 @@ macro EAI_SERVICE
macro EAI_SOCKTYPE
macro EAI_SYSTEM
macro EAI_OVERFLOW
+#endif
function void endhostent (void)
function void endnetent (void)
function void endprotoent (void)
function void endservent (void)
+#if !defined XPG4 && !defined UNIX98
function void freeaddrinfo (struct addrinfo*)
function {const char*} gai_strerror (int)
function int getaddrinfo (const char*, const char*, const struct addrinfo*, struct addrinfo**)
+#endif
function {struct hostent*} gethostbyaddr (const void*, socklen_t, int)
function {struct hostent*} gethostbyname (const char*)
function {struct hostent*} gethostent (void)
+#if !defined XPG4 && !defined UNIX98
function int getnameinfo (const struct sockaddr*, socklen_t, char*, socklen_t, char*, socklen_t, int)
+#endif
function {struct netent*} getnetbyaddr (uint32_t, int)
function {struct netent*} getnetbyname (const char*)
function {struct netent*} getnetent (void)
diff --git a/resolv/netdb.h b/resolv/netdb.h
index fe8f3ba..44cf692 100644
--- a/resolv/netdb.h
+++ b/resolv/netdb.h
@@ -561,8 +561,8 @@ extern int rresvport_af (int *__alport, sa_family_t __af);
#endif
-/* Extension from POSIX.1g. */
-#ifdef __USE_POSIX
+/* Extension from POSIX.1:2001. */
+#ifdef __USE_XOPEN2K
/* Structure to contain information about address of a service provider. */
struct addrinfo
{