-licenses, dce, etc
[oweals/gnunet.git] / src / gns / gns_api.c
index ddc8714096bc34e14abfd0c7589193a4ae0ce5e6..94142ea882978aef264e2bb97b8e201b37d8ae87 100644 (file)
@@ -159,6 +159,13 @@ struct PendingMessage
    */
   size_t size;
 
+  /**
+   * This message has been transmitted.  GNUNET_NO if the message is
+   * in the "pending" DLL, GNUNET_YES if it has been transmitted to
+   * the service via the current client connection.
+   */
+  int transmitted;
+
 };
 
 
@@ -287,9 +294,44 @@ reconnect_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 static void
 force_reconnect (struct GNUNET_GNS_Handle *h)
 {
+  struct GNUNET_GNS_ShortenRequest *st;
+  struct GNUNET_GNS_LookupRequest *lh;
+  struct GNUNET_GNS_GetAuthRequest *ga;
+  struct PendingMessage *p;
+
   GNUNET_CLIENT_disconnect (h->client);
   h->client = NULL;
   h->in_receive = GNUNET_NO;
+  for (st = h->shorten_head; NULL != st; st = st->next)
+  {
+    p = (struct PendingMessage*) &st[1];
+    if (GNUNET_NO == p->transmitted)
+      continue;
+    p->transmitted = GNUNET_NO;
+    GNUNET_CONTAINER_DLL_insert (h->pending_head,
+                                h->pending_tail,
+                                p);  
+  }
+  for (lh = h->lookup_head; NULL != lh; lh = lh->next)
+  {
+    p = (struct PendingMessage*) &lh[1];
+    if (GNUNET_NO == p->transmitted)
+      continue;
+    p->transmitted = GNUNET_NO;
+    GNUNET_CONTAINER_DLL_insert (h->pending_head,
+                                h->pending_tail,
+                                p);  
+  }
+  for (ga = h->get_auth_head; NULL != ga; ga = ga->next)
+  {
+    p = (struct PendingMessage*) &ga[1];
+    if (GNUNET_NO == p->transmitted)
+      continue;
+    p->transmitted = GNUNET_NO;
+    GNUNET_CONTAINER_DLL_insert (h->pending_head,
+                                h->pending_tail,
+                                p);  
+  }
   /* FIXME: 1s too long, exponential-backoff, starting at 1ms! (max = 1s might be OK) */
   h->reconnect_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS,
                                                     &reconnect_task,
@@ -300,7 +342,10 @@ force_reconnect (struct GNUNET_GNS_Handle *h)
 /**
  * Transmit the next pending message, called by notify_transmit_ready
  *
- * @param fixme
+ * @param cls the closure
+ * @param size size of pending data
+ * @param buf buffer with pending data
+ * @return size data transmitted
  */
 static size_t
 transmit_pending (void *cls, size_t size, void *buf);
@@ -319,19 +364,23 @@ process_message (void *cls, const struct GNUNET_MessageHeader *msg);
 /**
  * Try to send messages from list of messages to send
  *
- * @param fixme
+ * @param handle the GNS handle
  */
 static void
 process_pending_messages (struct GNUNET_GNS_Handle *handle)
 {
-  struct PendingMessage *p;
+  struct PendingMessage *p = handle->pending_head;
 
   if (NULL == handle->client)
     return; /* wait for reconnect */ 
   if (NULL != handle->th)
     return; /* transmission request already pending */ 
-  if (NULL == (p = handle->pending_head))
-    return; /* no messages pending */  
+
+  while ((NULL != p) && (p->transmitted == GNUNET_YES))
+    p = p->next;
+  if (NULL == p)
+    return; /* no messages pending */
+  
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
              "Trying to transmit %u bytes\n", 
              (unsigned int) p->size);
@@ -348,46 +397,51 @@ process_pending_messages (struct GNUNET_GNS_Handle *handle)
 /**
  * Transmit the next pending message, called by notify_transmit_ready
  *
- * @param fixme
+ * @param cls the closure
+ * @param size size of pending data
+ * @param buf buffer with pending data
+ * @return size data transmitted
  */
 static size_t
 transmit_pending (void *cls, size_t size, void *buf)
 {
-  struct GNUNET_GNS_Handle *handle = cls;
+  struct GNUNET_GNS_Handle *h = cls;
   char *cbuf = buf;
   struct PendingMessage *p;
   size_t tsize;
 
-  handle->th = NULL;  
+  h->th = NULL;  
   if ((0 == size) || (NULL == buf))
   {
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                "Transmission to GNS service failed!\n");
-    force_reconnect(handle);
+    force_reconnect (h);
     return 0;
   }  
-  if (NULL == (p = handle->pending_head))
+  if (NULL == (p = h->pending_head))
     return 0;
 
   tsize = 0;
-  while ((NULL != (p = handle->pending_head)) && (p->size <= size))
+  while ((NULL != (p = h->pending_head)) && (p->size <= size))
   {
     memcpy (&cbuf[tsize], &p[1], p->size);
     tsize += p->size;
     size -= p->size;
-    GNUNET_CONTAINER_DLL_remove (handle->pending_head, handle->pending_tail, p);
-    if (GNUNET_YES != handle->in_receive)
+    p->transmitted = GNUNET_YES;
+    GNUNET_CONTAINER_DLL_remove (h->pending_head,
+                                h->pending_tail,
+                                p);
+    if (GNUNET_YES != h->in_receive)
     {
-      GNUNET_CLIENT_receive (handle->client, &process_message, handle,
+      GNUNET_CLIENT_receive (h->client, &process_message, h,
                              GNUNET_TIME_UNIT_FOREVER_REL);
-      handle->in_receive = GNUNET_YES;
+      h->in_receive = GNUNET_YES;
     }
-    GNUNET_free (p);
   }
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               "Sending %u bytes\n",
              (unsigned int) tsize);
-  process_pending_messages (handle);
+  process_pending_messages (h);
   return tsize;
 }
 
