WiP
[oweals/gnunet.git] / src / util / resolver_api.c
index cbd8392bfff8b212061ed54ff745752c96b808e0..7daaaf1ccea47a1e41897f1fb16e58a270cf33f5 100644 (file)
@@ -25,6 +25,7 @@
  */
 #include "platform.h"
 #include "gnunet_getopt_lib.h"
+#include "gnunet_os_lib.h"
 #include "gnunet_client_lib.h"
 #include "gnunet_protocols.h"
 #include "gnunet_resolver_service.h"
 
 
 /**
- * FIXME.
+ * Maximum supported length for a hostname
  */
-struct GetAddressContext
+#define MAX_HOSTNAME 1024
+
+
+/**
+ * Possible hostnames for "loopback".
+ */
+static const char *loopback[] = {
+  "localhost",
+  "ip6-localnet",
+  NULL
+};
+
+
+/**
+ * Handle to a request given to the resolver.  Can be used to cancel
+ * the request prior to the timeout or successful execution.  Also
+ * used to track our internal state for the request.
+ */
+struct GNUNET_RESOLVER_RequestHandle
 {
 
   /**
-   * FIXME.
+   * Callback if this is an name resolution request,
+   * otherwise NULL.
    */
-  GNUNET_RESOLVER_AddressCallback callback;
+  GNUNET_RESOLVER_AddressCallback addr_callback;
 
   /**
-   * Closure for "callback".
+   * Callback if this is a reverse lookup request,
+   * otherwise NULL.
    */
-  void *cls;
+  GNUNET_RESOLVER_HostnameCallback name_callback;
 
   /**
-   * FIXME.
+   * Closure for the respective "callback".
    */
-  struct GNUNET_RESOLVER_GetMessage *msg;
+  void *cls;
 
   /**
-   * FIXME.
+   * Our connection to the resolver service.
    */
   struct GNUNET_CLIENT_Connection *client;
 
   /**
-   * FIXME.
+   * Name of the host that we are resolving.
+   */
+  const char *hostname;
+
+  /**
+   * When should this request time out?
    */
   struct GNUNET_TIME_Absolute timeout;
-};
 
+  /**
+   * Task handle for numeric lookups.
+   */
+  GNUNET_SCHEDULER_TaskIdentifier task;
+
+  /**
+   * Desired address family.
+   */
+  int domain;
 
-/**
- * Possible hostnames for "loopback".
- */
-static const char *loopback[] = {
-  "localhost",
-  "ip6-localnet",
-  NULL
+  /**
+   * Length of the "struct sockaddr" that follows this
+   * struct (only for reverse lookup).
+   */
+  socklen_t salen;
 };
 
 
@@ -84,44 +116,50 @@ check_config (const struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   char *hostname;
   unsigned int i;
-  struct in_addr v4;
-  struct in6_addr v6;
+  struct sockaddr_in v4;
+  struct sockaddr_in6 v6;
 
+  memset (&v4, 0, sizeof (v4));
+  v4.sin_addr.s_addr = htonl (INADDR_LOOPBACK);
+  v4.sin_family = AF_INET;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  v4.sin_len = sizeof (v4);
+#endif
+  memset (&v6, 0, sizeof (v6));
+  v6.sin6_family = AF_INET6;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  v6.sin6_len = sizeof (v6);
+#endif
   if (GNUNET_OK !=
       GNUNET_CONFIGURATION_get_value_string (cfg,
-                                            "resolver",
-                                            "HOSTNAME",
-                                            &hostname))
+                                             "resolver",
+                                             "HOSTNAME", &hostname))
     {
       GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                 _("Must specify `%s' for `%s' in configuration!\n"),
-                 "HOSTNAME",
-                 "resolver");
+                  _("Must specify `%s' for `%s' in configuration!\n"),
+                  "HOSTNAME", "resolver");
       GNUNET_assert (0);
     }
-  if ( (0 == inet_pton (AF_INET,
-                       hostname,
-                       &v4)) ||
-       (0 == inet_pton (AF_INET6,
-                       hostname,
-                       &v6)) )
+  if ((1 != inet_pton (AF_INET,
+                       hostname,
+                       &v4)) || (1 != inet_pton (AF_INET6, hostname, &v6)))
     {
       GNUNET_free (hostname);
       return;
     }
   i = 0;
   while (loopback[i] != NULL)
-    if (0 == strcmp (loopback[i++], hostname))
+    if (0 == strcasecmp (loopback[i++], hostname))
       {
-       GNUNET_free (hostname); 
-       return;
+        GNUNET_free (hostname);
+        return;
       }
   GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-             _("Must specify `%s' for `%s' in configuration!\n"),
