-never store NICKs anywhere but in '+', do not display nicks in gnunet-namestore
[oweals/gnunet.git] / src / namestore / namestore_api.c
index 9cdc70454be0e8400eaaa2d0f148dedd030880b0..c089929a64c40652ce979d45063c275e2e6d7fdb 100644 (file)
@@ -1,6 +1,6 @@
 /*
      This file is part of GNUnet.
-     (C) 2009, 2010 Christian Grothoff (and other contributing authors)
+     (C) 2010-2013 Christian Grothoff (and other contributing authors)
 
      GNUnet is free software; you can redistribute it and/or modify
      it under the terms of the GNU General Public License as published
@@ -23,6 +23,7 @@
  * @brief API to access the NAMESTORE service
  * @author Martin Schanzenbach
  * @author Matthias Wachs
+ * @author Christian Grothoff
  */
 
 #include "platform.h"
 #include "gnunet_dnsparser_lib.h"
 #include "gnunet_arm_service.h"
 #include "gnunet_signatures.h"
+#include "gnunet_gns_service.h"
 #include "gnunet_namestore_service.h"
 #include "namestore.h"
 
-#define DEBUG_GNS_API GNUNET_EXTRA_LOGGING
 
-#define LOG(kind,...) GNUNET_log_from (kind, "gns-api",__VA_ARGS__)
+#define LOG(kind,...) GNUNET_log_from (kind, "namestore-api",__VA_ARGS__)
+
 
 /**
- * A QueueEntry.
+ * An QueueEntry used to store information for a pending
+ * NAMESTORE record operation
  */
 struct GNUNET_NAMESTORE_QueueEntry
 {
@@ -55,22 +58,41 @@ struct GNUNET_NAMESTORE_QueueEntry
    */
   struct GNUNET_NAMESTORE_QueueEntry *prev;
 
+  /**
+   * Main handle to access the namestore.
+   */
   struct GNUNET_NAMESTORE_Handle *nsh;
 
-  uint32_t op_id;
-
+  /**
+   * Continuation to call
+   */
   GNUNET_NAMESTORE_ContinuationWithStatus cont;
+
+  /**
+   * Closure for 'cont'.
+   */
   void *cont_cls;
 
-  GNUNET_NAMESTORE_RecordProcessor proc;
+  /**
+   * Function to call with the records we get back; or NULL.
+   */
+  GNUNET_NAMESTORE_RecordMonitor proc;
+
+  /**
+   * Closure for @e proc.
+   */
   void *proc_cls;
 
-  char *data; /*stub data pointer*/
+  /**
+   * The operation id this zone iteration operation has
+   */
+  uint32_t op_id;
+
 };
 
 
 /**
- * Zone iterator
+ * Handle for a zone iterator operation
  */
 struct GNUNET_NAMESTORE_ZoneIterator
 {
@@ -85,15 +107,31 @@ struct GNUNET_NAMESTORE_ZoneIterator
    */
   struct GNUNET_NAMESTORE_ZoneIterator *prev;
 
-  uint32_t op_id;
-
+  /**
+   * Main handle to access the namestore.
+   */
   struct GNUNET_NAMESTORE_Handle *h;
-  GNUNET_NAMESTORE_RecordProcessor proc;
+
+  /**
+   * The continuation to call with the results
+   */
+  GNUNET_NAMESTORE_RecordMonitor proc;
+
+  /**
+   * Closure for @e proc.
+   */
   void* proc_cls;
-  GNUNET_HashCode zone;
-  uint32_t no_flags;
-  uint32_t flags;
-  int has_zone;
+
+  /**
+   * Private key of the zone.
+   */
+  struct GNUNET_CRYPTO_EcdsaPrivateKey zone;
+
+  /**
+   * The operation id this zone iteration operation has
+   */
+  uint32_t op_id;
+
 };
 
 
@@ -119,10 +157,6 @@ struct PendingMessage
    */
   size_t size;
 
-  /**
-   * Is this the 'START' message?
-   */
-  int is_init;
 };
 
 
@@ -148,59 +182,61 @@ struct GNUNET_NAMESTORE_Handle
   struct GNUNET_CLIENT_TransmitHandle *th;
 
   /**
-   * Reconnect task
+   * Head of linked list of pending messages to send to the service
    */
-  GNUNET_SCHEDULER_TaskIdentifier reconnect_task;
+  struct PendingMessage *pending_head;
 
   /**
-   * Pending messages to send to the service
+   * Tail of linked list of pending messages to send to the service
    */
+  struct PendingMessage *pending_tail;
 
-  struct PendingMessage * pending_head;
-  struct PendingMessage * pending_tail;
+  /**
+   * Head of pending namestore queue entries
+   */
+  struct GNUNET_NAMESTORE_QueueEntry *op_head;
 
   /**
-   * Should we reconnect to service due to some serious error?
+   * Tail of pending namestore queue entries
    */
-  int reconnect;
+  struct GNUNET_NAMESTORE_QueueEntry *op_tail;
 
+  /**
+   * Head of pending namestore zone iterator entries
+   */
+  struct GNUNET_NAMESTORE_ZoneIterator *z_head;
 
   /**
-   * Pending namestore queue entries
+   * Tail of pending namestore zone iterator entries
    */
-  struct GNUNET_NAMESTORE_QueueEntry * op_head;
-  struct GNUNET_NAMESTORE_QueueEntry * op_tail;
+  struct GNUNET_NAMESTORE_ZoneIterator *z_tail;
 
-  uint32_t op_id;
+  /**
+   * Reconnect task
+   */
+  GNUNET_SCHEDULER_TaskIdentifier reconnect_task;
 
   /**
-   * Pending namestore zone iterator entries
+   * Delay introduced before we reconnect.
    */
-  struct GNUNET_NAMESTORE_ZoneIterator * z_head;
-  struct GNUNET_NAMESTORE_ZoneIterator * z_tail;
-};
+  struct GNUNET_TIME_Relative reconnect_delay;
 
-struct GNUNET_NAMESTORE_SimpleRecord
-{
   /**
-   * DLL
+   * Should we reconnect to service due to some serious error?
    */
-  struct GNUNET_NAMESTORE_SimpleRecord *next;
+  int reconnect;
 
   /**
-   * DLL
+   * Did we start to receive yet?
    */
-  struct GNUNET_NAMESTORE_SimpleRecord *prev;
-  
-  const char *name;
-  const GNUNET_HashCode *zone;
-  uint32_t record_type;
-  struct GNUNET_TIME_Absolute expiration;
-  enum GNUNET_NAMESTORE_RecordFlags flags;
-  size_t data_size;
-  const void *data;
-};
+  int is_receiving;
 
+  /**
+   * The last operation id used for a NAMESTORE operation
+   */
+  uint32_t last_op_id_used;
+
+};
 
 
 /**
@@ -211,504 +247,519 @@ struct GNUNET_NAMESTORE_SimpleRecord
 static void
 force_reconnect (struct GNUNET_NAMESTORE_Handle *h);
 
-static void
-handle_lookup_name_response (struct GNUNET_NAMESTORE_QueueEntry *qe,
-                             struct LookupNameResponseMessage * msg,
-                             size_t size)
+
+/**
+ * Handle an incoming message of type
+ * #GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_STORE_RESPONSE
+ *
+ * @param qe the respective entry in the message queue
+ * @param msg the message we received
+ * @param size the message size
+ * @return #GNUNET_OK on success, #GNUNET_SYSERR on error and we did NOT notify the client
+ */
+static int
+handle_record_store_response (struct GNUNET_NAMESTORE_QueueEntry *qe,
+                             const struct RecordStoreResponseMessage* msg,
+                             size_t size)
 {
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received `%s' \n",
-              "LOOKUP_NAME_RESPONSE");
+  int res;
+  const char *emsg;
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received `%s' with result %i\n",
+       "RECORD_STORE_RESPONSE",
+       ntohl (msg->op_result));
+  /* TODO: add actual error message from namestore to response... */
+  res = ntohl (msg->op_result);
+  if (GNUNET_SYSERR == res)
+    emsg = _("Namestore failed to store record\n");
+  else
+    emsg = NULL;
+  if (NULL != qe->cont)
+    qe->cont (qe->cont_cls, res, emsg);
+  return GNUNET_OK;
+}
 
-  struct GNUNET_NAMESTORE_Handle *h = qe->nsh;
-  char *name;
-  char * rd_tmp;
 