@@ -404,26 +458,37 @@ process_shorten_reply (struct GNUNET_GNS_ShortenRequest *qe,
                        const struct GNUNET_GNS_ClientShortenResultMessage *msg)
 {
   struct GNUNET_GNS_Handle *h = qe->gns_handle;
+  struct PendingMessage *p = (struct PendingMessage *)&qe[1];
   const char *short_name;
   size_t mlen;
-
-  GNUNET_CONTAINER_DLL_remove (h->shorten_head, h->shorten_tail, qe);
-  mlen = ntohs (msg->header.size);
-  short_name = (const char *) &msg[1];
-
-  if ( (ntohs (msg->header.size) <= sizeof (struct GNUNET_GNS_ClientShortenResultMessage)) ||
-       ('\0' != short_name[mlen - sizeof (struct GNUNET_GNS_ClientShortenResultMessage) - 1]) )
+  
+  if (GNUNET_YES != p->transmitted)
   {
+    /* service send reply to query we never managed to send!? */
     GNUNET_break (0);
-    // FIXME: reconnect and queue management logic is broken...
-    qe->shorten_proc (qe->proc_cls, NULL);
-    GNUNET_free (qe);
     force_reconnect (h);
     return;
-  } 
+  }
+  mlen = ntohs (msg->header.size);
+  if (ntohs (msg->header.size) == sizeof (struct GNUNET_GNS_ClientShortenResultMessage))
+  {
+    /* service reports resolution failed */
+    short_name = NULL;
+  }
+  else
+  {
+    short_name = (const char *) &msg[1];
+    if ('\0' != short_name[mlen - sizeof (struct GNUNET_GNS_ClientShortenResultMessage) - 1])
+    {
+      GNUNET_break (0);
+      force_reconnect (h);
+      return;
+    } 
+  }
+  GNUNET_CONTAINER_DLL_remove (h->shorten_head, h->shorten_tail, qe);
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               "Received shortened reply `%s' from GNS service\n",
-              short_name);  
+              short_name);
   qe->shorten_proc (qe->proc_cls, short_name);
   GNUNET_free (qe);
 }