-             "localhost",
-             "resolver");
-  GNUNET_free (hostname); 
-  GNUNET_assert (0); 
+              _
+              ("Must specify `%s' or numeric IP address for `%s' of `%s' in configuration!\n"),
+              "localhost", "HOSTNAME", "resolver");
+  GNUNET_free (hostname);
+  GNUNET_assert (0);
 }
 
 
@@ -146,17 +184,27 @@ no_resolve (const struct sockaddr *sa, socklen_t salen)
     case AF_INET:
       if (salen != sizeof (struct sockaddr_in))
         return NULL;
-      inet_ntop (AF_INET,
-                 &((struct sockaddr_in *) sa)->sin_addr,
-                 inet4, INET_ADDRSTRLEN);
+      if (NULL == 
+         inet_ntop (AF_INET,
+                    &((struct sockaddr_in *) sa)->sin_addr,
+                    inet4, INET_ADDRSTRLEN))
+       {
+         GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "inet_ntop");
+         return NULL;
+       }
       ret = GNUNET_strdup (inet4);
       break;
     case AF_INET6:
       if (salen != sizeof (struct sockaddr_in6))
         return NULL;
-      inet_ntop (AF_INET6,
-                 &((struct sockaddr_in6 *) sa)->sin6_addr,
-                 inet6, INET6_ADDRSTRLEN);
+      if (NULL == 
+         inet_ntop (AF_INET6,
+                    &((struct sockaddr_in6 *) sa)->sin6_addr,
+                    inet6, INET6_ADDRSTRLEN))
+       {
+         GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "inet_ntop");
+         return NULL;
+       }
       ret = GNUNET_strdup (inet6);
       break;
     default:
@@ -168,35 +216,36 @@ no_resolve (const struct sockaddr *sa, socklen_t salen)
 
 
 /**
- * FIXME
+ * Process the reply from the resolver (which is presumably
+ * the numeric IP address for a name resolution request).
  *
- * @param cls FIXME
- * @param msg FIXME
+ * @param cls the "GNUNET_RESOLVER_RequestHandle" for which this is a reply
+ * @param msg reply from the resolver or NULL on error
  */
 static void
 handle_address_response (void *cls, const struct GNUNET_MessageHeader *msg)
 {
-  struct GetAddressContext *gac = cls;
+  struct GNUNET_RESOLVER_RequestHandle *rh = cls;
   uint16_t size;
   const struct sockaddr *sa;
   socklen_t salen;
 
-
   if (msg == NULL)
     {
       GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                  _("Timeout trying to resolve hostname.\n"));
-      gac->callback (gac->cls, NULL, 0);
-      GNUNET_CLIENT_disconnect (gac->client);
-      GNUNET_free (gac);
+                  _("Timeout trying to resolve hostname `%s'.\n"),
+                 rh->hostname);
+      rh->addr_callback (rh->cls, NULL, 0);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
   if (GNUNET_MESSAGE_TYPE_RESOLVER_RESPONSE != ntohs (msg->type))
     {
       GNUNET_break (0);
-      gac->callback (gac->cls, NULL, 0);
-      GNUNET_CLIENT_disconnect (gac->client);
-      GNUNET_free (gac);
+      rh->addr_callback (rh->cls, NULL, 0);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
 
@@ -205,11 +254,12 @@ handle_address_response (void *cls, const struct GNUNET_MessageHeader *msg)
     {
 #if DEBUG_RESOLVER
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  _("Received end message resolving hostname.\n"));
+                  _("Received end message resolving hostname `%s'.\n"),
+                 rh->hostname);
 #endif
-      gac->callback (gac->cls, NULL, 0);
-      GNUNET_CLIENT_disconnect (gac->client);
-      GNUNET_free (gac);
+      rh->addr_callback (rh->cls, NULL, 0);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
   sa = (const struct sockaddr *) &msg[1];
@@ -217,269 +267,254 @@ handle_address_response (void *cls, const struct GNUNET_MessageHeader *msg)
   if (salen < sizeof (struct sockaddr))
     {
       GNUNET_break (0);
-      gac->callback (gac->cls, NULL, 0);
-      GNUNET_CLIENT_disconnect (gac->client);
-      GNUNET_free (gac);
+      rh->addr_callback (rh->cls, NULL, 0);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
 #if DEBUG_RESOLVER
   {
     char *ips = no_resolve (sa, salen);
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, _("Resolver returns `%s'.\n"), ips);
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+               "Resolver returns `%s' for `%s'.\n", ips,
+               rh->hostname);
     GNUNET_free (ips);
   }
 #endif