-  struct GNUNET_CRYPTO_RsaSignature *signature = NULL;
-  struct GNUNET_TIME_Absolute expire;
-  struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded *public_key_tmp;
+/**
+ * Handle an incoming message of type
+ * #GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_LOOKUP_RESPONSE
+ *
+ * @param qe the respective entry in the message queue
+ * @param msg the message we received
+ * @param size the message size
+ * @return #GNUNET_OK on success, #GNUNET_SYSERR on error and we did NOT notify the client
+ */
+static int
+handle_lookup_result (struct GNUNET_NAMESTORE_QueueEntry *qe,
+                      const struct LabelLookupResponseMessage *msg,
+                      size_t size)
+{
+  const char *name;
+  const char *rd_tmp;
   size_t exp_msg_len;
-  size_t msg_len = 0;
-  size_t name_len = 0;
-  size_t rd_len = 0;
-  int contains_sig = GNUNET_NO;
-  int rd_count = 0;
+  size_t msg_len;
+  size_t name_len;
+  size_t rd_len;
+  unsigned int rd_count;
+  int found;
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received `%s'\n",
+       "RECORD_LOOKUP_RESULT");
 
   rd_len = ntohs (msg->rd_len);
   rd_count = ntohs (msg->rd_count);
   msg_len = ntohs (msg->gns_header.header.size);
   name_len = ntohs (msg->name_len);
-  contains_sig = ntohs (msg->contains_sig);
-  expire = GNUNET_TIME_absolute_ntoh(msg->expire);
-
-  exp_msg_len = sizeof (struct LookupNameResponseMessage) +
-      sizeof (struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded) +
-      name_len + rd_len;
-
+  found = ntohs (msg->found);
+  exp_msg_len = sizeof (struct LabelLookupResponseMessage) + name_len + rd_len;
   if (msg_len != exp_msg_len)
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Message size describes with `%u' bytes but calculated size is %u bytes \n",
-                msg_len, exp_msg_len);
-    GNUNET_break_op (0);
-    return;
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-
-  name = (char *) &msg[1];
-  rd_tmp = &name[name_len];
-
-  /* deserialize records */
-  struct GNUNET_NAMESTORE_RecordData rd[rd_count];
-  GNUNET_NAMESTORE_records_deserialize(rd_len, rd_tmp, rd_count, rd);
-
-  /* reset values if values not contained */
-  if (contains_sig == GNUNET_NO)
-    signature = NULL;
-  else
-    signature = &msg->signature;
-  if (name_len == 0)
-    name = NULL;
-
-  if (name != NULL)
-      public_key_tmp =  &msg->public_key;
-  else
-      public_key_tmp = NULL;
-
-  if (qe->proc != NULL)
+  name = (const char *) &msg[1];
+  if ( (name_len > 0) &&
+       ('\0' != name[name_len -1]) )
   {
-    qe->proc (qe->proc_cls, public_key_tmp, expire, name, rd_count, (rd_count > 0) ? rd : NULL, signature);
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-
-  /* Operation done, remove */
-  GNUNET_CONTAINER_DLL_remove(h->op_head, h->op_tail, qe);
-  GNUNET_free (qe);
-}
-
-
-static void
-handle_record_put_response (struct GNUNET_NAMESTORE_QueueEntry *qe,
-                             struct RecordPutResponseMessage* msg,
-                             size_t size)
-{
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received `%s' \n",
-              "RECORD_PUT_RESPONSE");
-  struct GNUNET_NAMESTORE_Handle *h = qe->nsh;
-  int res = ntohl (msg->op_result);
-
-  if (res == GNUNET_OK)
+  if (GNUNET_NO == found)
   {
-    if (qe->cont != NULL)
-    {
-      qe->cont (qe->cont_cls, res, _("Namestore added record successfully"));
-    }
-
+    /* label was not in namestore */
+    if (NULL != qe->proc)
+      qe->proc (qe->proc_cls,
+                &msg->private_key,
+                name,
+                0, NULL);
+    return GNUNET_OK;
   }
-  else if (res == GNUNET_SYSERR)
+
+  rd_tmp = &name[name_len];
   {
-    if (qe->cont != NULL)
+    struct GNUNET_GNSRECORD_Data rd[rd_count];
+
+    if (GNUNET_OK != GNUNET_GNSRECORD_records_deserialize(rd_len, rd_tmp, rd_count, rd))
     {
-      qe->cont (qe->cont_cls, res, _("Namestore failed to add record"));
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
     }
+    if (0 == name_len)
+      name = NULL;
+    if (NULL != qe->proc)
+      qe->proc (qe->proc_cls,
+                &msg->private_key,
+                name,
+                rd_count,
+                (rd_count > 0) ? rd : NULL);
   }
-  else
-  {
-    GNUNET_break_op (0);
-    return;
-  }
-
-  /* Operation done, remove */
-  GNUNET_CONTAINER_DLL_remove(h->op_head, h->op_tail, qe);
-
-  GNUNET_free (qe);
+  return GNUNET_OK;
 }
 
 
-static void
-handle_record_create_response (struct GNUNET_NAMESTORE_QueueEntry *qe,
-                             struct RecordCreateResponseMessage* msg,
-                             size_t size)
+/**
+ * Handle an incoming message of type
+ * #GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_RESULT
+ *
+ * @param qe the respective entry in the message queue
+ * @param msg the message we received
+ * @param size the message size
+ * @return #GNUNET_OK on success, #GNUNET_SYSERR on error and we did NOT notify the client
+ */
+static int
+handle_record_result (struct GNUNET_NAMESTORE_QueueEntry *qe,
+                     const struct RecordResultMessage *msg,
+                     size_t size)
 {
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received `%s' \n",
-              "RECORD_CREATE_RESPONSE");
-  struct GNUNET_NAMESTORE_Handle *h = qe->nsh;
-  int res = ntohl (msg->op_result);
+  const char *name;
+  const char *rd_tmp;
+  size_t exp_msg_len;
+  size_t msg_len;
+  size_t name_len;
+  size_t rd_len;
+  unsigned int rd_count;
 
-  if (res == GNUNET_YES)
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received `%s'\n",
+       "RECORD_RESULT");
+  rd_len = ntohs (msg->rd_len);
+  rd_count = ntohs (msg->rd_count);
+  msg_len = ntohs (msg->gns_header.header.size);
+  name_len = ntohs (msg->name_len);
+  GNUNET_break (0 == ntohs (msg->reserved));
+  exp_msg_len = sizeof (struct RecordResultMessage) + name_len + rd_len;
+  if (msg_len != exp_msg_len)
   {
-    if (qe->cont != NULL)
-    {
-      qe->cont (qe->cont_cls, res, _("Namestore added record successfully"));
-    }
-
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-  else if (res == GNUNET_NO)
+  name = (const char *) &msg[1];
+  if ( (name_len > 0) &&
+       ('\0' != name[name_len -1]) )
   {
-    if (qe->cont != NULL)
-    {
-      qe->cont (qe->cont_cls, res, _("Namestore record already existed"));
-    }
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-  else
+  rd_tmp = &name[name_len];
   {
-    if (qe->cont != NULL)
+    struct GNUNET_GNSRECORD_Data rd[rd_count];
+
+    if (GNUNET_OK != GNUNET_GNSRECORD_records_deserialize(rd_len, rd_tmp, rd_count, rd))
     {
-      qe->cont (qe->cont_cls, GNUNET_SYSERR, _("Namestore failed to add record\n"));
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
     }
+    if (0 == name_len)
+      name = NULL;
+    if (NULL != qe->proc)
+      qe->proc (qe->proc_cls,
+               &msg->private_key,
+               name,
+               rd_count,
+               (rd_count > 0) ? rd : NULL);
   }
-
-  /* Operation done, remove */
-  GNUNET_CONTAINER_DLL_remove(h->op_head, h->op_tail, qe);
-
-  GNUNET_free (qe);
+  return GNUNET_OK;
 }
 
 
-static void
-handle_record_remove_response (struct GNUNET_NAMESTORE_QueueEntry *qe,
-                             struct RecordRemoveResponseMessage* msg,
-                             size_t size)
-{
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received `%s' \n",
-              "RECORD_REMOVE_RESPONSE");
-
-  struct GNUNET_NAMESTORE_Handle *h = qe->nsh;
-  int res = ntohl (msg->op_result);
-
-  /**
-   *  result:
-   *  0 : successful
-   *  1 : No records for entry
-   *  2 : Could not find record to remove
-   *  3 : Failed to create new signature
-   *  4 : Failed to put new set of records in database
-   */
-  switch (res) {
-    case 0:
-      if (qe->cont != NULL)
-      {
-        qe->cont (qe->cont_cls, GNUNET_YES, _("Namestore removed record successfully"));
-      }
-
-      break;
-    case 1:
-      if (qe->cont != NULL)
-      {
-        qe->cont (qe->cont_cls, GNUNET_NO, _("No records for entry"));
-      }
-
-      break;
-    case 2:
-      if (qe->cont != NULL)
-      {
-        qe->cont (qe->cont_cls, GNUNET_NO, _("Could not find record to remove"));
-      }
-
-      break;
-    case 3:
-      if (qe->cont != NULL)
-      {
-        qe->cont (qe->cont_cls, GNUNET_SYSERR, _("Failed to create new signature"));
-      }
-
-      break;
-    case 4:
-      if (qe->cont != NULL)
-      {
-        qe->cont (qe->cont_cls, GNUNET_SYSERR, _("Failed to put new set of records in database"));
-      }
-      break;
-    default:
-        GNUNET_break_op (0);
-      break;
-  }
-
-  /* Operation done, remove */
-  GNUNET_CONTAINER_DLL_remove(h->op_head, h->op_tail, qe);
-
-  GNUNET_free (qe);
-}
-
-static void
+/**
+ * Handle an incoming message of type
+ * #GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_TO_NAME_RESPONSE.
+ *
+ * @param qe the respective entry in the message queue
+ * @param msg the message we received
+ * @param size the message size
+ * @return #GNUNET_OK on success, #GNUNET_NO if we notified the client about
+ *         the error, #GNUNET_SYSERR on error and we did NOT notify the client
+ */
+static int
 handle_zone_to_name_response (struct GNUNET_NAMESTORE_QueueEntry *qe,
-                             struct ZoneToNameResponseMessage* msg,
-                             size_t size)
+                             const struct ZoneToNameResponseMessage *msg,
+                             size_t size)
 {
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received `%s' \n",
-              "ZONE_TO_NAME_RESPONSE");
-
-  struct GNUNET_NAMESTORE_Handle *h = qe->nsh;
-  int res = ntohs (msg->res);
-
-  struct GNUNET_TIME_Absolute expire;
+  int res;
   size_t name_len;
   size_t rd_ser_len;
   unsigned int rd_count;
-
-  char * name_tmp;
-  char * rd_tmp;
-
-  if (res == GNUNET_SYSERR)
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "An error occured during zone to name operation\n");
-    if (qe->proc != NULL)
-      qe->proc (qe->proc_cls, NULL, GNUNET_TIME_absolute_get_zero(), NULL, 0, NULL, NULL);
-  }
-  else if (res == GNUNET_NO)
-  {
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Namestore has no result for zone to name mapping \n");
-      if (qe->proc != NULL)
-        qe->proc (qe->proc_cls, NULL, GNUNET_TIME_absolute_get_zero(), NULL, 0, NULL, NULL);
-  }
-  else if (res == GNUNET_YES)
+  const char *name_tmp;
+  const char *rd_tmp;
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received `%s'\n",
+       "ZONE_TO_NAME_RESPONSE");
+  res = ntohs (msg->res);
+  switch (res)
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Namestore has result for zone to name mapping \n");
-
+  case GNUNET_SYSERR:
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+        "An error occured during zone to name operation\n");
+    break;
+  case GNUNET_NO:
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+        "Namestore has no result for zone to name mapping \n");
+    if (NULL != qe->proc)
+      qe->proc (qe->proc_cls, &msg->zone, NULL, 0, NULL);
+    return GNUNET_NO;
+  case GNUNET_YES:
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+        "Namestore has result for zone to name mapping \n");
     name_len = ntohs (msg->name_len);
     rd_count = ntohs (msg->rd_count);
     rd_ser_len = ntohs (msg->rd_len);
