Revert Jim's 2006-10-23 patch.
authorSimon Josefsson <simon@josefsson.org>
Fri, 17 Nov 2006 13:47:43 +0000 (13:47 +0000)
committerSimon Josefsson <simon@josefsson.org>
Fri, 17 Nov 2006 13:47:43 +0000 (13:47 +0000)
ChangeLog
lib/getaddrinfo.c
modules/getaddrinfo

index 770dc45..24159e6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2006-11-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * modules/getaddinfo (Depends-on): Remove inttostr; add snprintf.
+       * lib/getaddrinfo.c: Include snprintf.h rather than intprops.h
+       and inttostr.h.  Use snprintf rather than uinttostr, so that
+       LGPLed code doesn't depend on GPLed.
+
 2006-11-17  Paul Eggert  <eggert@cs.ucla.edu>
 
        * modules/inline (License): Change from GPL to LGPL.
index 358214a..f523f76 100644 (file)
@@ -38,8 +38,7 @@
 #define N_(String) String
 
 #include "inet_ntop.h"
-#include "intprops.h"
-#include "inttostr.h"
+#include "snprintf.h"
 #include "strdup.h"
 
 /* BeOS has AF_INET, but not PF_INET.  */
@@ -408,11 +407,8 @@ int getnameinfo(const struct sockaddr *restrict sa, socklen_t salen,
        {
          unsigned short int port
            = ntohs (((const struct sockaddr_in *) sa)->sin_port);
-         char buf[INT_BUFSIZE_BOUND (port)];
-         char const *s = uinttostr (port, buf);
-         if (strlen (s) + 1 > servicelen)
+         if (servicelen <= snprintf (service, servicelen, "%u", port))
            return EAI_OVERFLOW;
-         memcpy (service, s, strlen (s) + 1);
        }
        break;
       }
index 8f5979d..b2b6a35 100644 (file)
@@ -10,7 +10,7 @@ m4/getaddrinfo.m4
 Depends-on:
 gettext-h
 inet_ntop
-inttostr
+snprintf
 socklen
 stdbool
 strdup