-  gac->callback (gac->cls, sa, salen);
-  GNUNET_CLIENT_receive (gac->client,
+  rh->addr_callback (rh->cls, sa, salen);
+  GNUNET_CLIENT_receive (rh->client,
                          &handle_address_response,
-                         gac,
-                         GNUNET_TIME_absolute_get_remaining (gac->timeout));
+                         rh,
+                         GNUNET_TIME_absolute_get_remaining (rh->timeout));
 }
 
 
 /**
- * FIXME
+ * We've been asked to lookup the address for a hostname and were 
+ * given a valid numeric string.  Perform the callbacks for the
+ * numeric addresses.
  *
- * @param cls FIXME
- * @param size number of bytes available in buf
- * @param buf target buffer, NULL on error
- * @return number of bytes written to buf
+ * @param cls struct GNUNET_RESOLVER_RequestHandle for the request
+ * @param tc unused scheduler context
  */
-static size_t
-transmit_get_ip (void *cls, size_t size, void *buf)
+static void
+numeric_resolution (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
-  struct GetAddressContext *actx = cls;
-  uint16_t ms;
+  struct GNUNET_RESOLVER_RequestHandle *rh = cls;
+  struct sockaddr_in v4;
+  struct sockaddr_in6 v6;
+
+  memset (&v4, 0, sizeof (v4));
+  v4.sin_family = AF_INET;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  v4.sin_len = sizeof (v4);
+#endif
+  memset (&v6, 0, sizeof (v6));
+  v6.sin6_family = AF_INET6;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  v6.sin6_len = sizeof (v6);
+#endif
 
-  if (buf == NULL)
+  if (((rh->domain == AF_UNSPEC) || (rh->domain == AF_INET)) &&
+      (1 == inet_pton (AF_INET, rh->hostname, &v4.sin_addr)))
+    {
+      rh->addr_callback (rh->cls, (const struct sockaddr *) &v4, sizeof (v4));
+      if ((rh->domain == AF_UNSPEC) &&
+          (1 == inet_pton (AF_INET6, rh->hostname, &v6.sin6_addr)))
+        {
+          /* this can happen on some systems IF "hostname" is "localhost" */
+          rh->addr_callback (rh->cls,
+                             (const struct sockaddr *) &v6, sizeof (v6));
+        }
+      rh->addr_callback (rh->cls, NULL, 0);
+      GNUNET_free (rh);
+      return;
+    }
+  if (((rh->domain == AF_UNSPEC) || (rh->domain == AF_INET6)) &&
+      (1 == inet_pton (AF_INET6, rh->hostname, &v6.sin6_addr)))
     {
-      /* timeout / error */
-      GNUNET_free (actx->msg);
-      actx->callback (actx->cls, NULL, 0);
-      GNUNET_CLIENT_disconnect (actx->client);
-      GNUNET_free (actx);
-      return 0;
+      rh->addr_callback (rh->cls, (const struct sockaddr *) &v6, sizeof (v6));
+      rh->addr_callback (rh->cls, NULL, 0);
+      GNUNET_free (rh);
+      return;
     }
-  ms = ntohs (actx->msg->header.size);
-  GNUNET_assert (size >= ms);
-  memcpy (buf, actx->msg, ms);
-  GNUNET_free (actx->msg);
-  actx->msg = NULL;
-  GNUNET_CLIENT_receive (actx->client,
-                         &handle_address_response,
-                         actx,
-                         GNUNET_TIME_absolute_get_remaining (actx->timeout));
-  return ms;
+  /* why are we here? this task should not have been scheduled! */
+  GNUNET_assert (0);
+  GNUNET_free (rh);
 }
 
 
 