-    expire = GNUNET_TIME_absolute_ntoh(msg->expire);
-
-    name_tmp = (char *) &msg[1];
+    name_tmp = (const char *) &msg[1];
+    if ( (name_len > 0) &&
+        ('\0' != name_tmp[name_len -1]) )
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
     rd_tmp = &name_tmp[name_len];
+    {
+      struct GNUNET_GNSRECORD_Data rd[rd_count];
 
-    struct GNUNET_NAMESTORE_RecordData rd[rd_count];
-    GNUNET_NAMESTORE_records_deserialize(rd_ser_len, rd_tmp, rd_count, rd);
-
-    if (qe->proc != NULL)
-      qe->proc (qe->proc_cls, &msg->zone_key, expire, name_tmp, rd_count, rd, &msg->signature);
+      if (GNUNET_OK != GNUNET_GNSRECORD_records_deserialize(rd_ser_len, rd_tmp, rd_count, rd))
+      {
+       GNUNET_break (0);
+       return GNUNET_SYSERR;
+      }
+      /* normal end, call continuation with result */
+      if (NULL != qe->proc)
+       qe->proc (qe->proc_cls,
+                 &msg->zone,
+                 name_tmp,
+                 rd_count, rd);
+      /* return is important here: break would call continuation with error! */
+      return GNUNET_OK;
+    }
+  default:
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-  else
-    GNUNET_break_op (0);
-
-  /* Operation done, remove */
-  GNUNET_CONTAINER_DLL_remove(h->op_head, h->op_tail, qe);
-  GNUNET_free (qe);
+  /* error case, call continuation with error */
+  if (NULL != qe->proc)
+    qe->proc (qe->proc_cls, NULL, NULL, 0, NULL);
+  return GNUNET_NO;
 }
 
 
