fixing shutdown timeout issue

diff --git a/TODO b/TODO
index 18cca80371203079ba0328ae071b6ded8a5172f9..c549759cebebab056c9d2967a3dcda0792de0d65 100644 (file)
--- a/TODO
+++ b/TODO
@@ -17,8 +17,12 @@ away), in order in which they will likely be done:
 * HOSTLIST/PEERINFO/CORE/TRANSPORT/TOPOLOGY:
   - hostlist server learns about 2nd peer, but soon forgets!
   - peers do not seem to connect (at least not for long!)
+  
 * TOPOLOGY:
   - needs more testing (especially F2F topology)
+  - calls 'GNUNET_CORE_peer_request_connect' with 'my_identity' as the PID,
+    despite having specific checks to prevent exactly that (!?)
+    => assertion failure in gnunet-service-core:2255 (hostlist test)
 * CORE:
   - needs more testing
 * UTIL:

diff --git a/src/core/gnunet-service-core.c b/src/core/gnunet-service-core.c
index 5acecc5f9380556ae22cc7fd572a29faf4d06746..654f93808c4d39d1e5e1728ab8ae237d2a880146 100644 (file)
--- a/src/core/gnunet-service-core.c
+++ b/src/core/gnunet-service-core.c
@@ -3580,7 +3580,7 @@ cleaning_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
   if (my_private_key != NULL)
     GNUNET_CRYPTO_rsa_key_free (my_private_key);
   if (stats != NULL)
-    GNUNET_STATISTICS_destroy (stats, GNUNET_YES);
+    GNUNET_STATISTICS_destroy (stats, GNUNET_NO);
 }
 
 

diff --git a/src/fs/gnunet-service-fs.c b/src/fs/gnunet-service-fs.c
index 1b3bad6bb2b5645adf6c85799755396512aa2d29..aa0f92f7eeee633023ec80a887638f884ad9d873 100644 (file)
--- a/src/fs/gnunet-service-fs.c
+++ b/src/fs/gnunet-service-fs.c
@@ -957,7 +957,7 @@ shutdown_task (void *cls,
   core = NULL;
   if (stats != NULL)
     {
-      GNUNET_STATISTICS_destroy (stats, GNUNET_YES);
+      GNUNET_STATISTICS_destroy (stats, GNUNET_NO);
       stats = NULL;
     }
   sched = NULL;

diff --git a/src/peerinfo/gnunet-service-peerinfo.c b/src/peerinfo/gnunet-service-peerinfo.c
index 1d84b5dcfc72fda463ffcdc2bb2b77064cc09f6e..5c3f2dfbf5fd1c26f310ba5b02ae30826e331ff3 100644 (file)
--- a/src/peerinfo/gnunet-service-peerinfo.c
+++ b/src/peerinfo/gnunet-service-peerinfo.c
@@ -775,7 +775,7 @@ shutdown_task (void *cls,
   notify_list = NULL;
   if (stats != NULL)
     {
-      GNUNET_STATISTICS_destroy (stats, GNUNET_YES);
+      GNUNET_STATISTICS_destroy (stats, GNUNET_NO);
       stats = NULL;
     }
 }

diff --git a/src/statistics/statistics_api.c b/src/statistics/statistics_api.c
index c8a341b0c8a8d6b69e4d9e8662bad0a781f48b1e..5c5588e918491c37475beeb3b775314be8befb10 100644 (file)
--- a/src/statistics/statistics_api.c
+++ b/src/statistics/statistics_api.c
@@ -32,7 +32,12 @@
 #include "gnunet_strings_lib.h"
 #include "statistics.h"
 
-#define SET_TRANSMIT_TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 30)
+/**
+ * How long do we wait until a statistics request for setting
+ * a value times out?  (The update will be lost if the
+ * service does not react within this timeframe).  
+ */
+#define SET_TRANSMIT_TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_MILLISECONDS, 250)
 
 
 /**

diff --git a/src/topology/gnunet-daemon-topology.c b/src/topology/gnunet-daemon-topology.c
index fa7168cf8bca857ae59ceea0e17d8b4a8ad9eb0a..b466dfbbe3e3b705f77f0ed66c48a30ea65f894a 100644 (file)
--- a/src/topology/gnunet-daemon-topology.c
+++ b/src/topology/gnunet-daemon-topology.c
@@ -1331,7 +1331,7 @@ cleaning_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
     }
   if (stats != NULL)
     {
-      GNUNET_STATISTICS_destroy (stats, GNUNET_YES);
+      GNUNET_STATISTICS_destroy (stats, GNUNET_NO);
       stats = NULL;
     }
 }

diff --git a/src/transport/gnunet-service-transport.c b/src/transport/gnunet-service-transport.c
index c268125e47feb64dea4d8bccf0ba76deeadd46c4..6f689100247e4cadb48cb5528924c6cba85769c3 100644 (file)
--- a/src/transport/gnunet-service-transport.c
+++ b/src/transport/gnunet-service-transport.c
@@ -3373,7 +3373,7 @@ shutdown_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
   validation_map = NULL;
   if (stats != NULL)
     {
-      GNUNET_STATISTICS_destroy (stats, GNUNET_YES);
+      GNUNET_STATISTICS_destroy (stats, GNUNET_NO);
       stats = NULL;
     }
 }

diff --git a/src/transport/transport.h b/src/transport/transport.h
index acaa20a527ff41e502878878d5a1f06bad3b3b58..d66b87b35cc4e4fd88fe8bb9783f1ad9434ed25f 100644 (file)
--- a/src/transport/transport.h
+++ b/src/transport/transport.h
@@ -30,9 +30,9 @@
 #include "gnunet_time_lib.h"
 #include "gnunet_transport_service.h"
 
-#define DEBUG_TRANSPORT GNUNET_NO
-#define DEBUG_TRANSPORT_TIMEOUT GNUNET_NO
-#define DEBUG_TRANSPORT_DISCONNECT GNUNET_NO
+#define DEBUG_TRANSPORT GNUNET_YES
+#define DEBUG_TRANSPORT_TIMEOUT GNUNET_YES
+#define DEBUG_TRANSPORT_DISCONNECT GNUNET_YES
 
 /**
  * For how long do we allow unused bandwidth