fix
[oweals/gnunet.git] / src / util / gnunet-service-resolver.c
index b0e2f0be4f76e883e398b4f7d2886c7c0106b38a..c15f0d3f54925bb742e6f90cab73961c1cbfda7b 100644 (file)
@@ -23,8 +23,6 @@
  * @brief code to do DNS resolution
  * @author Christian Grothoff
  */
-
-#include <stdlib.h>
 #include "platform.h"
 #include "gnunet_disk_lib.h"
 #include "gnunet_getopt_lib.h"
@@ -89,9 +87,9 @@ getnameinfo_resolve (struct IPCache *cache)
 {
   char hostname[256];
 
-  if (0 == getnameinfo (cache->sa,
-                        cache->salen,
-                        hostname, sizeof (hostname), NULL, 0, 0))
+  if (0 ==
+      getnameinfo (cache->sa, cache->salen, hostname, sizeof (hostname), NULL,
+                  0, 0))
     cache->addr = GNUNET_strdup (hostname);
 }
 #endif
@@ -111,12 +109,14 @@ gethostbyaddr_resolve (struct IPCache *cache)
   switch (cache->sa->sa_family)
     {
     case AF_INET:
-      ent = gethostbyaddr (&((struct sockaddr_in *) cache->sa)->sin_addr,
-                           sizeof (struct in_addr), AF_INET);
+      ent =
+       gethostbyaddr (&((struct sockaddr_in *) cache->sa)->sin_addr,
+                      sizeof (struct in_addr), AF_INET);
       break;
     case AF_INET6:
-      ent = gethostbyaddr (&((struct sockaddr_in6 *) cache->sa)->sin6_addr,
-                           sizeof (struct in6_addr), AF_INET6);
+      ent =
+       gethostbyaddr (&((struct sockaddr_in6 *) cache->sa)->sin6_addr,
+                      sizeof (struct in6_addr), AF_INET6);
       break;
     default:
       ent = NULL;
@@ -158,7 +158,7 @@ cache_resolve (struct IPCache *cache)
  */
 static void
 get_ip_as_string (struct GNUNET_SERVER_Client *client,
-                  const struct sockaddr *sa, socklen_t salen)
+                 const struct sockaddr *sa, socklen_t salen)
 {
   struct IPCache *cache;
   struct IPCache *prev;
@@ -175,29 +175,29 @@ get_ip_as_string (struct GNUNET_SERVER_Client *client,
   cache = head;
   prev = NULL;
   while ((cache != NULL) &&
-         ((cache->salen != salen) || (0 != memcmp (cache->sa, sa, salen))))
+        ((cache->salen != salen) || (0 != memcmp (cache->sa, sa, salen))))
     {
       if (GNUNET_TIME_absolute_get_duration (cache->last_request).rel_value <
-          60 * 60 * 1000)
-        {
-          if (prev != NULL)
-            {
-              prev->next = cache->next;
-              GNUNET_free_non_null (cache->addr);
-              GNUNET_free (cache->sa);
-              GNUNET_free (cache);
-              cache = prev->next;
-            }
-          else
-            {
-              head = cache->next;
-              GNUNET_free_non_null (cache->addr);
-              GNUNET_free (cache->sa);
-              GNUNET_free (cache);
-              cache = head;
-            }
-          continue;
-        }
+         60 * 60 * 1000)
+       {
+         if (prev != NULL)
+           {
+             prev->next = cache->next;
+             GNUNET_free_non_null (cache->addr);
+             GNUNET_free (cache->sa);
+             GNUNET_free (cache);
+             cache = prev->next;
+           }
+         else
+           {
+             head = cache->next;
+             GNUNET_free_non_null (cache->addr);
+             GNUNET_free (cache->sa);
+             GNUNET_free (cache);
+             cache = head;
+           }
+         continue;
+       }
       prev = cache;
       cache = cache->next;
     }