-static void
+/**
+ * Handle incoming messages for record operations
+ *
+ * @param qe the respective zone iteration handle
+ * @param msg the message we received
+ * @param type the message type in host byte order
+ * @param size the message size
+ * @return #GNUNET_OK on success, #GNUNET_NO if we notified the client about
+ *         the error, #GNUNET_SYSERR on error and we did NOT notify the client
+ */
+static int
 manage_record_operations (struct GNUNET_NAMESTORE_QueueEntry *qe,
                           const struct GNUNET_MessageHeader *msg,
-                          int type, size_t size)
+                          uint16_t type,
+                         size_t size)
 {
-
   /* handle different message type */
-  switch (type) {
-    case GNUNET_MESSAGE_TYPE_NAMESTORE_LOOKUP_NAME_RESPONSE:
-        if (size < sizeof (struct LookupNameResponseMessage))
-        {
-          GNUNET_break_op (0);
-          break;
-        }
-        handle_lookup_name_response (qe, (struct LookupNameResponseMessage *) msg, size);
-      break;
-    case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_PUT_RESPONSE:
-        if (size != sizeof (struct RecordPutResponseMessage))
-        {
-          GNUNET_break_op (0);
-          break;
-        }
-        handle_record_put_response (qe, (struct RecordPutResponseMessage *) msg, size);
-      break;
-    case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_CREATE_RESPONSE:
-        if (size != sizeof (struct RecordCreateResponseMessage))
-        {
-          GNUNET_break_op (0);
-          break;
-        }
-        handle_record_create_response (qe, (struct RecordCreateResponseMessage *) msg, size);
-      break;
-    case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_REMOVE_RESPONSE:
-        if (size != sizeof (struct RecordRemoveResponseMessage))
-        {
-          GNUNET_break_op (0);
-          break;
-        }
-        handle_record_remove_response (qe, (struct RecordRemoveResponseMessage *) msg, size);
-      break;
-    case GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_TO_NAME_RESPONSE:
-        if (size < sizeof (struct ZoneToNameResponseMessage))
-        {
-          GNUNET_break_op (0);
-          break;
-        }
-        handle_zone_to_name_response (qe, (struct ZoneToNameResponseMessage *) msg, size);
-      break;
-    default:
-      GNUNET_break_op (0);
-      break;
+  switch (type)
+  {
+  case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_STORE_RESPONSE:
+    if (size != sizeof (struct RecordStoreResponseMessage))
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
+    return handle_record_store_response (qe, (const struct RecordStoreResponseMessage *) msg, size);
+  case GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_TO_NAME_RESPONSE:
+    if (size < sizeof (struct ZoneToNameResponseMessage))
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
+    return handle_zone_to_name_response (qe, (const struct ZoneToNameResponseMessage *) msg, size);
+  case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_RESULT:
+    if (size < sizeof (struct RecordResultMessage))
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
+    return handle_record_result (qe, (const struct RecordResultMessage *) msg, size);
+  case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_LOOKUP_RESPONSE:
+    if (size < sizeof (struct LabelLookupResponseMessage))
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
+    return handle_lookup_result (qe, (const struct LabelLookupResponseMessage *) msg, size);
+  default:
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
 }
 
-static void
+
+/**
+ * Handle a response from NAMESTORE service for a zone iteration request
+ *
+ * @param ze the respective iterator for this operation
+ * @param msg the message containing the respoonse
+ * @param size the message size
+ * @return #GNUNET_YES on success, @a ze should be kept, #GNUNET_NO on success if @a ze should
+ *         not be kept any longer, #GNUNET_SYSERR on error (disconnect) and @a ze should be kept
+ */
+static int
 handle_zone_iteration_response (struct GNUNET_NAMESTORE_ZoneIterator *ze,
-                                struct ZoneIterationResponseMessage *msg,
+                                const struct RecordResultMessage *msg,
                                 size_t size)
 {
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received `%s' \n",
-              "ZONE_ITERATION_RESPONSE");
-
-  struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded pubdummy;
-  size_t msg_len = 0;
-  size_t exp_msg_len = 0;
-  size_t name_len = 0;
-  size_t rd_len = 0;
-  unsigned rd_count = 0;
-
-  char *name_tmp;
-  char *rd_ser_tmp;
-  struct GNUNET_TIME_Absolute expire;
-
+  static struct GNUNET_CRYPTO_EcdsaPrivateKey priv_dummy;
+  size_t msg_len;
+  size_t exp_msg_len;
+  size_t name_len;
+  size_t rd_len;
+  unsigned rd_count;
+  const char *name_tmp;
+  const char *rd_ser_tmp;
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received `%s'\n",
+       "ZONE_ITERATION_RESPONSE");
   msg_len = ntohs (msg->gns_header.header.size);
   rd_len = ntohs (msg->rd_len);
   rd_count = ntohs (msg->rd_count);
   name_len = ntohs (msg->name_len);
-  expire = GNUNET_TIME_absolute_ntoh (msg->expire);
-
-  exp_msg_len = sizeof (struct ZoneIterationResponseMessage) + name_len + rd_len;
+  exp_msg_len = sizeof (struct RecordResultMessage) + name_len + rd_len;
   if (msg_len != exp_msg_len)
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Message size describes with `%u' bytes but calculated size is %u bytes \n",
-                msg_len, exp_msg_len);
-    GNUNET_break_op (0);
-    return;
-  }
-  if (0 != ntohs (msg->reserved))
-  {
-    GNUNET_break_op (0);
-    return;
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-
-  memset (&pubdummy, '\0', sizeof (pubdummy));
-  if ((0 == name_len) && (0 == (memcmp (&msg->public_key, &pubdummy, sizeof (pubdummy)))))
+  if ( (0 == name_len) &&
+       (0 == (memcmp (&msg->private_key,
+                     &priv_dummy,
+                     sizeof (priv_dummy)))) )
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Zone iteration is completed!\n");
-
-    if (ze->proc != NULL)
-      ze->proc(ze->proc_cls, NULL, GNUNET_TIME_absolute_get_zero (), NULL , 0, NULL, NULL);
-
-    GNUNET_CONTAINER_DLL_remove(ze->h->z_head, ze->h->z_tail, ze);
-    GNUNET_free (ze);
-    return;
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+        "Zone iteration completed!\n");
+    if (NULL != ze->proc)
+      ze->proc (ze->proc_cls, NULL, NULL, 0, NULL);
+    return GNUNET_NO;
   }
-
-  name_tmp = (char *) &msg[1];
-  if ((name_tmp[name_len -1] != '\0') || (name_len > 256))
+  name_tmp = (const char *) &msg[1];
+  if ((name_tmp[name_len -1] != '\0') || (name_len > MAX_NAME_LEN))
   {
-    GNUNET_break_op (0);
-    return;
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
-  rd_ser_tmp = (char *) &name_tmp[name_len];
-  struct GNUNET_NAMESTORE_RecordData rd[rd_count];
-  if (GNUNET_OK != GNUNET_NAMESTORE_records_deserialize (rd_len, rd_ser_tmp, rd_count, rd))
+  rd_ser_tmp = (const char *) &name_tmp[name_len];
   {
-    GNUNET_break_op (0);
-    return;
-  }
+    struct GNUNET_GNSRECORD_Data rd[rd_count];
 
-  if (ze->proc != NULL)
-    ze->proc(ze->proc_cls, &msg->public_key, expire, name_tmp, rd_count, rd, &msg->signature);
+    if (GNUNET_OK != GNUNET_GNSRECORD_records_deserialize (rd_len,
+                                                          rd_ser_tmp,
+                                                          rd_count,
+                                                          rd))
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
+    if (NULL != ze->proc)
+      ze->proc (ze->proc_cls,
+               &msg->private_key,
+               name_tmp,
+               rd_count, rd);
+    return GNUNET_YES;
+  }
 }
 
 
-static void
+/**
+ * Handle incoming messages for zone iterations
+ *
+ * @param ze the respective zone iteration handle
+ * @param msg the message we received
+ * @param type the message type in HBO
+ * @param size the message size
+ * @return #GNUNET_YES on success, @a ze should be kept, #GNUNET_NO on success if @a ze should
+ *         not be kept any longer, #GNUNET_SYSERR on error (disconnect) and @a ze should be kept
+ */
+static int
 manage_zone_operations (struct GNUNET_NAMESTORE_ZoneIterator *ze,
                         const struct GNUNET_MessageHeader *msg,
                         int type, size_t size)
 {
-
   /* handle different message type */
-  switch (type) {
-    case GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_ITERATION_RESPONSE:
-        if (size < sizeof (struct ZoneIterationResponseMessage))
-        {
-          GNUNET_break_op (0);
-          break;
-        }
-        handle_zone_iteration_response (ze, (struct ZoneIterationResponseMessage *) msg, size);
-      break;
-    default:
-      GNUNET_break_op (0);
-      break;
+  switch (type)
+  {
+  case GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_RESULT:
+    if (size < sizeof (struct RecordResultMessage))
+    {
+      GNUNET_break (0);
+      return GNUNET_SYSERR;
+    }
+    return handle_zone_iteration_response (ze,
+                                          (const struct RecordResultMessage *) msg,
+                                          size);
+  default:
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
   }
 }
 
+
 /**
  * Type of a function to call when we receive a message
  * from the service.
  *
- * @param cls the 'struct GNUNET_NAMESTORE_SchedulingHandle'
+ * @param cls the `struct GNUNET_NAMESTORE_SchedulingHandle`
  * @param msg message received, NULL on timeout or fatal error
  */
 static void
-process_namestore_message (void *cls, const struct GNUNET_MessageHeader *msg)
+process_namestore_message (void *cls,
+                          const struct GNUNET_MessageHeader *msg)
 {
   struct GNUNET_NAMESTORE_Handle *h = cls;
-  struct GNUNET_NAMESTORE_Header * gm;
+  const struct GNUNET_NAMESTORE_Header *gm;
   struct GNUNET_NAMESTORE_QueueEntry *qe;
   struct GNUNET_NAMESTORE_ZoneIterator *ze;
   uint16_t size;
   uint16_t type;
-  uint32_t r_id = UINT32_MAX;
+  uint32_t r_id;
+  int ret;
 
   if (NULL == msg)
   {
     force_reconnect (h);
     return;
   }
-
   size = ntohs (msg->size);
   type = ntohs (msg->type);
-
   if (size < sizeof (struct GNUNET_NAMESTORE_Header))
   {
     GNUNET_break_op (0);
-    GNUNET_CLIENT_receive (h->client, &process_namestore_message, h,
+    GNUNET_CLIENT_receive (h->client,
+                          &process_namestore_message, h,
                            GNUNET_TIME_UNIT_FOREVER_REL);
     return;
   }
-
-  gm = (struct GNUNET_NAMESTORE_Header *) msg;
+  gm = (const struct GNUNET_NAMESTORE_Header *) msg;
   r_id = ntohl (gm->r_id);
 
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received message type %i size %i op %u\n", type, size, r_id);
-
-  /* Find matching operation */
-  if (r_id > h->op_id)
-  {
-    /* No matching pending operation found */
-    GNUNET_break_op (0);
-    GNUNET_CLIENT_receive (h->client, &process_namestore_message, h,
-                           GNUNET_TIME_UNIT_FOREVER_REL);
-    return;
-  }
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received message type %u size %u op %u\n",
+       (unsigned int) type,
+       (unsigned int) size,
+       (unsigned int) r_id);
 
   /* Is it a record related operation ? */
   for (qe = h->op_head; qe != NULL; qe = qe->next)
-  {
     if (qe->op_id == r_id)
       break;
-  }
-  if (qe != NULL)
+  if (NULL != qe)
   {
-    manage_record_operations (qe, msg, type, size);
+    ret = manage_record_operations (qe, msg, type, size);
+    if (GNUNET_SYSERR == ret)
+    {
+      /* protocol error, need to reconnect */
+      h->reconnect = GNUNET_YES;
+    }
+    else
+    {
+      /* client was notified about success or failure, clean up 'qe' */
+      GNUNET_CONTAINER_DLL_remove (h->op_head,
+                                  h->op_tail,
+                                  qe);
+      GNUNET_free (qe);
+    }
   }
-
-  /* Is it a zone iteration operation ? */
+  /* Is it a zone iteration operation? */
   for (ze = h->z_head; ze != NULL; ze = ze->next)
-  {
     if (ze->op_id == r_id)
       break;
+  if (NULL != ze)
+  {
+    ret = manage_zone_operations (ze, msg, type, size);
+    if (GNUNET_NO == ret)
+    {
+      /* end of iteration, clean up 'ze' */
+      GNUNET_CONTAINER_DLL_remove (h->z_head,
+                                  h->z_tail,
+                                  ze);
+      GNUNET_free (ze);
+    }
+    if (GNUNET_SYSERR == ret)
+    {
+      /* protocol error, need to reconnect */
+      h->reconnect = GNUNET_YES;
+    }
   }
-  if (ze != NULL)
+  if (GNUNET_YES == h->reconnect)
   {
-    manage_zone_operations (ze, msg, type, size);
+    force_reconnect (h);
+    return;
   }
-
   GNUNET_CLIENT_receive (h->client, &process_namestore_message, h,
                          GNUNET_TIME_UNIT_FOREVER_REL);
-
-  if (GNUNET_YES == h->reconnect)
-    force_reconnect (h);
-
 }
 
 
@@ -725,13 +776,15 @@ do_transmit (struct GNUNET_NAMESTORE_Handle *h);
 /**
  * We can now transmit a message to NAMESTORE. Do it.
  *
- * @param cls the 'struct GNUNET_NAMESTORE_Handle'
+ * @param cls the `struct GNUNET_NAMESTORE_Handle`
  * @param size number of bytes we can transmit
  * @param buf where to copy the messages
- * @return number of bytes copied into buf
+ * @return number of bytes copied into @a buf
  */
 static size_t
-transmit_message_to_namestore (void *cls, size_t size, void *buf)
+transmit_message_to_namestore (void *cls,
+                              size_t size,
+                              void *buf)
 {
   struct GNUNET_NAMESTORE_Handle *h = cls;
   struct PendingMessage *p;
@@ -739,22 +792,29 @@ transmit_message_to_namestore (void *cls, size_t size, void *buf)
   char *cbuf;
 
   h->th = NULL;
-  if ((size == 0) || (buf == NULL))
+  if ((0 == size) || (NULL == buf))
   {
     force_reconnect (h);
     return 0;
   }
   ret = 0;
   cbuf = buf;
-  while ((NULL != (p = h->pending_head)) && (p->size <= size))
+  while ( (NULL != (p = h->pending_head)) &&
+         (p->size <= size) )
   {
     memcpy (&cbuf[ret], &p[1], p->size);
     ret += p->size;
     size -= p->size;
-    GNUNET_CONTAINER_DLL_remove (h->pending_head, h->pending_tail, p);
-    if (GNUNET_YES == p->is_init)
-      GNUNET_CLIENT_receive (h->client, &process_namestore_message, h,
+    GNUNET_CONTAINER_DLL_remove (h->pending_head,
+                                h->pending_tail,
+                                p);
+    if (GNUNET_NO == h->is_receiving)
+    {
+      h->is_receiving = GNUNET_YES;
+      GNUNET_CLIENT_receive (h->client,
+                            &process_namestore_message, h,
                              GNUNET_TIME_UNIT_FOREVER_REL);
+    }
     GNUNET_free (p);
   }
   do_transmit (h);
@@ -774,48 +834,34 @@ do_transmit (struct GNUNET_NAMESTORE_Handle *h)
   struct PendingMessage *p;
 
   if (NULL != h->th)
-    return;
+    return; /* transmission request already pending */
   if (NULL == (p = h->pending_head))
-    return;
+    return; /* transmission queue empty */
   if (NULL == h->client)
     return;                     /* currently reconnecting */
-
   h->th = GNUNET_CLIENT_notify_transmit_ready (h->client, p->size,
-                                           GNUNET_TIME_UNIT_FOREVER_REL,
-                                           GNUNET_NO, &transmit_message_to_namestore,
-                                           h);
+                                              GNUNET_TIME_UNIT_FOREVER_REL,
+                                              GNUNET_NO, &transmit_message_to_namestore,
+                                              h);
+  GNUNET_break (NULL != h->th);
 }
 
 
 /**
  * Reconnect to namestore service.
  *
- * @param h the handle to the namestore service
+ * @param h the handle to the NAMESTORE service
  */
 static void
 reconnect (struct GNUNET_NAMESTORE_Handle *h)
 {
-  struct PendingMessage *p;
-  struct StartMessage *init;
-
   GNUNET_assert (NULL == h->client);
   h->client = GNUNET_CLIENT_connect ("namestore", h->cfg);
   GNUNET_assert (NULL != h->client);
-
-  if ((NULL == (p = h->pending_head)) || (GNUNET_YES != p->is_init))
-  {
-    p = GNUNET_malloc (sizeof (struct PendingMessage) +
-                       sizeof (struct StartMessage));
-    p->size = sizeof (struct StartMessage);
-    p->is_init = GNUNET_YES;
-    init = (struct StartMessage *) &p[1];
-    init->header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_START);
-    init->header.size = htons (sizeof (struct StartMessage));
-    GNUNET_CONTAINER_DLL_insert (h->pending_head, h->pending_tail, p);
-  }
   do_transmit (h);
 }
 
+
 /**
  * Re-establish the connection to the service.
  *
@@ -823,7 +869,8 @@ reconnect (struct GNUNET_NAMESTORE_Handle *h)
  * @param tc scheduler context
  */
 static void
-reconnect_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
+reconnect_task (void *cls,
+               const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct GNUNET_NAMESTORE_Handle *h = cls;
 
@@ -840,22 +887,37 @@ reconnect_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 static void
 force_reconnect (struct GNUNET_NAMESTORE_Handle *h)
 {
+  if (NULL != h->th)
+  {
+    GNUNET_CLIENT_notify_transmit_ready_cancel (h->th);
+    h->th = NULL;
+  }
   h->reconnect = GNUNET_NO;
-  GNUNET_CLIENT_disconnect (h->client, GNUNET_NO);
+  GNUNET_CLIENT_disconnect (h->client);
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Reconnecting to namestore\n");
+  h->is_receiving = GNUNET_NO;
   h->client = NULL;
-  h->reconnect_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS,
-                                    &reconnect_task,
-                                    h);
+  h->reconnect_delay = GNUNET_TIME_STD_BACKOFF (h->reconnect_delay);
+  h->reconnect_task = GNUNET_SCHEDULER_add_delayed (h->reconnect_delay,
+                                                   &reconnect_task,
+                                                   h);
 }
 
+
+/**
+ * Get a fresh operation id to distinguish between namestore requests
+ *
+ * @param h the namestore handle
+ * @return next operation id to use
+ */
 static uint32_t
 get_op_id (struct GNUNET_NAMESTORE_Handle *h)
 {
-  uint32_t op_id = h->op_id;
-  h->op_id ++;
-  return op_id;
+  return h->last_op_id_used++;
 }
 
+
 /**
  * Initialize the connection with the NAMESTORE service.
  *
@@ -867,44 +929,54 @@ GNUNET_NAMESTORE_connect (const struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   struct GNUNET_NAMESTORE_Handle *h;
 
-  h = GNUNET_malloc (sizeof (struct GNUNET_NAMESTORE_Handle));
+  h = GNUNET_new (struct GNUNET_NAMESTORE_Handle);
   h->cfg = cfg;
   h->reconnect_task = GNUNET_SCHEDULER_add_now (&reconnect_task, h);
-  h->op_id = 0;
+  h->last_op_id_used = 0;
   return h;
 }
 
-static void
-clean_up_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
+
+/**
+ * Disconnect from the namestore service (and free associated
+ * resources).
+ *
+ * @param h handle to the namestore
+ */
+void
+GNUNET_NAMESTORE_disconnect (struct GNUNET_NAMESTORE_Handle *h)
 {
   struct PendingMessage *p;
   struct GNUNET_NAMESTORE_QueueEntry *q;
   struct GNUNET_NAMESTORE_ZoneIterator *z;
-  struct GNUNET_NAMESTORE_Handle *h = cls;
-  GNUNET_assert (h != NULL);
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Cleaning up\n");
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG, "Cleaning up\n");
+  GNUNET_assert (NULL != h);
+  if (NULL != h->th)
+  {
+    GNUNET_CLIENT_notify_transmit_ready_cancel (h->th);
+    h->th = NULL;
+  }
   while (NULL != (p = h->pending_head))
   {
     GNUNET_CONTAINER_DLL_remove (h->pending_head, h->pending_tail, p);
     GNUNET_free (p);
   }
-
+  GNUNET_break (NULL == h->op_head);
   while (NULL != (q = h->op_head))
   {
-    GNUNET_break (0);
     GNUNET_CONTAINER_DLL_remove (h->op_head, h->op_tail, q);
     GNUNET_free (q);
   }
-
+  GNUNET_break (NULL == h->z_head);
   while (NULL != (z = h->z_head))
   {
     GNUNET_CONTAINER_DLL_remove (h->z_head, h->z_tail, z);
     GNUNET_free (z);
   }
-
   if (NULL != h->client)
   {
-    GNUNET_CLIENT_disconnect (h->client, GNUNET_NO);
+    GNUNET_CLIENT_disconnect (h->client);
     h->client = NULL;
   }
   if (GNUNET_SCHEDULER_NO_TASK != h->reconnect_task)
@@ -912,437 +984,171 @@ clean_up_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
     GNUNET_SCHEDULER_cancel (h->reconnect_task);
     h->reconnect_task = GNUNET_SCHEDULER_NO_TASK;
   }
-  GNUNET_free(h);
-  h = NULL;
-};
-
-/**
- * Disconnect from the namestore service (and free associated
- * resources).
- *
- * @param h handle to the namestore
- * @param drop set to GNUNET_YES to delete all data in namestore (!)
- */
-void
-GNUNET_NAMESTORE_disconnect (struct GNUNET_NAMESTORE_Handle *h, int drop)
-{
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Disconnecting from namestore service\n");
-  GNUNET_SCHEDULER_add_now (&clean_up_task, h);
-}
-
-
-/**
- * Store an item in the namestore.  If the item is already present,
- * the expiration time is updated to the max of the existing time and
- * the new time.  This API is used when we cache signatures from other
- * authorities.
- *
- * @param h handle to the namestore
- * @param zone_key public key of the zone
- * @param name name that is being mapped (at most 255 characters long)
- * @param expire when does the corresponding block in the DHT expire (until
- *               when should we never do a DHT lookup for the same name again)?
- * @param rd_count number of entries in 'rd' array
- * @param rd array of records with data to store
- * @param signature signature for all the records in the zone under the given name
- * @param cont continuation to call when done
- * @param cont_cls closure for cont
- * @return handle to abort the request
- */
-struct GNUNET_NAMESTORE_QueueEntry *
-GNUNET_NAMESTORE_record_put (struct GNUNET_NAMESTORE_Handle *h,
-                            const struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded *zone_key,
-                            const char *name,
-                            struct GNUNET_TIME_Absolute expire,
-                            unsigned int rd_count,
-                            const struct GNUNET_NAMESTORE_RecordData *rd,
-                            const struct GNUNET_CRYPTO_RsaSignature *signature,
-                            GNUNET_NAMESTORE_ContinuationWithStatus cont,
-                            void *cont_cls)
-{
-  struct GNUNET_NAMESTORE_QueueEntry *qe;
-  struct PendingMessage *pe;
-
-  /* pointer to elements */
-  char * rd_tmp;
-  char * name_tmp;
-
-  size_t msg_size = 0;
-  size_t name_len = 0;
-  size_t rd_ser_len = 0;
-  uint32_t rid = 0;
-
-  GNUNET_assert (NULL != h);
-  GNUNET_assert (NULL != zone_key);
-  GNUNET_assert (NULL != name);
-  GNUNET_assert (NULL != rd);
-  GNUNET_assert (NULL != signature);
-
-  name_len = strlen(name) + 1;
-  if (name_len > 256)
-  {
-    GNUNET_break (0);
-    return NULL;
-  }
-
-  rid = get_op_id(h);
-  qe = GNUNET_malloc(sizeof (struct GNUNET_NAMESTORE_QueueEntry));
-  qe->nsh = h;
-  qe->cont = cont;
-  qe->cont_cls = cont_cls;
-  qe->op_id = rid;
-  GNUNET_CONTAINER_DLL_insert_tail(h->op_head, h->op_tail, qe);
-
-  /* set msg_size*/
-  rd_ser_len = GNUNET_NAMESTORE_records_get_size(rd_count, rd);
-  char rd_ser[rd_ser_len];
-  GNUNET_NAMESTORE_records_serialize(rd_count, rd, rd_ser_len, rd_ser);
-
-  struct RecordPutMessage * msg;
-  msg_size = sizeof (struct RecordPutMessage) + name_len  + rd_ser_len;
-
-  /* create msg here */
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
-  pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
-  msg = (struct RecordPutMessage *) &pe[1];
-  name_tmp = (char *) &msg[1];
-  rd_tmp = &name_tmp[name_len];
-
-  msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_PUT);
-  msg->gns_header.header.size = htons (msg_size);
-  msg->gns_header.r_id = htonl (rid);
-  msg->signature = *signature;
-  msg->name_len = htons (name_len);
-  msg->expire = GNUNET_TIME_absolute_hton (expire);
-  msg->rd_len = htons (rd_ser_len);
-  msg->rd_count = htons (rd_count);
-
-  msg->public_key = *zone_key;
-  memcpy (name_tmp, name, name_len);
-  memcpy (rd_tmp, rd_ser, rd_ser_len);
-
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for name `%s' with size %u\n", "NAMESTORE_RECORD_PUT", name, msg_size);
-
-  GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
-
-  return qe;
+  GNUNET_free (h);
 }
 
 
-/**
- * Check if a signature is valid.  This API is used by the GNS Block
- * to validate signatures received from the network.
- *
- * @param public_key public key of the zone
- * @param name name that is being mapped (at most 255 characters long)
- * @param rd_count number of entries in 'rd' array
- * @param rd array of records with data to store
- * @param signature signature for all the records in the zone under the given name
- * @return GNUNET_OK if the signature is valid
- */
-int
-GNUNET_NAMESTORE_verify_signature (const struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded *public_key,
-                                  const char *name,
-                                  unsigned int rd_count,
-                                  const struct GNUNET_NAMESTORE_RecordData *rd,
-                                  const struct GNUNET_CRYPTO_RsaSignature *signature)
-{
-  int res = GNUNET_SYSERR;
-  size_t rd_ser_len = 0;
-  size_t name_len = 0;
-  char * name_tmp;
-  char * rd_tmp;
-  struct GNUNET_CRYPTO_RsaSignaturePurpose *sig_purpose;
-
-  GNUNET_assert (public_key != NULL);
-  GNUNET_assert (name != NULL);
-  GNUNET_assert (rd != NULL);
-  GNUNET_assert (signature != NULL);
-
-
-  rd_ser_len = GNUNET_NAMESTORE_records_get_size(rd_count, rd);
-  char rd_ser[rd_ser_len];
-  GNUNET_NAMESTORE_records_serialize(rd_count, rd, rd_ser_len, rd_ser);
-
-  name_len = strlen (name) + 1;
-  if (name_len > 256)
-  {
-    GNUNET_break (0);
-    return GNUNET_SYSERR;
-  }
-
-  sig_purpose = GNUNET_malloc(sizeof (struct GNUNET_CRYPTO_RsaSignaturePurpose) + rd_ser_len + name_len);
-  sig_purpose->size = htonl (sizeof (struct GNUNET_CRYPTO_RsaSignaturePurpose)+ rd_ser_len + name_len);
-  sig_purpose->purpose = htonl (GNUNET_SIGNATURE_PURPOSE_GNS_RECORD_SIGN);
-  name_tmp = (char *) &sig_purpose[1];
-  rd_tmp = &name_tmp[name_len];
-  memcpy (name_tmp, name, name_len);
-  memcpy (rd_tmp, rd_ser, rd_ser_len);
-
-  res = GNUNET_CRYPTO_rsa_verify(GNUNET_SIGNATURE_PURPOSE_GNS_RECORD_SIGN, sig_purpose, signature, public_key);
-
-  GNUNET_free (sig_purpose);
-
-  return res;
-}
-
 /**
  * Store an item in the namestore.  If the item is already present,
- * the expiration time is updated to the max of the existing time and
- * the new time.  This API is used by the authority of a zone.
+ * it is replaced with the new record.  Use an empty array to
+ * remove all records under the given name.
  *
  * @param h handle to the namestore
  * @param pkey private key of the zone
- * @param expire block expiration time
- * @param name name that is being mapped (at most 255 characters long)
- * @param rd record data to store
+ * @param label name that is being mapped (at most 255 characters long)
+ * @param rd_count number of records in the 'rd' array
+ * @param rd array of records with data to store
  * @param cont continuation to call when done
- * @param cont_cls closure for cont
+ * @param cont_cls closure for @a cont
  * @return handle to abort the request
  */
 struct GNUNET_NAMESTORE_QueueEntry *
-GNUNET_NAMESTORE_record_create (struct GNUNET_NAMESTORE_Handle *h,
-                               const struct GNUNET_CRYPTO_RsaPrivateKey *pkey,
-                               const struct GNUNET_TIME_Absolute expire,
-                               const char *name,
-                               const struct GNUNET_NAMESTORE_RecordData *rd,
+GNUNET_NAMESTORE_records_store (struct GNUNET_NAMESTORE_Handle *h,
+                               const struct GNUNET_CRYPTO_EcdsaPrivateKey *pkey,
+                               const char *label,
+                               unsigned int rd_count,
+                               const struct GNUNET_GNSRECORD_Data *rd,
                                GNUNET_NAMESTORE_ContinuationWithStatus cont,
                                void *cont_cls)
 {
   struct GNUNET_NAMESTORE_QueueEntry *qe;
   struct PendingMessage *pe;
-  char * name_tmp;
-  char * pkey_tmp;
-  char * rd_tmp;
-  size_t rd_ser_len = 0;
-  size_t msg_size = 0;
-  size_t name_len = 0;
-  size_t key_len = 0;
-  uint32_t rid = 0;
+  char *name_tmp;
+  char *rd_ser;
+  size_t rd_ser_len;
+  size_t msg_size;
+  size_t name_len;
+  uint32_t rid;
+  struct RecordStoreMessage *msg;
 
   GNUNET_assert (NULL != h);
   GNUNET_assert (NULL != pkey);
-  GNUNET_assert (NULL != name);
-  GNUNET_assert (NULL != rd);
-
-  name_len = strlen(name) + 1;
-  if (name_len > 256)
+  GNUNET_assert (NULL != label);
+  name_len = strlen (label) + 1;
+  if (name_len > MAX_NAME_LEN)
   {
     GNUNET_break (0);
     return NULL;
   }
-
-  rid = get_op_id(h);
-  qe = GNUNET_malloc(sizeof (struct GNUNET_NAMESTORE_QueueEntry));
+  rid = get_op_id (h);
+  qe = GNUNET_new (struct GNUNET_NAMESTORE_QueueEntry);
   qe->nsh = h;
   qe->cont = cont;
   qe->cont_cls = cont_cls;
   qe->op_id = rid;
-  GNUNET_CONTAINER_DLL_insert_tail(h->op_head, h->op_tail, qe);
-
-  /* set msg_size*/
-  struct GNUNET_CRYPTO_RsaPrivateKeyBinaryEncoded * pkey_enc = GNUNET_CRYPTO_rsa_encode_key (pkey);
-  GNUNET_assert (pkey_enc != NULL);
-  key_len = ntohs (pkey_enc->len);
-
-  rd_ser_len = GNUNET_NAMESTORE_records_get_size(1, rd);
-  char rd_ser[rd_ser_len];
-  GNUNET_NAMESTORE_records_serialize(1, rd, rd_ser_len, rd_ser);
+  GNUNET_CONTAINER_DLL_insert_tail (h->op_head, h->op_tail, qe);
 
-  struct RecordCreateMessage * msg;
-  msg_size = sizeof (struct RecordCreateMessage) + key_len + name_len + rd_ser_len;
-
-  /* create msg here */
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
+  /* setup msg */
+  rd_ser_len = GNUNET_GNSRECORD_records_get_size (rd_count, rd);
+  msg_size = sizeof (struct RecordStoreMessage) + name_len + rd_ser_len;
+  pe = GNUNET_malloc (sizeof (struct PendingMessage) + msg_size);
   pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
-  msg = (struct RecordCreateMessage *) &pe[1];
-
-  pkey_tmp = (char *) &msg[1];
-  name_tmp = &pkey_tmp[key_len];
-  rd_tmp = &name_tmp[name_len];
-
-  msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_CREATE);
+  msg = (struct RecordStoreMessage *) &pe[1];
+  msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_STORE);
   msg->gns_header.header.size = htons (msg_size);
   msg->gns_header.r_id = htonl (rid);
   msg->name_len = htons (name_len);
-  msg->rd_count = htons (1);
+  msg->rd_count = htons (rd_count);
   msg->rd_len = htons (rd_ser_len);
-  msg->pkey_len = htons (key_len);
-  msg->expire = GNUNET_TIME_absolute_hton(expire);
-  memcpy (pkey_tmp, pkey_enc, key_len);
-  memcpy (name_tmp, name, name_len);
-  memcpy (rd_tmp, rd_ser, rd_ser_len);
-  GNUNET_free (pkey_enc);
-
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for name `%s' with size %u\n", "NAMESTORE_RECORD_CREATE", name, msg_size);
+  msg->reserved = htons (0);
+  msg->private_key = *pkey;
 
+  name_tmp = (char *) &msg[1];
+  memcpy (name_tmp, label, name_len);
+  rd_ser = &name_tmp[name_len];
+  GNUNET_break (rd_ser_len ==
+               GNUNET_GNSRECORD_records_serialize (rd_count, rd,
+                                                   rd_ser_len,
+                                                   rd_ser));
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Sending `%s' message for name `%s' with size %u and %u records\n",
+       "NAMESTORE_RECORD_STORE", label, msg_size,
+       rd_count);
   GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
