[GNUnet-SVN] r20395 - gnunet/src/namestore

[gnunet]

Author: wachs
Date: 2012-03-09 11:29:25 +0100 (Fri, 09 Mar 2012)
New Revision: 20395

Modified:
   gnunet/src/namestore/gnunet-service-namestore.c
Log:
- fix


Modified: gnunet/src/namestore/gnunet-service-namestore.c
===================================================================
--- gnunet/src/namestore/gnunet-service-namestore.c     2012-03-09 09:41:46 UTC 
(rev 20394)
+++ gnunet/src/namestore/gnunet-service-namestore.c     2012-03-09 10:29:25 UTC 
(rev 20395)
@@ -298,6 +298,7 @@
   GNUNET_SERVER_client_drop(nc->client);
   GNUNET_CONTAINER_DLL_remove (client_head, client_tail, nc);
   GNUNET_free (nc);
+  nc = NULL;
 }
 
 
@@ -434,8 +435,9 @@
     if (GNUNET_CONTAINER_multihashmap_contains(zonekeys, &zone_key_hash))
     {
       cc = GNUNET_CONTAINER_multihashmap_get(zonekeys, &zone_key_hash);
-      signature_new = GNUNET_NAMESTORE_create_signature(cc->privkey, name, 
rd_selected, copied_elements);
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Creating signature for name `%s' 
in zone `%s'\n",name, GNUNET_h2s(&zone_key_hash));
+      signature_new = GNUNET_NAMESTORE_create_signature(cc->privkey, name, rd, 
rd_count);
+      GNUNET_assert (signature_new != NULL);
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Creating signature for name `%s' 
with %u records in zone `%s'\n",name, copied_elements, 
GNUNET_h2s(&zone_key_hash));
       authoritative = GNUNET_YES;
     }
     else
@@ -466,20 +468,25 @@
     memset(&lnr_msg->public_key, '\0', sizeof (lnr_msg->public_key));
 
   if (GNUNET_YES == authoritative)
-  {
+  { /* use new created signature */
     lnr_msg->contains_sig = htons (GNUNET_YES);
+    GNUNET_assert (signature_new != NULL);
     lnr_msg->signature = *signature_new;
     GNUNET_free (signature_new);
   }
-  if (GNUNET_YES == contains_signature)
+  else if (GNUNET_YES == contains_signature)
   {
+    /* use existing signature */
     lnr_msg->contains_sig = htons (GNUNET_YES);
+    GNUNET_assert (signature != NULL);
     lnr_msg->signature = *signature;
   }
   else
+  {
+    /* use no signature */
     memset (&lnr_msg->signature, '\0', sizeof (lnr_msg->signature));
+  }
 
-
   name_tmp = (char *) &lnr_msg[1];
   rd_tmp = &name_tmp[name_len];
 
@@ -721,7 +728,7 @@
   }
 
   if (exist == GNUNET_SYSERR)
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "NO existing records for `%s' to 
update!\n", crc->name);
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "New record does not exist for name 
`%s'!\n", crc->name);
 
   if (exist == GNUNET_SYSERR)
   {