+/**
+ * We've been asked to lookup the address for a hostname and were 
+ * given a variant of "loopback".  Perform the callbacks for the
+ * respective loopback numeric addresses.
+ *
+ * @param cls struct GNUNET_RESOLVER_RequestHandle for the request
+ * @param tc unused scheduler context
+ */
+static void
+loopback_resolution (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
+{
+  struct GNUNET_RESOLVER_RequestHandle *rh = cls;
+  struct sockaddr_in v4;
+  struct sockaddr_in6 v6;
+
+  memset (&v4, 0, sizeof (v4));
+  v4.sin_addr.s_addr = htonl (INADDR_LOOPBACK);
+  v4.sin_family = AF_INET;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  v4.sin_len = sizeof (v4);
+#endif
+  memset (&v6, 0, sizeof (v6));
+  v6.sin6_family = AF_INET6;
+#if HAVE_SOCKADDR_IN_SIN_LEN
+  v6.sin6_len = sizeof (v6);
+#endif
+  v6.sin6_addr = in6addr_loopback;
+  switch (rh->domain)
+    {
+    case AF_INET:
+      rh->addr_callback (rh->cls, (const struct sockaddr *) &v4, sizeof (v4));
+      break;
+    case AF_INET6:
+      rh->addr_callback (rh->cls, (const struct sockaddr *) &v6, sizeof (v6));
+      break;
+    case AF_UNSPEC:
+      rh->addr_callback (rh->cls, (const struct sockaddr *) &v6, sizeof (v6));
+      rh->addr_callback (rh->cls, (const struct sockaddr *) &v4, sizeof (v4));
+      break;
+    default:
+      GNUNET_break (0);
+      break;
+    }
+  rh->addr_callback (rh->cls, NULL, 0);
+  GNUNET_free (rh);
+}
+
+
 /**
  * Convert a string to one or more IP addresses.
  *
- * @param sched scheduler to use
  * @param cfg configuration to use
  * @param hostname the hostname to resolve
  * @param domain AF_INET or AF_INET6; use AF_UNSPEC for "any"
  * @param callback function to call with addresses
  * @param callback_cls closure for callback
  * @param timeout how long to try resolving
+ * @return handle that can be used to cancel the request, NULL on error
  */
-void
-GNUNET_RESOLVER_ip_get (struct GNUNET_SCHEDULER_Handle *sched,
-                        const struct GNUNET_CONFIGURATION_Handle *cfg,
+struct GNUNET_RESOLVER_RequestHandle *
+GNUNET_RESOLVER_ip_get (const struct GNUNET_CONFIGURATION_Handle *cfg,
                         const char *hostname,
                         int domain,
                         struct GNUNET_TIME_Relative timeout,
-                        GNUNET_RESOLVER_AddressCallback callback, 
-                       void *callback_cls)
+                        GNUNET_RESOLVER_AddressCallback callback,
+                        void *callback_cls)
 {
   struct GNUNET_CLIENT_Connection *client;
   struct GNUNET_RESOLVER_GetMessage *msg;
-  struct GetAddressContext *actx;
+  struct GNUNET_RESOLVER_RequestHandle *rh;
   size_t slen;
   unsigned int i;
-  struct sockaddr_in v4;
-  struct sockaddr_in6 v6;
+  struct in_addr v4;
+  struct in6_addr v6;
+  char buf[GNUNET_SERVER_MAX_MESSAGE_SIZE - 1];
 
-  memset (&v4, 0, sizeof(v4));
-  v4.sin_addr.s_addr = htonl (INADDR_LOOPBACK);  
-  v4.sin_family = AF_INET;
-#if HAVE_SOCKADDR_IN_SIN_LEN
-  v4.sin_len = sizeof(v4);
-#endif
-  memset (&v6, 0, sizeof(v6)); 
-  v6.sin6_family = AF_INET6;
-#if HAVE_SOCKADDR_IN_SIN_LEN
-  v6.sin6_len = sizeof(v6);
-#endif
-  /* first, check if this is a numeric address */
-  if ( ( (domain == AF_UNSPEC) || (domain == AF_INET) ) && 
-       (0 == inet_pton (AF_INET,
-                       hostname,
-                       &v4.sin_addr)) )
+  check_config (cfg);
+  slen = strlen (hostname) + 1;
+  if (slen + sizeof (struct GNUNET_RESOLVER_GetMessage) >= GNUNET_SERVER_MAX_MESSAGE_SIZE)
     {
-      callback (callback_cls,
-               (const struct sockaddr*) &v4,
-               sizeof(v4));
-      if ( (domain == AF_UNSPEC) && 
-          (0 == inet_pton (AF_INET6,
-                           hostname,
-                           &v6.sin6_addr)) )
-       {
-         /* this can happen on some systems IF "hostname" is "localhost" */
-         callback (callback_cls,
-                   (const struct sockaddr*) &v6,
-                   sizeof(v6));
-       }
-      callback (callback_cls, NULL, 0);      
-      return;
+      GNUNET_break (0);
+      return NULL;
     }
-  if ( ( (domain == AF_UNSPEC) ||(domain == AF_INET) ) && 
-       (0 == inet_pton (AF_INET6,
-                       hostname,
-                       &v6.sin6_addr)) )
+  rh = GNUNET_malloc (sizeof (struct GNUNET_RESOLVER_RequestHandle) + slen);
+  rh->domain = domain;
+  rh->addr_callback = callback;
+  rh->cls = callback_cls;
+  memcpy (&rh[1], hostname, slen);
+  rh->hostname = (const char *) &rh[1];
+  rh->timeout = GNUNET_TIME_relative_to_absolute (timeout);
+
+  /* first, check if this is a numeric address */
+  if (((1 == inet_pton (AF_INET,
+                        hostname,
+                        &v4)) &&
+       ((domain == AF_INET) || (domain == AF_UNSPEC))) ||
+      ((1 == inet_pton (AF_INET6,
+                        hostname,
+                        &v6)) &&
+       ((domain == AF_INET6) || (domain == AF_UNSPEC))))
     {
-      callback (callback_cls,
-               (const struct sockaddr*) &v6,
-               sizeof(v6));
-      callback (callback_cls, NULL, 0);
-      return;
+      rh->task = GNUNET_SCHEDULER_add_now (&numeric_resolution, rh);
+      return rh;
     }
-  check_config (cfg);
   /* then, check if this is a loopback address */
   i = 0;
   while (loopback[i] != NULL)
-    if (0 == strcmp (loopback[i++], hostname))
+    if (0 == strcasecmp (loopback[i++], hostname))
       {
-       v4.sin_addr.s_addr = htonl (INADDR_LOOPBACK);  
-       v6.sin6_addr = in6addr_loopback;
-       switch (domain)
-         {
-         case AF_INET:
-           callback (callback_cls, 
-                     (const struct sockaddr*) &v4,
-                     sizeof(v4));
-           break;
-         case AF_INET6:
-           callback (callback_cls, 
-                     (const struct sockaddr*) &v6,
-                     sizeof(v6));
-           break;
-         case AF_UNSPEC:
-           callback (callback_cls, 
-                     (const struct sockaddr*) &v6,
-                     sizeof(v6));
-           callback (callback_cls, 
-                     (const struct sockaddr*) &v4,
-                     sizeof(v4));
-           break;
-         }
-       callback (callback_cls, NULL, 0);
-       return;
+        rh->task = GNUNET_SCHEDULER_add_now (&loopback_resolution, rh);
+        return rh;
       }
-  slen = strlen (hostname) + 1;
-  if (slen + sizeof (struct GNUNET_RESOLVER_GetMessage) >
-      GNUNET_SERVER_MAX_MESSAGE_SIZE)
-    {
-      GNUNET_break (0);
-      callback (callback_cls, NULL, 0);
-      return;
-    }
-  client = GNUNET_CLIENT_connect (sched, "resolver", cfg);
+
+  client = GNUNET_CLIENT_connect ("resolver", cfg);
   if (client == NULL)
     {
-      callback (callback_cls, NULL, 0);
-      return;
+      GNUNET_free (rh);
+      return NULL;
     }
-  msg = GNUNET_malloc (sizeof (struct GNUNET_RESOLVER_GetMessage) + slen);
+  rh->client = client;
+
+  msg = (struct GNUNET_RESOLVER_GetMessage *) buf;
   msg->header.size =
     htons (sizeof (struct GNUNET_RESOLVER_GetMessage) + slen);
   msg->header.type = htons (GNUNET_MESSAGE_TYPE_RESOLVER_REQUEST);
   msg->direction = htonl (GNUNET_NO);
   msg->domain = htonl (domain);
   memcpy (&msg[1], hostname, slen);
-  actx = GNUNET_malloc (sizeof (struct GetAddressContext));
-  actx->callback = callback;
-  actx->cls = callback_cls;
-  actx->client = client;
-  actx->timeout = GNUNET_TIME_relative_to_absolute (timeout);
-  actx->msg = msg;
 
 #if DEBUG_RESOLVER
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               _("Resolver requests DNS resolution of hostname `%s'.\n"),
               hostname);
 #endif
-  if (NULL ==
-      GNUNET_CLIENT_notify_transmit_ready (client,
-                                           slen +
-                                           sizeof (struct
-                                                   GNUNET_RESOLVER_GetMessage),
-                                           timeout, &transmit_get_ip, actx))
+  if (GNUNET_OK !=
+      GNUNET_CLIENT_transmit_and_get_response (client,
+                                               &msg->header,
+                                               timeout,
+                                               GNUNET_YES,
+                                               &handle_address_response, rh))
     {
-      GNUNET_free (msg);
-      GNUNET_free (actx);
-      callback (callback_cls, NULL, 0);
-      GNUNET_CLIENT_disconnect (client);
-      return;
+      GNUNET_free (rh);
+      GNUNET_CLIENT_disconnect (client, GNUNET_NO);
+      return NULL;
     }
+  return rh;
 }
 
 
 /**
- * FIXME.
- */
-struct GetHostnameContext
-{
-
-  /**
-   * FIXME.
-   */
-  GNUNET_RESOLVER_HostnameCallback callback;
-  
-  /**
-   * FIXME.
-   */
-  void *cls;
-  
-  /**
-   * FIXME.
-   */ 
-  struct GNUNET_RESOLVER_GetMessage *msg;
-  
-  /**
-   * FIXME.
-   */  
-  struct GNUNET_CLIENT_Connection *client;
-  
-  /**
-   * FIXME.
-   */ 
-  struct GNUNET_TIME_Absolute timeout;
-};
-
-
-/**
- * FIXME.
+ * Process response with a hostname for a reverse DNS lookup.
  *
- * @param cls FIXME
- * @param msg FIXME
+ * @param cls our "struct GNUNET_RESOLVER_RequestHandle" context
+ * @param msg message with the hostname, NULL on error
  */
 static void
 handle_hostname_response (void *cls, const struct GNUNET_MessageHeader *msg)
 {
-  struct GetHostnameContext *ghc = cls;
+  struct GNUNET_RESOLVER_RequestHandle *rh = cls;
   uint16_t size;
   const char *hostname;
 
   if (msg == NULL)
     {
       GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                  _("Timeout trying to resolve IP address.\n"));
-      ghc->callback (ghc->cls, NULL);
-      GNUNET_CLIENT_disconnect (ghc->client);
-      GNUNET_free (ghc);
+                  _("Timeout trying to resolve IP address `%s'.\n"),
+                 GNUNET_a2s ((const void*) &rh[1], rh->salen));
+      rh->name_callback (rh->cls, NULL);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
   size = ntohs (msg->size);