+  do_transmit (h);
   return qe;
 }
 
-
 /**
- * Explicitly remove some content from the database.  The
- * "cont"inuation will be called with status "GNUNET_OK" if content
- * was removed, "GNUNET_NO" if no matching entry was found and
- * "GNUNET_SYSERR" on all other types of errors.
- * This API is used by the authority of a zone.
+ * Set the desired nick name for a zone
  *
  * @param h handle to the namestore
  * @param pkey private key of the zone
- * @param name name that is being mapped (at most 255 characters long)
- * @param rd record data
+ * @param nick the nick name to set
  * @param cont continuation to call when done
- * @param cont_cls closure for cont
+ * @param cont_cls closure for 'cont'
  * @return handle to abort the request
  */
 struct GNUNET_NAMESTORE_QueueEntry *
-GNUNET_NAMESTORE_record_remove (struct GNUNET_NAMESTORE_Handle *h,
-                               const struct GNUNET_CRYPTO_RsaPrivateKey *pkey,
-                               const char *name,
-                               const struct GNUNET_NAMESTORE_RecordData *rd,
-                               GNUNET_NAMESTORE_ContinuationWithStatus cont,
-                               void *cont_cls)
+GNUNET_NAMESTORE_set_nick (struct GNUNET_NAMESTORE_Handle *h,
+                           const struct GNUNET_CRYPTO_EcdsaPrivateKey *pkey,
+                           const char *nick,
+                           GNUNET_NAMESTORE_ContinuationWithStatus cont,
+                           void *cont_cls)
 {
-  struct GNUNET_NAMESTORE_QueueEntry *qe;
-  struct PendingMessage *pe;
-  char *pkey_tmp;
-  char *rd_tmp;
-  char *name_tmp;
-  size_t rd_ser_len = 0;
-  size_t msg_size = 0;
-  size_t name_len = 0;
-  size_t key_len = 0;
-  uint32_t rid = 0;
-
-  GNUNET_assert (NULL != h);
-
-  rid = get_op_id(h);
-  qe = GNUNET_malloc(sizeof (struct GNUNET_NAMESTORE_QueueEntry));
-  qe->nsh = h;
-  qe->cont = cont;
-  qe->cont_cls = cont_cls;
-  qe->op_id = rid;
-  GNUNET_CONTAINER_DLL_insert_tail(h->op_head, h->op_tail, qe);
-
-  /* set msg_size*/
-  struct GNUNET_CRYPTO_RsaPrivateKeyBinaryEncoded * pkey_enc = GNUNET_CRYPTO_rsa_encode_key (pkey);
-  GNUNET_assert (pkey_enc != NULL);
-  key_len = ntohs (pkey_enc->len);
-
-  rd_ser_len = GNUNET_NAMESTORE_records_get_size(1, rd);
-  char rd_ser[rd_ser_len];
-  GNUNET_NAMESTORE_records_serialize(1, rd, rd_ser_len, rd_ser);
-
-  name_len = strlen (name) + 1;
-
-  struct RecordRemoveMessage * msg;
-  msg_size = sizeof (struct RecordRemoveMessage) + key_len + name_len + rd_ser_len;
-
-  /* create msg here */
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
-  pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
-  msg = (struct RecordRemoveMessage *) &pe[1];
-
-  pkey_tmp = (char *) &msg[1];
-  name_tmp = &pkey_tmp[key_len];
-  rd_tmp = &name_tmp[name_len];
-
-  msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_REMOVE);
-  msg->gns_header.header.size = htons (msg_size);
-  msg->gns_header.r_id = htonl (rid);
-  msg->name_len = htons (name_len);
-  msg->rd_len = htons (rd_ser_len);
-  msg->rd_count = htons (1);
-  msg->pkey_len = htons (key_len);
-  memcpy (pkey_tmp, pkey_enc, key_len);
-  memcpy (name_tmp, name, name_len);
-  memcpy (rd_tmp, rd_ser, rd_ser_len);
-
-  GNUNET_free (pkey_enc);
-
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for name `%s' with size %u\n", "NAMESTORE_RECORD_REMOVE", name, msg_size);
-
-  GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
-  return qe;
+  struct GNUNET_GNSRECORD_Data rd;
+
+  memset (&rd, 0, sizeof (rd));
+  rd.data = nick;
+  rd.data_size = strlen (nick) +1;
+  rd.record_type = GNUNET_GNSRECORD_TYPE_NICK;
+  rd.expiration_time = GNUNET_TIME_UNIT_FOREVER_ABS.abs_value_us;
+  rd.flags |= GNUNET_GNSRECORD_RF_PRIVATE;
+  return GNUNET_NAMESTORE_records_store(h, pkey, GNUNET_GNS_MASTERZONE_STR, 1, &rd, cont, cont_cls);
 }
 
 
 /**
- * Get a result for a particular key from the namestore.  The processor
- * will only be called once.  
+ * Lookup an item in the namestore.
  *
  * @param h handle to the namestore
- * @param zone zone to look up a record from
- * @param name name to look up
- * @param record_type desired record type, 0 for all
- * @param proc function to call on the matching records, or with
- *        NULL (rd_count == 0) if there are no matching records
- * @param proc_cls closure for proc
- * @return a handle that can be used to
- *         cancel
+ * @param pkey private key of the zone
+ * @param label name that is being mapped (at most 255 characters long)
+ * @param rm function to call with the result (with 0 records if we don't have that label)
+ * @param rm_cls closure for @a rm
+ * @return handle to abort the request
  */
 struct GNUNET_NAMESTORE_QueueEntry *
-GNUNET_NAMESTORE_lookup_record (struct GNUNET_NAMESTORE_Handle *h, 
-                             const GNUNET_HashCode *zone,
-                             const char *name,
-                             uint32_t record_type,
-                             GNUNET_NAMESTORE_RecordProcessor proc, void *proc_cls)
+GNUNET_NAMESTORE_records_lookup (struct GNUNET_NAMESTORE_Handle *h,
+                                 const struct GNUNET_CRYPTO_EcdsaPrivateKey *pkey,
+                                 const char *label,
+                                 GNUNET_NAMESTORE_RecordMonitor rm,
+                                 void *rm_cls)
 {
   struct GNUNET_NAMESTORE_QueueEntry *qe;
   struct PendingMessage *pe;
-  size_t msg_size = 0;
-  size_t name_len = 0;
-  uint32_t rid = 0;
+  struct LabelLookupMessage * msg;
+  size_t msg_size;
+  size_t label_len;
 
   GNUNET_assert (NULL != h);
-  GNUNET_assert (NULL != zone);
-  GNUNET_assert (NULL != name);
+  GNUNET_assert (NULL != pkey);
+  GNUNET_assert (NULL != label);
 
-  name_len = strlen (name) + 1;
-  if ((name_len == 0) || (name_len > 256))
-  {
-    GNUNET_break (0);
+  if (1 == (label_len = strlen (label) + 1))
     return NULL;
-  }
 
-  rid = get_op_id(h);
-  qe = GNUNET_malloc(sizeof (struct GNUNET_NAMESTORE_QueueEntry));
+  qe = GNUNET_new (struct GNUNET_NAMESTORE_QueueEntry);
   qe->nsh = h;
-  qe->proc = proc;
-  qe->proc_cls = proc_cls;
-  qe->op_id = rid;
-  GNUNET_CONTAINER_DLL_insert_tail(h->op_head, h->op_tail, qe);
-
-  /* set msg_size*/
-  msg_size = sizeof (struct LookupNameMessage) + name_len;
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
+  qe->proc = rm;
+  qe->proc_cls = rm_cls;
+  qe->op_id = get_op_id(h);
+  GNUNET_CONTAINER_DLL_insert_tail (h->op_head, h->op_tail, qe);
 
-  /* create msg here */
-  struct LookupNameMessage * msg;
+  msg_size = sizeof (struct LabelLookupMessage) + label_len;
+  pe = GNUNET_malloc (sizeof (struct PendingMessage) + msg_size);
   pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
-  msg = (struct LookupNameMessage *) &pe[1];
-  msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_LOOKUP_NAME);
+  msg = (struct LabelLookupMessage *) &pe[1];
+  msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_RECORD_LOOKUP);
   msg->gns_header.header.size = htons (msg_size);
-  msg->gns_header.r_id = htonl (rid);
-  msg->record_type = htonl (record_type);
-  msg->name_len = htonl (name_len);
-  msg->zone = *zone;
-  memcpy (&msg[1], name, name_len);
-
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for name `%s'\n", "NAMESTORE_LOOKUP_NAME", name);
+  msg->gns_header.r_id = htonl (qe->op_id);
+  msg->zone = *pkey;
+  msg->label_len = htonl(label_len);
+  memcpy (&msg[1], label, label_len);
 
   /* transmit message */
   GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