@@ -441,20 +506,33 @@ process_get_auth_reply (struct GNUNET_GNS_GetAuthRequest *qe,
                        const struct GNUNET_GNS_ClientGetAuthResultMessage *msg)
 {
   struct GNUNET_GNS_Handle *h = qe->gns_handle;
+  struct PendingMessage *p = (struct PendingMessage *)&qe[1];
   const char *auth_name;
+  size_t mlen;
 
-  GNUNET_CONTAINER_DLL_remove (h->get_auth_head, h->get_auth_tail, qe);
-  auth_name = (char*)&msg[1];
-
-  if (ntohs (((struct GNUNET_MessageHeader*)msg)->size) <
-      sizeof (struct GNUNET_GNS_ClientGetAuthResultMessage))
+  if (GNUNET_YES != p->transmitted)
   {
-    GNUNET_free(qe);
+    /* service send reply to query we never managed to send!? */
     GNUNET_break (0);
     force_reconnect (h);
     return;
   }
-  
+  mlen = ntohs (msg->header.size);
+  if (mlen == sizeof (struct GNUNET_GNS_ClientGetAuthResultMessage))
+  {
+    auth_name = NULL;
+  }
+  else
+  {
+    auth_name = (const char*) &msg[1];
+    if ('\0' != auth_name[mlen - sizeof (struct GNUNET_GNS_ClientGetAuthResultMessage) - 1])
+    {
+      GNUNET_break (0);
+      force_reconnect (h);
+      return;
+    }
+  }
+  GNUNET_CONTAINER_DLL_remove (h->get_auth_head, h->get_auth_tail, qe);
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               "Received GET_AUTH reply `%s' from GNS service\n",
               auth_name);
@@ -474,41 +552,41 @@ process_lookup_reply (struct GNUNET_GNS_LookupRequest *qe,
                       const struct GNUNET_GNS_ClientLookupResultMessage *msg)
 {
   struct GNUNET_GNS_Handle *h = qe->gns_handle;
-  int rd_count = ntohl(msg->rd_count);
-  size_t len = ntohs (((struct GNUNET_MessageHeader*)msg)->size);
+  struct PendingMessage *p = (struct PendingMessage *) &qe[1];
+  uint32_t rd_count = ntohl (msg->rd_count);
   struct GNUNET_NAMESTORE_RecordData rd[rd_count];
+  size_t mlen;
 
-  GNUNET_CONTAINER_DLL_remove (h->lookup_head, h->lookup_tail, qe);
-
-  if (len < sizeof (struct GNUNET_GNS_ClientLookupResultMessage))
+  if (GNUNET_YES != p->transmitted)
   {
-    GNUNET_free(qe);
+    /* service send reply to query we never managed to send!? */
     GNUNET_break (0);
     force_reconnect (h);
     return;
   }
-
-  len -= sizeof (struct GNUNET_GNS_ClientLookupResultMessage);
-  if (GNUNET_SYSERR == GNUNET_NAMESTORE_records_deserialize (len,
-                                                             (char*)&msg[1],
+  mlen = ntohs (msg->header.size);
+  mlen -= sizeof (struct GNUNET_GNS_ClientLookupResultMessage);
+  if (GNUNET_SYSERR == GNUNET_NAMESTORE_records_deserialize (mlen,
+                                                             (const char*) &msg[1],
                                                              rd_count,
                                                              rd))
   {
     GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                "Failed to serialize lookup reply from GNS service!\n");
+                _("Failed to serialize lookup reply from GNS service!\n"));
     qe->lookup_proc (qe->proc_cls, 0, NULL);
   }
   else
-  {
-  
+  { 
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                "Received lookup reply from GNS service (count=%d)\n",
-                ntohl (msg->rd_count));
+                "Received lookup reply from GNS service (%u records)\n",
+                (unsigned int) rd_count);
     qe->lookup_proc (qe->proc_cls, rd_count, rd);
   }
+  GNUNET_CONTAINER_DLL_remove (h->lookup_head, h->lookup_tail, qe);
   GNUNET_free (qe);
 }
 