@@ -487,75 +522,69 @@ handle_hostname_response (void *cls, const struct GNUNET_MessageHeader *msg)
     {
 #if DEBUG_RESOLVER
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  _("Received end message resolving IP address.\n"));
+                  _("Received end message resolving IP address `%s'.\n"),
+                 GNUNET_a2s ((const void*) &rh[1], rh->salen));
 #endif
-      ghc->callback (ghc->cls, NULL);
-      GNUNET_CLIENT_disconnect (ghc->client);
-      GNUNET_free (ghc);
+      rh->name_callback (rh->cls, NULL);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
   hostname = (const char *) &msg[1];
   if (hostname[size - sizeof (struct GNUNET_MessageHeader) - 1] != '\0')
     {
       GNUNET_break (0);
-      ghc->callback (ghc->cls, NULL);
-      GNUNET_CLIENT_disconnect (ghc->client);
-      GNUNET_free (ghc);
+      rh->name_callback (rh->cls, NULL);
+      GNUNET_CLIENT_disconnect (rh->client, GNUNET_NO);
+      GNUNET_free (rh);
       return;
     }
 #if DEBUG_RESOLVER
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              _("Resolver returns `%s'.\n"), hostname);
+              _("Resolver returns `%s' for IP `%s'.\n"), 
+             hostname,
+             GNUNET_a2s ((const void*) &rh[1], rh->salen));
 #endif