-
+  do_transmit (h);
   return qe;
 }
 
@@ -1352,45 +1158,40 @@ GNUNET_NAMESTORE_lookup_record (struct GNUNET_NAMESTORE_Handle *h,
  * Returns at most one result to the processor.
  *
  * @param h handle to the namestore
- * @param zone hash of public key of the zone to look up in, never NULL
- * @param value_zone hash of the public key of the target zone (value), never NULL
+ * @param zone public key of the zone to look up in, never NULL
+ * @param value_zone public key of the target zone (value), never NULL
  * @param proc function to call on the matching records, or with
  *        NULL (rd_count == 0) if there are no matching records
- * @param proc_cls closure for proc
+ * @param proc_cls closure for @a proc
  * @return a handle that can be used to
  *         cancel
  */
 struct GNUNET_NAMESTORE_QueueEntry *
 GNUNET_NAMESTORE_zone_to_name (struct GNUNET_NAMESTORE_Handle *h,
-                               const GNUNET_HashCode *zone,
-                               const GNUNET_HashCode *value_zone,
-                               GNUNET_NAMESTORE_RecordProcessor proc, void *proc_cls)
+                              const struct GNUNET_CRYPTO_EcdsaPrivateKey *zone,
+                              const struct GNUNET_CRYPTO_EcdsaPublicKey *value_zone,
+                              GNUNET_NAMESTORE_RecordMonitor proc, void *proc_cls)
 {
   struct GNUNET_NAMESTORE_QueueEntry *qe;
   struct PendingMessage *pe;
-  size_t msg_size = 0;
-  uint32_t rid = 0;
+  struct ZoneToNameMessage * msg;
+  size_t msg_size;
+  uint32_t rid;
 
   GNUNET_assert (NULL != h);
   GNUNET_assert (NULL != zone);
   GNUNET_assert (NULL != value_zone);
-
   rid = get_op_id(h);
-  qe = GNUNET_malloc(sizeof (struct GNUNET_NAMESTORE_QueueEntry));
+  qe = GNUNET_new (struct GNUNET_NAMESTORE_QueueEntry);
   qe->nsh = h;
   qe->proc = proc;
   qe->proc_cls = proc_cls;
   qe->op_id = rid;
-  GNUNET_CONTAINER_DLL_insert_tail(h->op_head, h->op_tail, qe);
+  GNUNET_CONTAINER_DLL_insert_tail (h->op_head, h->op_tail, qe);
 
-  /* set msg_size*/
   msg_size = sizeof (struct ZoneToNameMessage);
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
-
-  /* create msg here */
-  struct ZoneToNameMessage * msg;
+  pe = GNUNET_malloc (sizeof (struct PendingMessage) + msg_size);
   pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
   msg = (struct ZoneToNameMessage *) &pe[1];
   msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_TO_NAME);
   msg->gns_header.header.size = htons (msg_size);