+
 /**
  * Handler for messages received from the GNS service
  *
@@ -538,55 +616,55 @@ process_message (void *cls, const struct GNUNET_MessageHeader *msg)
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                 "Got LOOKUP_RESULT msg\n");
     if (ntohs (msg->size) < sizeof (struct GNUNET_GNS_ClientLookupResultMessage))
-      {
-       GNUNET_break (0);
-       force_reconnect (handle);
-       return;
-      }
+    {
+      GNUNET_break (0);
+      force_reconnect (handle);
+      return;
+    }
     lookup_msg = (const struct GNUNET_GNS_ClientLookupResultMessage *) msg;
     r_id = ntohl (lookup_msg->id);   
     for (lr = handle->lookup_head; NULL != lr; lr = lr->next)    
       if (lr->r_id == r_id)
-       {
-         process_lookup_reply(lr, lookup_msg);    
-         break;
-       }
+      {
+       process_lookup_reply(lr, lookup_msg);    
+       break;
+      }
     break;
   case GNUNET_MESSAGE_TYPE_GNS_SHORTEN_RESULT:  
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                 "Got SHORTEN_RESULT msg\n");
     if (ntohs (msg->size) < sizeof (struct GNUNET_GNS_ClientShortenResultMessage))
-      {
-       GNUNET_break (0);
-       force_reconnect (handle);
-       return;
-      }
+    {
+      GNUNET_break (0);
+      force_reconnect (handle);
+      return;
+    }
     shorten_msg = (const struct GNUNET_GNS_ClientShortenResultMessage *) msg;   
     r_id = ntohl (shorten_msg->id);
     for (sr = handle->shorten_head; NULL != sr; sr = sr->next)    
       if (sr->r_id == r_id)
-       {
-         process_shorten_reply (sr, shorten_msg);
-         break;
-       }
+      {
+       process_shorten_reply (sr, shorten_msg);
+       break;
+      }
     break;
-  case GNUNET_MESSAGE_TYPE_GNS_GET_AUTH_RESULT:  
+  case GNUNET_MESSAGE_TYPE_GNS_GET_AUTH_RESULT:
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                 "Got GET_AUTH_RESULT msg\n");
     if (ntohs (msg->size) < sizeof (struct GNUNET_GNS_ClientGetAuthResultMessage))
-      {
-       GNUNET_break (0);
-       force_reconnect (handle);
-       return;
-      }
+    {
+      GNUNET_break (0);
+      force_reconnect (handle);
+      return;
+    }
     get_auth_msg = (const struct GNUNET_GNS_ClientGetAuthResultMessage *) msg;
     r_id = ntohl (get_auth_msg->id);
     for (gar = handle->get_auth_head; NULL != gar; gar = gar->next)
       if (gar->r_id == r_id)
-       {
-         process_get_auth_reply (gar, get_auth_msg);
-         break;
-       }
+      {
+       process_get_auth_reply (gar, get_auth_msg);
+       break;
+      }
     break;
   default:
     GNUNET_break (0);
@@ -633,8 +711,7 @@ GNUNET_GNS_disconnect (struct GNUNET_GNS_Handle *handle)
   GNUNET_assert (NULL == handle->lookup_head);
   GNUNET_assert (NULL == handle->shorten_head);
   GNUNET_assert (NULL == handle->get_auth_head);
-  GNUNET_free(handle);
-  /* disco from GNS */
+  GNUNET_free (handle);
 }
 
 
@@ -646,28 +723,16 @@ GNUNET_GNS_disconnect (struct GNUNET_GNS_Handle *handle)
 void
 GNUNET_GNS_cancel_lookup_request (struct GNUNET_GNS_LookupRequest *lr)
 {
-  struct PendingMessage *tmp;
+  struct PendingMessage *p = (struct PendingMessage*) &lr[1];
 
   GNUNET_assert (NULL != lr->gns_handle); 
-  for (tmp = lr->gns_handle->pending_head;
-       NULL != tmp; tmp = tmp->next)
-  {
-    if (tmp->r_id == lr->r_id)
-      break;
-  }
-
-  if (NULL != tmp)
-  {
+  if (GNUNET_NO == p->transmitted)
     GNUNET_CONTAINER_DLL_remove (lr->gns_handle->pending_head,
                                  lr->gns_handle->pending_tail,
-                                 tmp);
-    GNUNET_free (tmp);
-  }
-
+                                 p);
   GNUNET_CONTAINER_DLL_remove (lr->gns_handle->lookup_head,
                                lr->gns_handle->lookup_tail,
                                lr);
-
   GNUNET_free (lr);
 }
 