-  ghc->callback (ghc->cls, hostname);
-  GNUNET_CLIENT_receive (ghc->client,
+  rh->name_callback (rh->cls, hostname);
+  GNUNET_CLIENT_receive (rh->client,
                          &handle_hostname_response,
-                         ghc,
-                         GNUNET_TIME_absolute_get_remaining (ghc->timeout));
+                         rh,
+                         GNUNET_TIME_absolute_get_remaining (rh->timeout));
 }
 
 
+
 /**
- * FIXME
+ * We've been asked to convert an address to a string without
+ * a reverse lookup.  Do it.
  *
- * @param cls FIXME
- * @param size number of bytes available in buf
- * @param buf target buffer, NULL on error
- * @return number of bytes written to buf
+ * @param cls struct GNUNET_RESOLVER_RequestHandle for the request
+ * @param tc unused scheduler context
  */
-static size_t
-transmit_get_hostname (void *cls, size_t size, void *buf)
+static void
+numeric_reverse (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
-  struct GetHostnameContext *hctx = cls;
-  uint16_t msize;
+  struct GNUNET_RESOLVER_RequestHandle *rh = cls;
+  char *result;
 
-  if (buf == NULL)
+  result = no_resolve ((const struct sockaddr *) &rh[1], rh->salen);
+#if DEBUG_RESOLVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, _("Resolver returns `%s'.\n"), result);
+#endif
+  if (result != NULL)
     {
-      GNUNET_free (hctx->msg);
-      hctx->callback (hctx->cls, NULL);
-      GNUNET_CLIENT_disconnect (hctx->client);
-      GNUNET_free (hctx);
-      return 0;
+      rh->name_callback (rh->cls, result);
+      GNUNET_free (result);
     }
-  msize = ntohs (hctx->msg->header.size);
-  GNUNET_assert (size >= msize);
-  memcpy (buf, hctx->msg, msize);
-  GNUNET_free (hctx->msg);
-  hctx->msg = NULL;
-  GNUNET_CLIENT_receive (hctx->client,
-                         &handle_hostname_response,
-                         hctx,
-                         GNUNET_TIME_absolute_get_remaining (hctx->timeout));
-  return msize;
+  rh->name_callback (rh->cls, NULL);
+  GNUNET_free (rh);
 }
 
 
 