@@ -1398,112 +1199,69 @@ GNUNET_NAMESTORE_zone_to_name (struct GNUNET_NAMESTORE_Handle *h,
   msg->zone = *zone;
   msg->value_zone = *value_zone;
 
-  char * z_tmp = strdup (GNUNET_h2s (zone));
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for zone `%s' in zone `%s'\n",
-      "NAMESTORE_ZONE_TO_NAME",
-      z_tmp,
-      GNUNET_h2s (value_zone));
-  GNUNET_free (z_tmp);
-
   /* transmit message */
   GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
-
+  do_transmit (h);
   return qe;
 }
 
 
-
 /**
  * Starts a new zone iteration (used to periodically PUT all of our
- * records into our DHT). This MUST lock the GNUNET_NAMESTORE_Handle
- * for any other calls than GNUNET_NAMESTORE_zone_iterator_next and
- * GNUNET_NAMESTORE_zone_iteration_stop.  "proc" will be called once
+ * records into our DHT). This MUST lock the struct GNUNET_NAMESTORE_Handle
+ * for any other calls than #GNUNET_NAMESTORE_zone_iterator_next and
+ * #GNUNET_NAMESTORE_zone_iteration_stop. @a proc will be called once
  * immediately, and then again after
- * "GNUNET_NAMESTORE_zone_iterator_next" is invoked.
+ * #GNUNET_NAMESTORE_zone_iterator_next is invoked.
  *
  * @param h handle to the namestore
  * @param zone zone to access, NULL for all zones
- * @param must_have_flags flags that must be set for the record to be returned
- * @param must_not_have_flags flags that must NOT be set for the record to be returned
  * @param proc function to call on each name from the zone; it
  *        will be called repeatedly with a value (if available)
  *        and always once at the end with a name of NULL.
- * @param proc_cls closure for proc
+ * @param proc_cls closure for @a proc
  * @return an iterator handle to use for iteration
  */
 struct GNUNET_NAMESTORE_ZoneIterator *
 GNUNET_NAMESTORE_zone_iteration_start (struct GNUNET_NAMESTORE_Handle *h,
-                                      const GNUNET_HashCode *zone,
-                                      enum GNUNET_NAMESTORE_RecordFlags must_have_flags,
-                                      enum GNUNET_NAMESTORE_RecordFlags must_not_have_flags,
-                                      GNUNET_NAMESTORE_RecordProcessor proc,
+                                      const struct GNUNET_CRYPTO_EcdsaPrivateKey *zone,
+                                      GNUNET_NAMESTORE_RecordMonitor proc,
                                       void *proc_cls)
 {
   struct GNUNET_NAMESTORE_ZoneIterator *it;
   struct PendingMessage *pe;
-  size_t msg_size = 0;
-  uint32_t rid = 0;
+  struct ZoneIterationStartMessage * msg;
+  size_t msg_size;
+  uint32_t rid;
 
   GNUNET_assert (NULL != h);
-
-
   rid = get_op_id(h);
-  it = GNUNET_malloc (sizeof (struct GNUNET_NAMESTORE_ZoneIterator));
+  it = GNUNET_new (struct GNUNET_NAMESTORE_ZoneIterator);
   it->h = h;
   it->proc = proc;
-  it->proc_cls = proc;
+  it->proc_cls = proc_cls;
   it->op_id = rid;
-
   if (NULL != zone)
-  {
     it->zone = *zone;
-    it->has_zone = GNUNET_YES;
-  }
-  else
-  {
-    memset (&it->zone, '\0', sizeof (it->zone));
-    it->has_zone = GNUNET_NO;
-  }
-  GNUNET_CONTAINER_DLL_insert_tail(h->z_head, h->z_tail, it);
+  GNUNET_CONTAINER_DLL_insert_tail (h->z_head, h->z_tail, it);
 
-  /* set msg_size*/
   msg_size = sizeof (struct ZoneIterationStartMessage);
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
-
-  /* create msg here */
-  struct ZoneIterationStartMessage * msg;
+  pe = GNUNET_malloc (sizeof (struct PendingMessage) + msg_size);
   pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
   msg = (struct ZoneIterationStartMessage *) &pe[1];
   msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_ITERATION_START);
   msg->gns_header.header.size = htons (msg_size);
   msg->gns_header.r_id = htonl (rid);
   if (NULL != zone)
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for zone `%s'\n", "ZONE_ITERATION_START", GNUNET_h2s(zone));
     msg->zone = *zone;
-  }
-  else
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for all zones\n", "ZONE_ITERATION_START");
-    memset (&msg->zone, '\0', sizeof (msg->zone));
-  }
-  msg->must_have_flags = ntohs (must_have_flags);
-  msg->must_not_have_flags = ntohs (must_not_have_flags);
-
-
-
-  /* transmit message */
   GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
-
+  do_transmit (h);
   return it;
 }
 
 
 /**
- * Calls the record processor specified in GNUNET_NAMESTORE_zone_iteration_start
+ * Calls the record processor specified in #GNUNET_NAMESTORE_zone_iteration_start
  * for the next record.
  *
  * @param it the iterator
@@ -1512,30 +1270,24 @@ void
 GNUNET_NAMESTORE_zone_iterator_next (struct GNUNET_NAMESTORE_ZoneIterator *it)
 {
   struct GNUNET_NAMESTORE_Handle *h;
+  struct ZoneIterationNextMessage * msg;
   struct PendingMessage *pe;
-  size_t msg_size = 0;
+  size_t msg_size;
 
   GNUNET_assert (NULL != it);
   h = it->h;
-
-  /* set msg_size*/
   msg_size = sizeof (struct ZoneIterationNextMessage);
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
-
-  /* create msg here */
-  struct ZoneIterationNextMessage * msg;
+  pe = GNUNET_malloc (sizeof (struct PendingMessage) + msg_size);
   pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
   msg = (struct ZoneIterationNextMessage *) &pe[1];
   msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_ITERATION_NEXT);
   msg->gns_header.header.size = htons (msg_size);
   msg->gns_header.r_id = htonl (it->op_id);
-
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message\n", "ZONE_ITERATION_NEXT");
-
-  /* transmit message */
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Sending `%s' message\n",
+       "ZONE_ITERATION_NEXT");
   GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