@@ -205,13 +205,13 @@ get_ip_as_string (struct GNUNET_SERVER_Client *client,
     {
       cache->last_request = now;
       if (GNUNET_TIME_absolute_get_duration (cache->last_request).rel_value <
-          60 * 60 * 1000)
-        {
-          GNUNET_free_non_null (cache->addr);
-          cache->addr = NULL;
-          cache->salen = 0;
-          cache_resolve (cache);
-        }
+         60 * 60 * 1000)
+       {
+         GNUNET_free_non_null (cache->addr);
+         cache->addr = NULL;
+         cache->salen = 0;
+         cache_resolve (cache);
+       }
     }
   else
     {
@@ -228,8 +228,7 @@ get_ip_as_string (struct GNUNET_SERVER_Client *client,
     }
   tc = GNUNET_SERVER_transmit_context_create (client);
   if (cache->addr != NULL)
-    GNUNET_SERVER_transmit_context_append_data (tc,
-                                               cache->addr,
+    GNUNET_SERVER_transmit_context_append_data (tc, cache->addr,
                                                strlen (cache->addr) + 1,
                                                GNUNET_MESSAGE_TYPE_RESOLVER_RESPONSE);
   GNUNET_SERVER_transmit_context_append_data (tc, NULL, 0,
@@ -241,7 +240,7 @@ get_ip_as_string (struct GNUNET_SERVER_Client *client,
 #if HAVE_GETADDRINFO
 static int
 getaddrinfo_resolve (struct GNUNET_SERVER_TransmitContext *tc,
-                     const char *hostname, int domain)
+                    const char *hostname, int domain)
 {
   int s;
   struct addrinfo hints;
@@ -255,25 +254,25 @@ getaddrinfo_resolve (struct GNUNET_SERVER_TransmitContext *tc,
 #else
   hints.ai_family = AF_INET;
 #endif
-  hints.ai_socktype = SOCK_STREAM;      /* go for TCP */
+  hints.ai_socktype = SOCK_STREAM;     /* go for TCP */
 
   if (0 != (s = getaddrinfo (hostname, NULL, &hints, &result)))
     {
       GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                  _("Could not resolve `%s' (%s): %s\n"), hostname,
-                  (domain ==
-                   AF_INET) ? "IPv4" : ((domain ==
-                                         AF_INET6) ? "IPv6" : "any"),
-                  gai_strerror (s));
-      if ((s == EAI_BADFLAGS) || (s == EAI_MEMORY) ||
+                 _("Could not resolve `%s' (%s): %s\n"), hostname,
+                 (domain ==
+                  AF_INET) ? "IPv4" : ((domain ==
+                                        AF_INET6) ? "IPv6" : "any"),
+                 gai_strerror (s));
+      if ((s == EAI_BADFLAGS) || (s == EAI_MEMORY)
 #ifndef MINGW
-          (s == EAI_SYSTEM)
+         || (s == EAI_SYSTEM)
 #else
-          // FIXME NILS
-          1
+         // FIXME NILS
+         || 1
 #endif
-        )
-        return GNUNET_NO;       /* other function may still succeed */
+       )
+       return GNUNET_NO;       /* other function may still succeed */
       return GNUNET_SYSERR;
     }
   if (result == NULL)
@@ -281,8 +280,7 @@ getaddrinfo_resolve (struct GNUNET_SERVER_TransmitContext *tc,
   pos = result;
   while (pos != NULL)
     {
-      GNUNET_SERVER_transmit_context_append_data (tc,
-                                                 pos->ai_addr,
+      GNUNET_SERVER_transmit_context_append_data (tc, pos->ai_addr,
                                                  pos->ai_addrlen,
                                                  GNUNET_MESSAGE_TYPE_RESOLVER_RESPONSE);
       pos = pos->ai_next;
@@ -295,7 +293,7 @@ getaddrinfo_resolve (struct GNUNET_SERVER_TransmitContext *tc,
 #if HAVE_GETHOSTBYNAME2
 static int
 gethostbyname2_resolve (struct GNUNET_SERVER_TransmitContext *tc,
-                        const char *hostname, int domain)
+                       const char *hostname, int domain)
 {
   struct hostent *hp;
   struct sockaddr_in a4;
@@ -308,17 +306,17 @@ gethostbyname2_resolve (struct GNUNET_SERVER_TransmitContext *tc,
       ret1 = gethostbyname2_resolve (tc, hostname, AF_INET);
       ret2 = gethostbyname2_resolve (tc, hostname, AF_INET6);
       if ((ret1 == GNUNET_OK) || (ret2 == GNUNET_OK))
-        return GNUNET_OK;
+       return GNUNET_OK;
       if ((ret1 == GNUNET_SYSERR) || (ret2 == GNUNET_SYSERR))
-        return GNUNET_SYSERR;
+       return GNUNET_SYSERR;
       return GNUNET_NO;
     }
   hp = gethostbyname2 (hostname, domain);
   if (hp == NULL)
     {
       GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                  _("Could not find IP of host `%s': %s\n"),
-                  hostname, hstrerror (h_errno));
+                 _("Could not find IP of host `%s': %s\n"), hostname,
+                 hstrerror (h_errno));
       return GNUNET_SYSERR;
     }
   GNUNET_assert (hp->h_addrtype == domain);
@@ -327,10 +325,11 @@ gethostbyname2_resolve (struct GNUNET_SERVER_TransmitContext *tc,
       GNUNET_assert (hp->h_length == sizeof (struct in_addr));
       memset (&a4, 0, sizeof (a4));
       a4.sin_family = AF_INET;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+      a4.sin_len = (u_char) sizeof (struct sockaddr_in);
+#endif
       memcpy (&a4.sin_addr, hp->h_addr_list[0], hp->h_length);
-      GNUNET_SERVER_transmit_context_append_data (tc,
-                                                 &a4,
-                                                 sizeof (a4),
+      GNUNET_SERVER_transmit_context_append_data (tc, &a4, sizeof (a4),
                                                  GNUNET_MESSAGE_TYPE_RESOLVER_RESPONSE);
     }
   else
@@ -338,10 +337,11 @@ gethostbyname2_resolve (struct GNUNET_SERVER_TransmitContext *tc,
       GNUNET_assert (hp->h_length == sizeof (struct in6_addr));
       memset (&a6, 0, sizeof (a6));
       a6.sin6_family = AF_INET6;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+      a6.sin6_len = (u_char) sizeof (struct sockaddr_in6);
+#endif
       memcpy (&a6.sin6_addr, hp->h_addr_list[0], hp->h_length);
-      GNUNET_SERVER_transmit_context_append_data (tc,
-                                                 &a6,
-                                                 sizeof (a6),
+      GNUNET_SERVER_transmit_context_append_data (tc, &a6, sizeof (a6),
                                                  GNUNET_MESSAGE_TYPE_RESOLVER_RESPONSE);
     }
   return GNUNET_OK;
@@ -351,7 +351,7 @@ gethostbyname2_resolve (struct GNUNET_SERVER_TransmitContext *tc,
 #if HAVE_GETHOSTBYNAME
 static int
 gethostbyname_resolve (struct GNUNET_SERVER_TransmitContext *tc,
-                       const char *hostname)
+                      const char *hostname)
 {
   struct hostent *hp;
   struct sockaddr_in addr;
@@ -360,8 +360,8 @@ gethostbyname_resolve (struct GNUNET_SERVER_TransmitContext *tc,
   if (hp == NULL)
     {
       GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                  _("Could not find IP of host `%s': %s\n"),
-                  hostname, hstrerror (h_errno));
+                 _("Could not find IP of host `%s': %s\n"), hostname,
+                 hstrerror (h_errno));
       return GNUNET_SYSERR;
     }
   if (hp->h_addrtype != AF_INET)
@@ -372,10 +372,11 @@ gethostbyname_resolve (struct GNUNET_SERVER_TransmitContext *tc,
   GNUNET_assert (hp->h_length == sizeof (struct in_addr));
   memset (&addr, 0, sizeof (addr));
   addr.sin_family = AF_INET;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  addr.sin_len = (u_char) sizeof (struct sockaddr_in);
+#endif
   memcpy (&addr.sin_addr, hp->h_addr_list[0], hp->h_length);
-  GNUNET_SERVER_transmit_context_append_data (tc,
-                                             &addr,
-                                             sizeof (addr),
+  GNUNET_SERVER_transmit_context_append_data (tc, &addr, sizeof (addr),
                                              GNUNET_MESSAGE_TYPE_RESOLVER_RESPONSE);
   return GNUNET_OK;
 }
@@ -391,7 +392,7 @@ gethostbyname_resolve (struct GNUNET_SERVER_TransmitContext *tc,
  */
 static void
 get_ip_from_hostname (struct GNUNET_SERVER_Client *client,
-                      const char *hostname, int domain)
+                     const char *hostname, int domain)
 {
   int ret;
   struct GNUNET_SERVER_TransmitContext *tc;
@@ -424,13 +425,13 @@ get_ip_from_hostname (struct GNUNET_SERVER_Client *client,
  * @param message the actual message
  */
 static void
-handle_get (void *cls,
-            struct GNUNET_SERVER_Client *client,
-            const struct GNUNET_MessageHeader *message)
+handle_get (void *cls, struct GNUNET_SERVER_Client *client,
+           const struct GNUNET_MessageHeader *message)
 {
   uint16_t msize;
   const struct GNUNET_RESOLVER_GetMessage *msg;
   const char *hostname;
+  const struct sockaddr *sa;
   uint16_t size;
   int direction;
   int domain;
@@ -451,24 +452,63 @@ handle_get (void *cls,
       /* IP from hostname */
       hostname = (const char *) &msg[1];
       if (hostname[size - 1] != '\0')
-        {
-          GNUNET_break (0);
-          GNUNET_SERVER_receive_done (client, GNUNET_SYSERR);
-          return;
-        }
+       {
+         GNUNET_break (0);
+         GNUNET_SERVER_receive_done (client, GNUNET_SYSERR);
+         return;
+       }
 #if DEBUG_RESOLVER
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  _("Resolver asked to look up `%s'.\n"), hostname);
+                 _("Resolver asked to look up `%s'.\n"), hostname);
 #endif
       get_ip_from_hostname (client, hostname, domain);
     }
   else
     {
+#if DEBUG_RESOLVER
+      char buf[INET6_ADDRSTRLEN];
+#endif
+      if (size < sizeof (struct sockaddr))
+       {
+         GNUNET_break (0);
+         GNUNET_SERVER_receive_done (client, GNUNET_SYSERR);
+         return;
+       }
+      sa = (const struct sockaddr *) &msg[1];
+      switch (sa->sa_family)
+       {
+       case AF_INET:
+         if (size != sizeof (struct sockaddr_in))
+           {
+             GNUNET_break (0);
+             GNUNET_SERVER_receive_done (client, GNUNET_SYSERR);
+             return;
+           }
+#if DEBUG_RESOLVER
+         inet_ntop (AF_INET, sa, buf, size);
+#endif
+         break;
+       case AF_INET6:
+         if (size != sizeof (struct sockaddr_in6))
+           {
+             GNUNET_break (0);
+             GNUNET_SERVER_receive_done (client, GNUNET_SYSERR);
+             return;
+           }
+#if DEBUG_RESOLVER
+         inet_ntop (AF_INET6, sa, buf, size);
+#endif
+         break;
+       default:
+         GNUNET_break (0);
+         GNUNET_SERVER_receive_done (client, GNUNET_SYSERR);
+         return;
+       }
 #if DEBUG_RESOLVER
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  _("Resolver asked to look up IP address.\n"));
+                 _("Resolver asked to look up IP address `%s'.\n"), buf);
 #endif
-      get_ip_as_string (client, (const struct sockaddr *) &msg[1], size);
+      get_ip_as_string (client, sa, size);
     }
 }
 
@@ -477,14 +517,11 @@ handle_get (void *cls,
  * Process resolver requests.
  *
  * @param cls closure
- * @param sched scheduler to use
  * @param server the initialized server
  * @param cfg configuration to use
  */
 static void
-run (void *cls,
-     struct GNUNET_SCHEDULER_Handle *sched,
-     struct GNUNET_SERVER_Handle *server,
+run (void *cls, struct GNUNET_SERVER_Handle *server,
      const struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   static const struct GNUNET_SERVER_MessageHandler handlers[] = {
@@ -508,11 +545,10 @@ main (int argc, char *const *argv)
   int ret;
   struct IPCache *pos;
 
-  ret = (GNUNET_OK ==
-         GNUNET_SERVICE_run (argc,
-                             argv,
-                             "resolver", GNUNET_SERVICE_OPTION_NONE,
-                             &run, NULL)) ? 0 : 1;
+  ret =
+    (GNUNET_OK ==
+     GNUNET_SERVICE_run (argc, argv, "resolver", GNUNET_SERVICE_OPTION_NONE,
+                        &run, NULL)) ? 0 : 1;
 
   while (head != NULL)
     {