@@ -680,28 +745,16 @@ GNUNET_GNS_cancel_lookup_request (struct GNUNET_GNS_LookupRequest *lr)
 void
 GNUNET_GNS_cancel_shorten_request (struct GNUNET_GNS_ShortenRequest *sr)
 {
-  struct PendingMessage *tmp;
+  struct PendingMessage *p = (struct PendingMessage*) &sr[1];
 
   GNUNET_assert (NULL != sr->gns_handle);
-  for (tmp = sr->gns_handle->pending_head;
-       NULL != tmp; tmp = tmp->next)
-  {
-    if (tmp->r_id == sr->r_id)
-      break;
-  }
-
-  if (NULL != tmp)
-  {
+  if (GNUNET_NO == p->transmitted)
     GNUNET_CONTAINER_DLL_remove (sr->gns_handle->pending_head,
                                  sr->gns_handle->pending_tail,
-                                 tmp);
-    GNUNET_free (tmp);
-  }
-
+                                 p);
   GNUNET_CONTAINER_DLL_remove (sr->gns_handle->shorten_head,
                                sr->gns_handle->shorten_tail,
                                sr);
-
   GNUNET_free (sr);
 }
 
@@ -714,28 +767,16 @@ GNUNET_GNS_cancel_shorten_request (struct GNUNET_GNS_ShortenRequest *sr)
 void
 GNUNET_GNS_cancel_get_auth_request (struct GNUNET_GNS_GetAuthRequest *gar)
 {
-  struct PendingMessage *tmp;
+  struct PendingMessage *p = (struct PendingMessage*) &gar[1];
 
   GNUNET_assert (NULL != gar->gns_handle); 
-  for (tmp = gar->gns_handle->pending_head;
-       NULL != tmp; tmp = tmp->next)
-  {
-    if (tmp->r_id == gar->r_id)
-      break;
-  }
-
-  if (NULL != tmp)
-  {
+  if (GNUNET_NO == p->transmitted)
     GNUNET_CONTAINER_DLL_remove (gar->gns_handle->pending_head,
                                  gar->gns_handle->pending_tail,
-                                 tmp);
-    GNUNET_free (tmp);
-  }
-
+                                 p);  
   GNUNET_CONTAINER_DLL_remove (gar->gns_handle->get_auth_head,
                                gar->gns_handle->get_auth_tail,
                                gar);
-
   GNUNET_free (gar);
 }
 
