[GNUnet-SVN] r19603 - gnunet/src/transport

[gnunet]

Author: wachs
Date: 2012-02-01 18:08:21 +0100 (Wed, 01 Feb 2012)
New Revision: 19603

Modified:
   gnunet/src/transport/Makefile.am
   gnunet/src/transport/gnunet-service-transport_blacklist.c
   gnunet/src/transport/test_transport_api_blacklisting.c
   gnunet/src/transport/transport_api_blacklist.c
Log:
fixing and testing blacklisting api and service


Modified: gnunet/src/transport/Makefile.am
===================================================================
--- gnunet/src/transport/Makefile.am    2012-02-01 17:07:27 UTC (rev 19602)
+++ gnunet/src/transport/Makefile.am    2012-02-01 17:08:21 UTC (rev 19603)
@@ -351,7 +351,8 @@
  $(top_builddir)/src/transport/libgnunettransport.la \
  $(top_builddir)/src/hello/libgnunethello.la \
  $(top_builddir)/src/statistics/libgnunetstatistics.la \
- $(top_builddir)/src/util/libgnunetutil.la
+ $(top_builddir)/src/util/libgnunetutil.la \
+ $(top_builddir)/src/transport/libgnunettransporttesting.la 
 
 test_transport_api_disconnect_tcp_SOURCES = \
  test_transport_api_disconnect.c
@@ -378,7 +379,7 @@
  $(top_builddir)/src/hello/libgnunethello.la \
  $(top_builddir)/src/util/libgnunetutil.la  \
  $(top_builddir)/src/transport/libgnunettransporttesting.la
-
+  
 test_transport_api_bidirectional_connect_SOURCES = \
  test_transport_api_bidirectional_connect.c
 test_transport_api_bidirectional_connect_LDADD = \
@@ -404,7 +405,7 @@
  $(top_builddir)/src/statistics/libgnunetstatistics.la \
  $(top_builddir)/src/util/libgnunetutil.la \
  $(top_builddir)/src/transport/libgnunettransporttesting.la 
-
+  
 test_transport_api_limited_sockets_tcp_SOURCES = \
  test_transport_api_limited_sockets.c
 test_transport_api_limited_sockets_tcp_LDADD = \
@@ -444,7 +445,7 @@
  $(top_builddir)/src/hello/libgnunethello.la \
  $(top_builddir)/src/util/libgnunetutil.la \
  $(top_builddir)/src/transport/libgnunettransporttesting.la 
-
+ 
 test_transport_api_timeout_https_SOURCES = \
  test_transport_api_timeout.c
 test_transport_api_timeout_https_LDADD = \
@@ -452,7 +453,7 @@
  $(top_builddir)/src/hello/libgnunethello.la \
  $(top_builddir)/src/util/libgnunetutil.la \
  $(top_builddir)/src/transport/libgnunettransporttesting.la 
-
+ 
 test_transport_api_reliability_tcp_nat_SOURCES = \
  test_transport_api_reliability.c
 test_transport_api_reliability_tcp_nat_LDADD = \
@@ -516,7 +517,7 @@
  $(top_builddir)/src/hello/libgnunethello.la \
  $(top_builddir)/src/util/libgnunetutil.la \
  $(top_builddir)/src/transport/libgnunettransporttesting.la 
-
+ 
  test_transport_api_http_nat_SOURCES = \
  test_transport_api.c
 test_transport_api_http_nat_LDADD = \

Modified: gnunet/src/transport/gnunet-service-transport_blacklist.c
===================================================================
--- gnunet/src/transport/gnunet-service-transport_blacklist.c   2012-02-01 
17:07:27 UTC (rev 19602)
+++ gnunet/src/transport/gnunet-service-transport_blacklist.c   2012-02-01 
17:08:21 UTC (rev 19603)
@@ -489,6 +489,7 @@
                 GNUNET_i2s (&bc->peer));
 #endif
     bc->cont (bc->cont_cls, &bc->peer, GNUNET_OK);
+    GNUNET_CONTAINER_DLL_remove(bc_head, bc_tail, bc);
     GNUNET_free (bc);
     return;
   }

Modified: gnunet/src/transport/test_transport_api_blacklisting.c
===================================================================
--- gnunet/src/transport/test_transport_api_blacklisting.c      2012-02-01 
17:07:27 UTC (rev 19602)
+++ gnunet/src/transport/test_transport_api_blacklisting.c      2012-02-01 
17:08:21 UTC (rev 19603)
@@ -35,7 +35,7 @@
 #include "transport.h"
 #include "transport-testing.h"
 