+  do_transmit (h);
 }
 
 
@@ -1547,32 +1299,29 @@ GNUNET_NAMESTORE_zone_iterator_next (struct GNUNET_NAMESTORE_ZoneIterator *it)
 void
 GNUNET_NAMESTORE_zone_iteration_stop (struct GNUNET_NAMESTORE_ZoneIterator *it)
 {
-  GNUNET_assert (NULL != it);
+  struct GNUNET_NAMESTORE_Handle *h;
   struct PendingMessage *pe;
-  size_t msg_size = 0;
-  struct GNUNET_NAMESTORE_Handle *h = it->h;
+  size_t msg_size;
+  struct ZoneIterationStopMessage * msg;
 
-  /* set msg_size*/
+  GNUNET_assert (NULL != it);
+  h = it->h;
+  GNUNET_CONTAINER_DLL_remove (h->z_head,
+                              h->z_tail,
+                              it);
   msg_size = sizeof (struct ZoneIterationStopMessage);
-  pe = GNUNET_malloc(sizeof (struct PendingMessage) + msg_size);
-
-  /* create msg here */
-  struct ZoneIterationStopMessage * msg;
+  pe = GNUNET_malloc (sizeof (struct PendingMessage) + msg_size);
   pe->size = msg_size;
-  pe->is_init = GNUNET_NO;
   msg = (struct ZoneIterationStopMessage *) &pe[1];
   msg->gns_header.header.type = htons (GNUNET_MESSAGE_TYPE_NAMESTORE_ZONE_ITERATION_STOP);
   msg->gns_header.header.size = htons (msg_size);
   msg->gns_header.r_id = htonl (it->op_id);
-
-  if (GNUNET_YES == it->has_zone)
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for zone `%s'\n", "ZONE_ITERATION_STOP", GNUNET_h2s(&it->zone));
-  else
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending `%s' message for all zones\n", "ZONE_ITERATION_STOP");
-
-  /* transmit message */
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Sending `%s' message\n",
+             "ZONE_ITERATION_STOP");
   GNUNET_CONTAINER_DLL_insert_tail (h->pending_head, h->pending_tail, pe);
-  do_transmit(h);
+  do_transmit (h);
+  GNUNET_free (it);
 }
 
 
@@ -1587,11 +1336,10 @@ GNUNET_NAMESTORE_cancel (struct GNUNET_NAMESTORE_QueueEntry *qe)
 {
   struct GNUNET_NAMESTORE_Handle *h = qe->nsh;
 
-  GNUNET_assert (qe != NULL);
-
-  GNUNET_CONTAINER_DLL_remove(h->op_head, h->op_tail, qe);
+  GNUNET_assert (NULL != qe);
+  GNUNET_CONTAINER_DLL_remove (h->op_head, h->op_tail, qe);
   GNUNET_free(qe);
-
 }
 
+
 /* end of namestore_api.c */