[GNUnet-SVN] r19145 - in gnunet/src: exit vpn

[gnunet]

Author: grothoff
Date: 2012-01-15 00:16:53 +0100 (Sun, 15 Jan 2012)
New Revision: 19145

Modified:
   gnunet/src/exit/gnunet-daemon-exit.c
   gnunet/src/vpn/gnunet-service-vpn.c
Log:
-fixing crash, notes

Modified: gnunet/src/exit/gnunet-daemon-exit.c
===================================================================
--- gnunet/src/exit/gnunet-daemon-exit.c        2012-01-14 23:00:49 UTC (rev 
19144)
+++ gnunet/src/exit/gnunet-daemon-exit.c        2012-01-14 23:16:53 UTC (rev 
19145)
@@ -25,7 +25,14 @@
  * @author Christian Grothoff
  *
  * TODO:
+ * - need some logging
+ * - need some statistics
+ * - test
+ *
+ * Code cleanup:
  * - factor out crc computations from DNS/EXIT/VPN into shared library?
+ *
+ * Design:
  * - which code should advertise services? the service model is right
  *   now a bit odd, especially as this code DOES the exit and knows
  *   the DNS "name", but OTOH this is clearly NOT the place to advertise

Modified: gnunet/src/vpn/gnunet-service-vpn.c
===================================================================
--- gnunet/src/vpn/gnunet-service-vpn.c 2012-01-14 23:00:49 UTC (rev 19144)
+++ gnunet/src/vpn/gnunet-service-vpn.c 2012-01-14 23:16:53 UTC (rev 19145)
@@ -28,6 +28,8 @@
  *
  * TODO:
  * Basics:
+ * - need some logging
+ * - need some statistics
  * - test!
  * - better message queue management (bounded state, drop oldest/RED?)
  * - actually destroy "stale" tunnels once we have too many!
@@ -2086,12 +2088,14 @@
 static void
 client_disconnect (void *cls, struct GNUNET_SERVER_Client *client)
 {
-  GNUNET_CONTAINER_multihashmap_iterate (tunnel_map,
-                                        &cleanup_tunnel_client,
-                                        client);
-  GNUNET_CONTAINER_multihashmap_iterate (destination_map,
-                                        &cleanup_destination_client,
-                                        client);
+  if (NULL != tunnel_map)
+    GNUNET_CONTAINER_multihashmap_iterate (tunnel_map,
+                                          &cleanup_tunnel_client,
+                                          client);
+  if (NULL != destination_map)
+    GNUNET_CONTAINER_multihashmap_iterate (destination_map,
+                                          &cleanup_destination_client,
+                                          client);
 }