-#define VERBOSE GNUNET_YES
+#define VERBOSE GNUNET_EXTRA_LOGGING
 #define VERBOSE_ARM GNUNET_EXTRA_LOGGING
 
 #define START_ARM GNUNET_YES
@@ -67,6 +67,9 @@
 
 
 static int ok;
+static int connected;
+static int blacklist_request_p1;
+static int blacklist_request_p2;
 
 struct GNUNET_TRANSPORT_Blacklist * blacklist_p1;
 
@@ -76,6 +79,8 @@
 
 static GNUNET_SCHEDULER_TaskIdentifier send_task;
 
+static GNUNET_SCHEDULER_TaskIdentifier shutdown_task;
+
 #if VERBOSE
 #define OKPP do { ok++; FPRINTF (stderr, "Now at stage %u at %s:%u\n", ok, 
__FILE__, __LINE__); } while (0)
 #else
@@ -84,7 +89,7 @@
 
 
 static void
-end ()
+end (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Stopping\n");
 
@@ -92,28 +97,73 @@
     GNUNET_SCHEDULER_cancel (send_task);
 
   if (die_task != GNUNET_SCHEDULER_NO_TASK)
+  {
     GNUNET_SCHEDULER_cancel (die_task);
+    die_task = GNUNET_SCHEDULER_NO_TASK;
+  }
 
+  if (cc != NULL)
+  {
+    GNUNET_TRANSPORT_TESTING_connect_peers_cancel(tth, cc);
+    cc = NULL;
+  }
+
   if (th != NULL)
     GNUNET_TRANSPORT_notify_transmit_ready_cancel (th);
   th = NULL;
 
-  GNUNET_TRANSPORT_blacklist_cancel (blacklist_p1);
+  if (blacklist_p1 != NULL)
+  {
+    GNUNET_TRANSPORT_blacklist_cancel (blacklist_p1);
+    blacklist_p1 = NULL;
+  }
 
-  GNUNET_TRANSPORT_blacklist_cancel (blacklist_p2);
+  if (blacklist_p2 != NULL)
+  {
+    GNUNET_TRANSPORT_blacklist_cancel (blacklist_p2);
+    blacklist_p2 = NULL;
+  }
 
-  GNUNET_TRANSPORT_TESTING_stop_peer (tth, p1);
-  GNUNET_TRANSPORT_TESTING_stop_peer (tth, p2);
+  if (p1 != NULL)
+  {
+    GNUNET_TRANSPORT_TESTING_stop_peer (tth, p1);
+    p1 = NULL;
+  }
+  if (p2 != NULL)
+  {
+    GNUNET_TRANSPORT_TESTING_stop_peer (tth, p2);
+    p2 = NULL;
+  }
 
-  ok = 0;
+  if ((blacklist_request_p1 == GNUNET_YES) &&
+      (blacklist_request_p2 == GNUNET_YES) &&
+      (connected == GNUNET_NO))
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, _("Peers were not connected, 
success\n"));
+    ok = 0;
+  }
+  else
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, _("Peers were not connected, 
fail\n"));
+    ok = 1;
+  }
 }
 
 static void
 end_badly (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   if (send_task != GNUNET_SCHEDULER_NO_TASK)
+  {
     GNUNET_SCHEDULER_cancel (send_task);
+    send_task = GNUNET_SCHEDULER_NO_TASK;
+  }
 
+  if (shutdown_task != GNUNET_SCHEDULER_NO_TASK)
+  {
+    GNUNET_SCHEDULER_cancel (shutdown_task);
+    shutdown_task = GNUNET_SCHEDULER_NO_TASK;
+  }
+
   if (cc != NULL)
   {
     GNUNET_log (GNUNET_ERROR_TYPE_ERROR, _("Fail! Could not connect peers\n"));
@@ -166,13 +216,13 @@
       (MSIZE == ntohs (message->size)))
   {
     ok = 0;
-    end ();
+    shutdown_task = GNUNET_SCHEDULER_add_now(&end, NULL);
   }
   else
   {
     GNUNET_break (0);
     ok = 1;
-    end ();
+    shutdown_task = GNUNET_SCHEDULER_add_now(&end, NULL);
   }
 }
 
@@ -190,7 +240,10 @@
     GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
                 "Timeout occurred while waiting for transmit_ready\n");
     if (GNUNET_SCHEDULER_NO_TASK != die_task)
+    {
       GNUNET_SCHEDULER_cancel (die_task);
+      die_task = GNUNET_SCHEDULER_NO_TASK;
+    }
     die_task = GNUNET_SCHEDULER_add_now (&end_badly, NULL);
     ok = 42;
     return 0;