-
 /**
  * Get an IP address as a string.
  *
- * @param sched scheduler to use
  * @param cfg configuration to use
  * @param sa host address
  * @param salen length of host address
@@ -563,10 +592,10 @@ transmit_get_hostname (void *cls, size_t size, void *buf)
  * @param timeout how long to try resolving
  * @param callback function to call with hostnames
  * @param cls closure for callback
+ * @return handle that can be used to cancel the request
  */
-void
-GNUNET_RESOLVER_hostname_get (struct GNUNET_SCHEDULER_Handle *sched,
-                              const struct GNUNET_CONFIGURATION_Handle *cfg,
+struct GNUNET_RESOLVER_RequestHandle *
+GNUNET_RESOLVER_hostname_get (const struct GNUNET_CONFIGURATION_Handle *cfg,
                               const struct sockaddr *sa,
                               socklen_t salen,
                               int do_resolve,
@@ -574,41 +603,39 @@ GNUNET_RESOLVER_hostname_get (struct GNUNET_SCHEDULER_Handle *sched,
                               GNUNET_RESOLVER_HostnameCallback callback,
                               void *cls)
 {
-  char *result;
   struct GNUNET_CLIENT_Connection *client;
   struct GNUNET_RESOLVER_GetMessage *msg;
-  struct GetHostnameContext *hctx;
+  struct GNUNET_RESOLVER_RequestHandle *rh;
+  char buf[GNUNET_SERVER_MAX_MESSAGE_SIZE - 1];
 
   check_config (cfg);
+  rh = GNUNET_malloc (sizeof (struct GNUNET_RESOLVER_RequestHandle) + salen);
+  rh->name_callback = callback;
+  rh->cls = cls;
+  rh->timeout = GNUNET_TIME_relative_to_absolute (timeout);
+  rh->salen = salen;
+  memcpy (&rh[1], sa, salen);
+
   if (GNUNET_NO == do_resolve)
     {
-      result = no_resolve (sa, salen);
-#if DEBUG_RESOLVER
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  _("Resolver returns `%s'.\n"), result);
-#endif
-      callback (cls, result);
-      if (result != NULL)
-        {
-          GNUNET_free (result);
-          callback (cls, NULL);
-        }
-      return;
+      rh->task = GNUNET_SCHEDULER_add_now (&numeric_reverse, rh);
+      return rh;
     }
-  if (salen + sizeof (struct GNUNET_RESOLVER_GetMessage) >
-      GNUNET_SERVER_MAX_MESSAGE_SIZE)
+  if (salen + sizeof (struct GNUNET_RESOLVER_GetMessage) >= GNUNET_SERVER_MAX_MESSAGE_SIZE)
     {
       GNUNET_break (0);
-      callback (cls, NULL);
-      return;
+      GNUNET_free (rh);
+      return NULL;
     }
-  client = GNUNET_CLIENT_connect (sched, "resolver", cfg);
+  client = GNUNET_CLIENT_connect ("resolver", cfg);
   if (client == NULL)
     {
-      callback (cls, NULL);
-      return;
+      GNUNET_free (rh);
+      return NULL;
     }
-  msg = GNUNET_malloc (sizeof (struct GNUNET_RESOLVER_GetMessage) + salen);
+  rh->client = client;
+
+  msg = (struct GNUNET_RESOLVER_GetMessage *) buf;
   msg->header.size =
     htons (sizeof (struct GNUNET_RESOLVER_GetMessage) + salen);
   msg->header.type = htons (GNUNET_MESSAGE_TYPE_RESOLVER_REQUEST);
@@ -619,69 +646,106 @@ GNUNET_RESOLVER_hostname_get (struct GNUNET_SCHEDULER_Handle *sched,
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               _("Resolver requests DNS resolution of IP address.\n"));
 #endif
-  hctx = GNUNET_malloc (sizeof (struct GetHostnameContext));
-  hctx->callback = callback;
-  hctx->cls = cls;
-  hctx->client = client;
-  hctx->timeout = GNUNET_TIME_relative_to_absolute (timeout);
-  hctx->msg = msg;
-  if (NULL ==
-      GNUNET_CLIENT_notify_transmit_ready (client,
-                                           sizeof (struct
-                                                   GNUNET_RESOLVER_GetMessage)
-                                           + salen, timeout,
-                                           &transmit_get_hostname, hctx))
+  if (GNUNET_OK !=
+      GNUNET_CLIENT_transmit_and_get_response (client,
+                                               &msg->header,
+                                               timeout,
+                                               GNUNET_YES,
+                                               &handle_hostname_response, rh))
     {
-      GNUNET_free (msg);
-      callback (cls, NULL);
-      GNUNET_CLIENT_disconnect (client);
-      GNUNET_free (hctx);
+      GNUNET_CLIENT_disconnect (client, GNUNET_NO);
+      GNUNET_free (rh);
+      return NULL;
     }
+  return rh;
 }
 
+
 /**
- * Maximum supported length of hostname
+ * Get local fully qualified domain name
+ * @return fqdn
  */
-#define MAX_HOSTNAME 1024
+char *
+GNUNET_RESOLVER_local_fqdn_get ( void )
+{
+  struct hostent *host;
+  char hostname[GNUNET_OS_get_hostname_max_length() + 1];
+
 
+  if (0 != gethostname (hostname, sizeof (hostname) - 1))
+    {
+      GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR |
+                           GNUNET_ERROR_TYPE_BULK, "gethostname");
+      return NULL;
+    }
+#if DEBUG_RESOLVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              _("Resolving our FQDN `%s'\n"), hostname);
+#endif
+  host = gethostbyname ( hostname );
+  if ( NULL == host)
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                _("Could not resolve our FQDN : %s\n"),
+                hstrerror (h_errno));
+    return NULL;
+  }
+  return GNUNET_strdup (host->h_name);
+}
 
 /**
- * Resolve our hostname to an IP address.
+ * Looking our own hostname.
  *
- * @param sched scheduler to use
  * @param cfg configuration to use
  * @param domain AF_INET or AF_INET6; use AF_UNSPEC for "any"
  * @param callback function to call with addresses
  * @param cls closure for callback
  * @param timeout how long to try resolving
+ * @return handle that can be used to cancel the request, NULL on error
  */
