diff --git a/ChangeLog b/ChangeLog index f5df1b9bbf..e211a52879 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2008-12-08 Ulrich Drepper + + [BZ #7080] + * inet/getnameinfo.c (getnameinfo): Check for output buffers being + NULL when NI_NAMEREQD is set. + Patch mostly by Yang Hongyang . + * inet/Makefile (tests): Add tst-getni1. + * inet/tst-getni1.c: New file. + 2008-12-03 Petr Baudis [BZ #7067] diff --git a/inet/Makefile b/inet/Makefile index d7139c1d7f..64154bb096 100644 --- a/inet/Makefile +++ b/inet/Makefile @@ -52,7 +52,8 @@ routines := htonl htons \ aux := check_pf check_native ifreq tests := htontest test_ifindex tst-ntoa tst-ether_aton tst-network \ - tst-gethnm test-ifaddrs bug-if1 test-inet6_opt tst-ether_line + tst-gethnm test-ifaddrs bug-if1 test-inet6_opt tst-ether_line \ + tst-getni1 include ../Rules diff --git a/inet/getnameinfo.c b/inet/getnameinfo.c index 50240383f3..db6b5c7eee 100644 --- a/inet/getnameinfo.c +++ b/inet/getnameinfo.c @@ -178,6 +178,9 @@ getnameinfo (const struct sockaddr *sa, socklen_t addrlen, char *host, if (sa == NULL || addrlen < sizeof (sa_family_t)) return EAI_FAMILY; + if ((flags & NI_NAMEREQD) && host == NULL && serv == NULL) + return EAI_NONAME; + switch (sa->sa_family) { case AF_LOCAL: diff --git a/inet/tst-getni1.c b/inet/tst-getni1.c new file mode 100644 index 0000000000..0e8a792f49 --- /dev/null +++ b/inet/tst-getni1.c @@ -0,0 +1,36 @@ +#include +#include +#include + +static int +do_test (void) +{ + int retval = 0; + + struct sockaddr_in s; + s.sin_family = AF_INET; + s.sin_port = 80; + s.sin_addr.s_addr = INADDR_LOOPBACK; + int r = getnameinfo((struct sockaddr *) &s, sizeof (s), NULL, 0, NULL, 0, + NI_NUMERICHOST | NI_NUMERICSERV); + printf("r = %d\n", r); + if (r != 0) + { + puts ("failed without NI_NAMEREQD"); + retval = 1; + } + + r = getnameinfo((struct sockaddr *) &s, sizeof (s), NULL, 0, NULL, 0, + NI_NUMERICHOST | NI_NUMERICSERV | NI_NAMEREQD); + printf("r = %d\n", r); + if (r != EAI_NONAME) + { + puts ("did not fail with EAI_NONAME with NI_NAMEREQD set"); + retval = 1; + } + + return retval; +} + +#define TEST_FUNCTION do_test () +#include "../test-skeleton.c"