@@ -286,6 +339,8 @@
               p1->no, p1_c, p2->no, GNUNET_i2s (&p2->id));
   GNUNET_free (p1_c);
 
+
+
   send_task = GNUNET_SCHEDULER_add_now (&sendtask, NULL);
 }
 
@@ -294,8 +349,27 @@
                  const struct
                  GNUNET_PeerIdentity * pid)
 {
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Blacklist request\n", GNUNET_i2s 
(pid));
-  return GNUNET_YES;
+  struct PeerContext * p = cls;
+  int res = GNUNET_SYSERR;
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Peer  %u : Blacklist request for peer 
`%s'\n", p->no, GNUNET_i2s (pid));
+
+  if (p == p1)
+  {
+    blacklist_request_p1 = GNUNET_YES;
+    res = GNUNET_OK;
+  }
+  else if (p == p2)
+  {
+    blacklist_request_p2 = GNUNET_YES;
+    res = GNUNET_SYSERR;
+  }
+
+  if (((blacklist_request_p2 == GNUNET_YES) && (blacklist_request_p1 == 
GNUNET_YES)) && (shutdown_task == GNUNET_SCHEDULER_NO_TASK))
+  {
+    shutdown_task = GNUNET_SCHEDULER_add_delayed 
(GNUNET_TIME_relative_multiply(GNUNET_TIME_UNIT_SECONDS, 3), &end, NULL);
+  }
+
+  return res;
 }
 
 void
@@ -328,6 +402,9 @@
      const struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   die_task = GNUNET_SCHEDULER_add_delayed (TIMEOUT, &end_badly, NULL);
+  connected = GNUNET_NO;
+  blacklist_request_p1 = GNUNET_NO;
+  blacklist_request_p2 = GNUNET_NO;
 
   p1 = GNUNET_TRANSPORT_TESTING_start_peer (tth, 
"test_transport_api_tcp_peer1.conf", 1,
                                             &notify_receive, &notify_connect,
@@ -343,7 +420,7 @@
                               &blacklist_cb,
                               p1);
 
-  blacklist_p2 = GNUNET_TRANSPORT_blacklist (p1->cfg,
+  blacklist_p2 = GNUNET_TRANSPORT_blacklist (p2->cfg,
                               &blacklist_cb,
                               p2);
 

Modified: gnunet/src/transport/transport_api_blacklist.c
===================================================================
--- gnunet/src/transport/transport_api_blacklist.c      2012-02-01 17:07:27 UTC 
(rev 19602)
+++ gnunet/src/transport/transport_api_blacklist.c      2012-02-01 17:08:21 UTC 
(rev 19603)
@@ -145,6 +145,7 @@
   struct GNUNET_TRANSPORT_Blacklist *br = cls;
   struct GNUNET_MessageHeader req;
 
+  br->th = NULL;
   if (buf == NULL)
   {
     reconnect (br);
@@ -153,7 +154,6 @@
   req.size = htons (sizeof (struct GNUNET_MessageHeader));
   req.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_BLACKLIST_INIT);
   memcpy (buf, &req, sizeof (req));
-  br->th = NULL;
   receive (br);
   return sizeof (req);
 }
@@ -194,6 +194,7 @@
   struct GNUNET_TRANSPORT_Blacklist *br = cls;
   struct BlacklistMessage req;
 
+  br->th = NULL;
   if (buf == NULL)
   {
     reconnect (br);
@@ -218,6 +219,7 @@
 static void
 reply (struct GNUNET_TRANSPORT_Blacklist *br)
 {
+  GNUNET_assert (br->th == NULL);
   br->th =
       GNUNET_CLIENT_notify_transmit_ready (br->client,
                                            sizeof (struct BlacklistMessage),
@@ -261,6 +263,7 @@
   ret->cfg = cfg;
   ret->cb = cb;
   ret->cb_cls = cb_cls;
+  GNUNET_assert (ret->th == NULL);
   ret->th =
       GNUNET_CLIENT_notify_transmit_ready (client,
                                            sizeof (struct 
GNUNET_MessageHeader),
@@ -281,7 +284,10 @@
 GNUNET_TRANSPORT_blacklist_cancel (struct GNUNET_TRANSPORT_Blacklist *br)
 {
   if (br->th != NULL)
+  {
     GNUNET_CLIENT_notify_transmit_ready_cancel (br->th);
+    br->th = NULL;
+  }
   GNUNET_CLIENT_disconnect (br->client, GNUNET_NO);
   GNUNET_free (br);
 }