-void
-GNUNET_RESOLVER_hostname_resolve (struct GNUNET_SCHEDULER_Handle *sched,
-                                  const struct GNUNET_CONFIGURATION_Handle *cfg,
-                                  int domain,
+struct GNUNET_RESOLVER_RequestHandle *
+GNUNET_RESOLVER_hostname_resolve (const struct GNUNET_CONFIGURATION_Handle
+                                  *cfg, int domain,
                                   struct GNUNET_TIME_Relative timeout,
                                   GNUNET_RESOLVER_AddressCallback callback,
                                   void *cls)
 {
-  char hostname[MAX_HOSTNAME];
+  char hostname[GNUNET_OS_get_hostname_max_length() + 1];
 
   check_config (cfg);
   if (0 != gethostname (hostname, sizeof (hostname) - 1))
     {
       GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR |
                            GNUNET_ERROR_TYPE_BULK, "gethostname");
-      callback (cls, NULL, 0);
-      return;
+      return NULL;
     }
 #if DEBUG_RESOLVER
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               _("Resolving our hostname `%s'\n"), hostname);
 #endif
-  GNUNET_RESOLVER_ip_get (sched,
-                          cfg, hostname, domain, timeout, callback, cls);
+  return GNUNET_RESOLVER_ip_get (cfg, hostname, domain, timeout, callback,
+                                 cls);
 }
 
 
+/**
+ * Cancel a request that is still pending with the resolver.
+ * Note that a client MUST NOT cancel a request that has
+ * been completed (i.e, the callback has been called to
+ * signal timeout or the final result).
+ *
+ * @param h handle of request to cancel
+ */
+void
+GNUNET_RESOLVER_request_cancel (struct GNUNET_RESOLVER_RequestHandle *h)
+{
+  if (h->client != NULL)
+    GNUNET_CLIENT_disconnect (h->client, GNUNET_NO);
+  if (h->task != GNUNET_SCHEDULER_NO_TASK)
+    GNUNET_SCHEDULER_cancel (h->task);
+  GNUNET_free (h);
+}
+
 
 
 /* end of resolver_api.c */