@@ -755,13 +796,13 @@ GNUNET_GNS_cancel_get_auth_request (struct GNUNET_GNS_GetAuthRequest *gar)
  */
 struct GNUNET_GNS_LookupRequest*
 GNUNET_GNS_lookup_zone (struct GNUNET_GNS_Handle *handle,
-                   const char * name,
-                   struct GNUNET_CRYPTO_ShortHashCode *zone,
-                   enum GNUNET_GNS_RecordType type,
-                   int only_cached,
-                   struct GNUNET_CRYPTO_RsaPrivateKey *shorten_key,
-                   GNUNET_GNS_LookupResultProcessor proc,
-                   void *proc_cls)
+                       const char *name,
+                       struct GNUNET_CRYPTO_ShortHashCode *zone,
+                       enum GNUNET_GNS_RecordType type,
+                       int only_cached,
+                       struct GNUNET_CRYPTO_RsaPrivateKey *shorten_key,
+                       GNUNET_GNS_LookupResultProcessor proc,
+                       void *proc_cls)
 {
   /* IPC to shorten gns names, return shorten_handle */
   struct GNUNET_GNS_ClientLookupMessage *lookup_msg;
@@ -776,8 +817,7 @@ GNUNET_GNS_lookup_zone (struct GNUNET_GNS_Handle *handle,
   {
     GNUNET_break (0);
     return NULL;
-  }
-  
+  } 
   if (NULL != shorten_key)
   {
     pkey_enc = GNUNET_CRYPTO_rsa_encode_key (shorten_key);
@@ -789,23 +829,28 @@ GNUNET_GNS_lookup_zone (struct GNUNET_GNS_Handle *handle,
     pkey_enc = NULL;
     key_len = 0;
   }
-
-  msize = sizeof (struct GNUNET_GNS_ClientLookupMessage)
-    + key_len + strlen (name) + 1;
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-             "Trying to lookup %s in GNS\n", 
+             "Trying to lookup `%s' in GNS\n", 
              name);
-  lr = GNUNET_malloc (sizeof (struct GNUNET_GNS_LookupRequest));
+  msize = sizeof (struct GNUNET_GNS_ClientLookupMessage)
+    + key_len + strlen (name) + 1;
+  if (msize > UINT16_MAX)
+  {
+    GNUNET_break (0);
+    GNUNET_free (pkey_enc);
+    return NULL;
+  }
+  lr = GNUNET_malloc (sizeof (struct GNUNET_GNS_LookupRequest) +
+                     sizeof (struct PendingMessage) + msize);
   lr->gns_handle = handle;
   lr->lookup_proc = proc;
   lr->proc_cls = proc_cls;
   lr->r_id = handle->r_id++;
-  GNUNET_CONTAINER_DLL_insert_tail (handle->lookup_head,
-                                    handle->lookup_tail, lr);
-
-  pending = GNUNET_malloc (sizeof (struct PendingMessage) + msize);
+  pending = (struct PendingMessage *)&lr[1];
   pending->size = msize;
   pending->r_id = lr->r_id;
+  GNUNET_CONTAINER_DLL_insert_tail (handle->lookup_head,
+                                    handle->lookup_tail, lr);
 
   lookup_msg = (struct GNUNET_GNS_ClientLookupMessage *) &pending[1];
   lookup_msg->header.type = htons (GNUNET_MESSAGE_TYPE_GNS_LOOKUP);
@@ -854,9 +899,9 @@ GNUNET_GNS_lookup_zone (struct GNUNET_GNS_Handle *handle,
  * @param proc_cls closure for processor
  * @return handle to the lookup request
  */
-struct GNUNET_GNS_LookupRequest*
+struct GNUNET_GNS_LookupRequest *
 GNUNET_GNS_lookup (struct GNUNET_GNS_Handle *handle,
-                   const char * name,
+                   const char *name,
                    enum GNUNET_GNS_RecordType type,
                    int only_cached,
                    struct GNUNET_CRYPTO_RsaPrivateKey *shorten_key,
@@ -885,7 +930,7 @@ GNUNET_GNS_lookup (struct GNUNET_GNS_Handle *handle,
  */
 struct GNUNET_GNS_ShortenRequest*
 GNUNET_GNS_shorten_zone (struct GNUNET_GNS_Handle *handle,
-                         const char * name,
+                         const char *name,
                          struct GNUNET_CRYPTO_ShortHashCode *private_zone,
                          struct GNUNET_CRYPTO_ShortHashCode *shorten_zone,
                          struct GNUNET_CRYPTO_ShortHashCode *zone,
@@ -903,28 +948,31 @@ GNUNET_GNS_shorten_zone (struct GNUNET_GNS_Handle *handle,
     GNUNET_break (0);
     return NULL;
   }
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Trying to shorten %s in GNS\n", name);
-
-  msize = sizeof (struct GNUNET_GNS_ClientShortenMessage) + strlen(name) + 1;
-  sr = GNUNET_malloc (sizeof (struct GNUNET_GNS_ShortenRequest));
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Trying to shorten `%s' in GNS\n", name);
+  msize = sizeof (struct GNUNET_GNS_ClientShortenMessage) + strlen (name) + 1;
+  if (msize > UINT16_MAX)
+  {
+    GNUNET_break (0);
+    return NULL;
+  }
+  sr = GNUNET_malloc (sizeof (struct GNUNET_GNS_ShortenRequest) +
+                     sizeof (struct PendingMessage) + msize);
   sr->gns_handle = handle;
   sr->shorten_proc = proc;
   sr->proc_cls = proc_cls;
   sr->r_id = handle->r_id++;
   GNUNET_CONTAINER_DLL_insert_tail (handle->shorten_head,
                                     handle->shorten_tail, sr);
-
-  pending = GNUNET_malloc (sizeof (struct PendingMessage) + msize);
+  pending = (struct PendingMessage *)&sr[1];
   pending->size = msize;
   pending->r_id = sr->r_id;
 
   shorten_msg = (struct GNUNET_GNS_ClientShortenMessage *) &pending[1];
   shorten_msg->header.type = htons (GNUNET_MESSAGE_TYPE_GNS_SHORTEN);
-  shorten_msg->header.size = htons (msize);
+  shorten_msg->header.size = htons ((uint16_t) msize);
   shorten_msg->id = htonl (sr->r_id);
   shorten_msg->private_zone = *private_zone;
-  shorten_msg->shorten_zone = *shorten_zone;
-  
+  shorten_msg->shorten_zone = *shorten_zone;  
   if (NULL != zone)
   {
     shorten_msg->use_default_zone = htonl (GNUNET_NO);
@@ -935,13 +983,10 @@ GNUNET_GNS_shorten_zone (struct GNUNET_GNS_Handle *handle,
   {
     shorten_msg->use_default_zone = htonl (GNUNET_YES);
     memset (&shorten_msg->zone, 0, sizeof (struct GNUNET_CRYPTO_ShortHashCode));
-  }
-  
+  } 
   memcpy (&shorten_msg[1], name, strlen (name) + 1);
-
   GNUNET_CONTAINER_DLL_insert_tail (handle->pending_head, handle->pending_tail,
-                               pending);
-  
+                               pending);  
   process_pending_messages (handle);
   return sr;
 }
@@ -960,7 +1005,7 @@ GNUNET_GNS_shorten_zone (struct GNUNET_GNS_Handle *handle,
  */
 struct GNUNET_GNS_ShortenRequest*
 GNUNET_GNS_shorten (struct GNUNET_GNS_Handle *handle,
-                    const char * name,
+                    const char *name,
                     struct GNUNET_CRYPTO_ShortHashCode *private_zone,
                     struct GNUNET_CRYPTO_ShortHashCode *shorten_zone,
                     GNUNET_GNS_ShortenResultProcessor proc,
@@ -983,9 +1028,9 @@ GNUNET_GNS_shorten (struct GNUNET_GNS_Handle *handle,
  */
 struct GNUNET_GNS_GetAuthRequest*
 GNUNET_GNS_get_authority (struct GNUNET_GNS_Handle *handle,
-                    const char * name,
-                    GNUNET_GNS_GetAuthResultProcessor proc,
-                    void *proc_cls)
+                         const char *name,
+                         GNUNET_GNS_GetAuthResultProcessor proc,
+                         void *proc_cls)
 {
   struct GNUNET_GNS_ClientGetAuthMessage *get_auth_msg;
   struct GNUNET_GNS_GetAuthRequest *gar;
@@ -999,9 +1044,14 @@ GNUNET_GNS_get_authority (struct GNUNET_GNS_Handle *handle,
   }
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               "Trying to look up authority for %s in GNS\n", name);
-
   msize = sizeof (struct GNUNET_GNS_ClientGetAuthMessage) + strlen (name) + 1;
-  gar = GNUNET_malloc (sizeof (struct GNUNET_GNS_GetAuthRequest));
+  if (msize > UINT16_MAX)
+  {
+    GNUNET_break (0);
+    return NULL;
+  }
+  gar = GNUNET_malloc (sizeof (struct GNUNET_GNS_GetAuthRequest) +
+                      sizeof (struct PendingMessage) + msize);
   gar->gns_handle = handle;
   gar->auth_proc = proc;
   gar->proc_cls = proc_cls;
@@ -1009,15 +1059,14 @@ GNUNET_GNS_get_authority (struct GNUNET_GNS_Handle *handle,
   GNUNET_CONTAINER_DLL_insert_tail (handle->get_auth_head,
                                     handle->get_auth_tail, gar);
 
-  pending = GNUNET_malloc (sizeof (struct PendingMessage) + msize);
+  pending = (struct PendingMessage *) &gar[1];
   pending->size = msize;
   pending->r_id = gar->r_id;
   get_auth_msg = (struct GNUNET_GNS_ClientGetAuthMessage *) &pending[1];
   get_auth_msg->header.type = htons (GNUNET_MESSAGE_TYPE_GNS_GET_AUTH);
   get_auth_msg->header.size = htons (msize);
   get_auth_msg->id = htonl (gar->r_id);
-  memcpy (&get_auth_msg[1], name, strlen(name) + 1);
-
+  memcpy (&get_auth_msg[1], name, strlen (name) + 1);
   GNUNET_CONTAINER_DLL_insert_tail (handle->pending_head, 
                                    handle->pending_tail,
                                    pending);