1 files changed, 583 insertions, 579 deletions

diff --git a/src/cadet/gnunet-service-cadet_core.c b/src/cadet/gnunet-service-cadet_core.c
index a08c80aae..7d1791f3c 100644
--- a/src/cadet/gnunet-service-cadet_core.c
+++ b/src/cadet/gnunet-service-cadet_core.c
@@ -39,7 +39,7 @@
 #include "gnunet_statistics_service.h"
 #include "cadet_protocol.h"
 
-#define LOG(level, ...) GNUNET_log_from(level, "cadet-cor", __VA_ARGS__)
+#define LOG(level, ...) GNUNET_log_from (level, "cadet-cor", __VA_ARGS__)
 
 /**
  * Information we keep per direction for a route.
@@ -52,7 +52,8 @@ struct RouteDirection;
  * routes that have the current maximum of messages in the buffer (in
  * case we have to purge).
  */
-struct Rung {
+struct Rung
+{
   /**
    * Rung of RouteDirections with one more buffer entry each.
    */
@@ -89,7 +90,8 @@ struct Rung {
 /**
  * Information we keep per direction for a route.
  */
-struct RouteDirection {
+struct RouteDirection
+{
   /**
    * DLL of other route directions within the same `struct Rung`.
    */
@@ -145,7 +147,8 @@ struct RouteDirection {
  * the complete `struct CadetPath` that is formed by the individual
  * routes.
  */
-struct CadetRoute {
+struct CadetRoute
+{
   /**
    * Information about the next hop on this route.
    */
@@ -231,10 +234,10 @@ static struct GNUNET_SCHEDULER_Task *timeout_task;
  * @param cid hash generated from the connection identifier
  */
 static struct CadetRoute *
-get_route(const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid)
+get_route (const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid)
 {
-  return GNUNET_CONTAINER_multishortmap_get(routes,
-                                            &cid->connection_of_tunnel);
+  return GNUNET_CONTAINER_multishortmap_get (routes,
+                                             &cid->connection_of_tunnel);
 }
 
 
@@ -244,22 +247,22 @@ get_route(const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid)
  * @param dir direction to lower in rung.
  */
 static void
-lower_rung(struct RouteDirection *dir)
+lower_rung (struct RouteDirection *dir)
 {
   struct Rung *rung = dir->rung;
   struct Rung *prev;
 
-  GNUNET_CONTAINER_DLL_remove(rung->rd_head, rung->rd_tail, dir);
+  GNUNET_CONTAINER_DLL_remove (rung->rd_head, rung->rd_tail, dir);
   prev = rung->prev;
-  GNUNET_assert(NULL != prev);
+  GNUNET_assert (NULL != prev);
   if (prev->rung_off != rung->rung_off - 1)
-    {
-      prev = GNUNET_new(struct Rung);
-      prev->rung_off = rung->rung_off - 1;
-      GNUNET_CONTAINER_DLL_insert_after(rung_head, rung_tail, rung->prev, prev);
-    }
-  GNUNET_assert(NULL != prev);
-  GNUNET_CONTAINER_DLL_insert(prev->rd_head, prev->rd_tail, dir);
+  {
+    prev = GNUNET_new (struct Rung);
+    prev->rung_off = rung->rung_off - 1;
+    GNUNET_CONTAINER_DLL_insert_after (rung_head, rung_tail, rung->prev, prev);
+  }
+  GNUNET_assert (NULL != prev);
+  GNUNET_CONTAINER_DLL_insert (prev->rd_head, prev->rd_tail, dir);
   dir->rung = prev;
 }
 
@@ -272,13 +275,13 @@ lower_rung(struct RouteDirection *dir)
  * @param env envelope to discard
  */
 static void
-discard_buffer(struct RouteDirection *dir, struct GNUNET_MQ_Envelope *env)
+discard_buffer (struct RouteDirection *dir, struct GNUNET_MQ_Envelope *env)
 {
-  GNUNET_MQ_dll_remove(&dir->env_head, &dir->env_tail, env);
+  GNUNET_MQ_dll_remove (&dir->env_head, &dir->env_tail, env);
   cur_buffers--;
-  GNUNET_MQ_discard(env);
-  lower_rung(dir);
-  GNUNET_STATISTICS_set(stats, "# buffer use", cur_buffers, GNUNET_NO);
+  GNUNET_MQ_discard (env);
+  lower_rung (dir);
+  GNUNET_STATISTICS_set (stats, "# buffer use", cur_buffers, GNUNET_NO);
 }
 
 
@@ -286,24 +289,24 @@ discard_buffer(struct RouteDirection *dir, struct GNUNET_MQ_Envelope *env)
  * Discard all messages from the highest rung, to make space.
  */
 static void
-discard_all_from_rung_tail()
+discard_all_from_rung_tail ()
 {
   struct Rung *tail = rung_tail;
   struct RouteDirection *dir;
 
   while (NULL != (dir = tail->rd_head))
-    {
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Queue full due new message %s on connection %s, dropping old message\n",
-          GNUNET_sh2s(&dir->my_route->cid.connection_of_tunnel));
-      GNUNET_STATISTICS_update(stats,
-                               "# messages dropped due to full buffer",
-                               1,
-                               GNUNET_NO);
-      discard_buffer(dir, dir->env_head);
-    }
-  GNUNET_CONTAINER_DLL_remove(rung_head, rung_tail, tail);
-  GNUNET_free(tail);
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Queue full due new message %s on connection %s, dropping old message\n",
+         GNUNET_sh2s (&dir->my_route->cid.connection_of_tunnel));
+    GNUNET_STATISTICS_update (stats,
+                              "# messages dropped due to full buffer",
+                              1,
+                              GNUNET_NO);
+    discard_buffer (dir, dir->env_head);
+  }
+  GNUNET_CONTAINER_DLL_remove (rung_head, rung_tail, tail);
+  GNUNET_free (tail);
 }
 
 
@@ -317,10 +320,10 @@ discard_all_from_rung_tail()
  * @param msg the message to forward
  */
 static void
-route_message(struct CadetPeer *prev,
-              const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid,
-              const struct GNUNET_MessageHeader *msg,
-              const enum GNUNET_MQ_PriorityPreferences priority)
+route_message (struct CadetPeer *prev,
+               const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid,
+               const struct GNUNET_MessageHeader *msg,
+               const enum GNUNET_MQ_PriorityPreferences priority)
 {
   struct CadetRoute *route;
   struct RouteDirection *dir;
@@ -328,133 +331,133 @@ route_message(struct CadetPeer *prev,
   struct Rung *nxt;
   struct GNUNET_MQ_Envelope *env;
 
-  route = get_route(cid);
+  route = get_route (cid);
   if (NULL == route)
+  {
+    struct GNUNET_MQ_Envelope *env;
+    struct GNUNET_CADET_ConnectionBrokenMessage *bm;
+
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Failed to route message of type %u from %s on connection %s: no route\n",
+         ntohs (msg->type),
+         GCP_2s (prev),
+         GNUNET_sh2s (&cid->connection_of_tunnel));
+    switch (ntohs (msg->type))
     {
-      struct GNUNET_MQ_Envelope *env;
-      struct GNUNET_CADET_ConnectionBrokenMessage *bm;
-
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Failed to route message of type %u from %s on connection %s: no route\n",
-          ntohs(msg->type),
-          GCP_2s(prev),
-          GNUNET_sh2s(&cid->connection_of_tunnel));
-      switch (ntohs(msg->type))
-        {
-        case GNUNET_MESSAGE_TYPE_CADET_CONNECTION_DESTROY:
-        case GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN:
-          /* No need to respond to these! */
-          return;
-        }
-      env = GNUNET_MQ_msg(bm, GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN);
-      bm->cid = *cid;
-      bm->peer1 = my_full_id;
-      GCP_send_ooo(prev, env);
+    case GNUNET_MESSAGE_TYPE_CADET_CONNECTION_DESTROY:
+    case GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN:
+      /* No need to respond to these! */
       return;
     }
-  route->last_use = GNUNET_TIME_absolute_get();
-  GNUNET_CONTAINER_heap_update_cost(route->hn, route->last_use.abs_value_us);
+    env = GNUNET_MQ_msg (bm, GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN);
+    bm->cid = *cid;
+    bm->peer1 = my_full_id;
+    GCP_send_ooo (prev, env);
+    return;
+  }
+  route->last_use = GNUNET_TIME_absolute_get ();
+  GNUNET_CONTAINER_heap_update_cost (route->hn, route->last_use.abs_value_us);
   dir = (prev == route->prev.hop) ? &route->next : &route->prev;
   if (GNUNET_YES == dir->is_ready)
-    {
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Routing message of type %u from %s to %s on connection %s\n",
-          ntohs(msg->type),
-          GCP_2s(prev),
-          GNUNET_i2s(GCP_get_id(dir->hop)),
-          GNUNET_sh2s(&cid->connection_of_tunnel));
-      dir->is_ready = GNUNET_NO;
-      GCP_send(dir->mqm, GNUNET_MQ_msg_copy(msg));
-      return;
-    }
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Routing message of type %u from %s to %s on connection %s\n",
+         ntohs (msg->type),
+         GCP_2s (prev),
+         GNUNET_i2s (GCP_get_id (dir->hop)),
+         GNUNET_sh2s (&cid->connection_of_tunnel));
+    dir->is_ready = GNUNET_NO;
+    GCP_send (dir->mqm, GNUNET_MQ_msg_copy (msg));
+    return;
+  }
   /* Check if low latency is required and if the previous message was
      unreliable; if so, make sure we only queue one message per
      direction (no buffering). */
   if ((0 != (priority & GNUNET_MQ_PREF_LOW_LATENCY)) &&
       (NULL != dir->env_head) &&
       (0 ==
-       (GNUNET_MQ_env_get_options(dir->env_head) & GNUNET_MQ_PREF_UNRELIABLE)))
-    discard_buffer(dir, dir->env_head);
+       (GNUNET_MQ_env_get_options (dir->env_head) & GNUNET_MQ_PREF_UNRELIABLE)))
+    discard_buffer (dir, dir->env_head);
   /* Check for duplicates */
   for (const struct GNUNET_MQ_Envelope *env = dir->env_head; NULL != env;
-       env = GNUNET_MQ_env_next(env))
+       env = GNUNET_MQ_env_next (env))
+  {
+    const struct GNUNET_MessageHeader *hdr = GNUNET_MQ_env_get_msg (env);
+
+    if ((hdr->size == msg->size) && (0 == memcmp (hdr, msg, ntohs (msg->size))))
     {
-      const struct GNUNET_MessageHeader *hdr = GNUNET_MQ_env_get_msg(env);
-
-      if ((hdr->size == msg->size) && (0 == memcmp(hdr, msg, ntohs(msg->size))))
-        {
-          LOG(GNUNET_ERROR_TYPE_DEBUG,
-              "Received duplicate of message already in buffer, dropping\n");
-          GNUNET_STATISTICS_update(stats,
-                                   "# messages dropped due to duplicate in buffer",
-                                   1,
-                                   GNUNET_NO);
-          return;
-        }
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "Received duplicate of message already in buffer, dropping\n");
+      GNUNET_STATISTICS_update (stats,
+                                "# messages dropped due to duplicate in buffer",
+                                1,
+                                GNUNET_NO);
+      return;
     }
+  }
 
   rung = dir->rung;
   if (cur_buffers == max_buffers)
+  {
+    /* Need to make room. */
+    if (NULL != rung->next)
     {
-      /* Need to make room. */
-      if (NULL != rung->next)
-        {
-          /* Easy case, drop messages from route directions in highest rung */
-          discard_all_from_rung_tail();
-        }
-      else
-        {
-          /* We are in the highest rung, drop our own! */
-          LOG(GNUNET_ERROR_TYPE_DEBUG,
-              "Queue full due new message %s on connection %s, dropping old message\n",
-              GNUNET_sh2s(&dir->my_route->cid.connection_of_tunnel));
-          GNUNET_STATISTICS_update(stats,
-                                   "# messages dropped due to full buffer",
-                                   1,
-                                   GNUNET_NO);
-          discard_buffer(dir, dir->env_head);
-          rung = dir->rung;
-        }
+      /* Easy case, drop messages from route directions in highest rung */
+      discard_all_from_rung_tail ();
     }
+    else
+    {
+      /* We are in the highest rung, drop our own! */
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "Queue full due new message %s on connection %s, dropping old message\n",
+           GNUNET_sh2s (&dir->my_route->cid.connection_of_tunnel));
+      GNUNET_STATISTICS_update (stats,
+                                "# messages dropped due to full buffer",
+                                1,
+                                GNUNET_NO);
+      discard_buffer (dir, dir->env_head);
+      rung = dir->rung;
+    }
+  }
   /* remove 'dir' from current rung */
-  GNUNET_CONTAINER_DLL_remove(rung->rd_head, rung->rd_tail, dir);
+  GNUNET_CONTAINER_DLL_remove (rung->rd_head, rung->rd_tail, dir);
   /* make 'nxt' point to the next higher rung, create if necessary */
   nxt = rung->next;
   if ((NULL == nxt) || (rung->rung_off + 1 != nxt->rung_off))
-    {
-      nxt = GNUNET_new(struct Rung);
-      nxt->rung_off = rung->rung_off + 1;
-      GNUNET_CONTAINER_DLL_insert_after(rung_head, rung_tail, rung, nxt);
-    }
+  {
+    nxt = GNUNET_new (struct Rung);
+    nxt->rung_off = rung->rung_off + 1;
+    GNUNET_CONTAINER_DLL_insert_after (rung_head, rung_tail, rung, nxt);
+  }
   /* insert 'dir' into next higher rung */
-  GNUNET_CONTAINER_DLL_insert(nxt->rd_head, nxt->rd_tail, dir);
+  GNUNET_CONTAINER_DLL_insert (nxt->rd_head, nxt->rd_tail, dir);
   dir->rung = nxt;
 
   /* add message into 'dir' buffer */
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Queueing new message of type %u from %s to %s on connection %s\n",
-      ntohs(msg->type),
-      GCP_2s(prev),
-      GNUNET_i2s(GCP_get_id(dir->hop)),
-      GNUNET_sh2s(&cid->connection_of_tunnel));
-  env = GNUNET_MQ_msg_copy(msg);
-  GNUNET_MQ_env_set_options(env, priority);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Queueing new message of type %u from %s to %s on connection %s\n",
+       ntohs (msg->type),
+       GCP_2s (prev),
+       GNUNET_i2s (GCP_get_id (dir->hop)),
+       GNUNET_sh2s (&cid->connection_of_tunnel));
+  env = GNUNET_MQ_msg_copy (msg);
+  GNUNET_MQ_env_set_options (env, priority);
   if ((0 != (priority & GNUNET_MQ_PREF_LOW_LATENCY)) &&
       (0 != (priority & GNUNET_MQ_PREF_OUT_OF_ORDER)) &&
       (NULL != dir->env_head) &&
-      (0 == (GNUNET_MQ_env_get_options(dir->env_head) &
-             GNUNET_MQ_PREF_LOW_LATENCY)))
-    GNUNET_MQ_dll_insert_head(&dir->env_head, &dir->env_tail, env);
+      (0 == (GNUNET_MQ_env_get_options (dir->env_head)
+             & GNUNET_MQ_PREF_LOW_LATENCY)))
+    GNUNET_MQ_dll_insert_head (&dir->env_head, &dir->env_tail, env);
   else
-    GNUNET_MQ_dll_insert_tail(&dir->env_head, &dir->env_tail, env);
+    GNUNET_MQ_dll_insert_tail (&dir->env_head, &dir->env_tail, env);
   cur_buffers++;
-  GNUNET_STATISTICS_set(stats, "# buffer use", cur_buffers, GNUNET_NO);
+  GNUNET_STATISTICS_set (stats, "# buffer use", cur_buffers, GNUNET_NO);
   /* Clean up 'rung' if now empty (and not head) */
   if ((NULL == rung->rd_head) && (rung != rung_head))
-    {
-      GNUNET_CONTAINER_DLL_remove(rung_head, rung_tail, rung);
-      GNUNET_free(rung);
-    }
+  {
+    GNUNET_CONTAINER_DLL_remove (rung_head, rung_tail, rung);
+    GNUNET_free (rung);
+  }
 }
 
 
@@ -467,16 +470,16 @@ route_message(struct CadetPeer *prev,
  * @return #GNUNET_YES if size is correct, #GNUNET_NO otherwise.
  */
 static int
-check_connection_create(void *cls,
-                        const struct GNUNET_CADET_ConnectionCreateMessage *msg)
+check_connection_create (void *cls,
+                         const struct GNUNET_CADET_ConnectionCreateMessage *msg)
 {
-  uint16_t size = ntohs(msg->header.size) - sizeof(*msg);
+  uint16_t size = ntohs (msg->header.size) - sizeof(*msg);
 
   if (0 != (size % sizeof(struct GNUNET_PeerIdentity)))
-    {
-      GNUNET_break_op(0);
-      return GNUNET_NO;
-    }
+  {
+    GNUNET_break_op (0);
+    return GNUNET_NO;
+  }
   return GNUNET_YES;
 }
 
@@ -487,24 +490,24 @@ check_connection_create(void *cls,
  * @param dir direction to destroy (do NOT free memory of 'dir' itself)
  */
 static void
-destroy_direction(struct RouteDirection *dir)
+destroy_direction (struct RouteDirection *dir)
 {
   struct GNUNET_MQ_Envelope *env;
 
   while (NULL != (env = dir->env_head))
-    {
-      GNUNET_STATISTICS_update(stats,
-                               "# messages dropped due to route destruction",
-                               1,
-                               GNUNET_NO);
-      discard_buffer(dir, env);
-    }
+  {
+    GNUNET_STATISTICS_update (stats,
+                              "# messages dropped due to route destruction",
+                              1,
+                              GNUNET_NO);
+    discard_buffer (dir, env);
+  }
   if (NULL != dir->mqm)
-    {
-      GCP_request_mq_cancel(dir->mqm, NULL);
-      dir->mqm = NULL;
-    }
-  GNUNET_CONTAINER_DLL_remove(rung_head->rd_head, rung_head->rd_tail, dir);
+  {
+    GCP_request_mq_cancel (dir->mqm, NULL);
+    dir->mqm = NULL;
+  }
+  GNUNET_CONTAINER_DLL_remove (rung_head->rd_head, rung_head->rd_tail, dir);
 }
 
 
@@ -514,26 +517,26 @@ destroy_direction(struct RouteDirection *dir)
  * @param route route to destroy
  */
 static void
-destroy_route(struct CadetRoute *route)
+destroy_route (struct CadetRoute *route)
 {
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Destroying route from %s to %s of connection %s\n",
-      GNUNET_i2s(GCP_get_id(route->prev.hop)),
-      GNUNET_i2s2(GCP_get_id(route->next.hop)),
-      GNUNET_sh2s(&route->cid.connection_of_tunnel));
-  GNUNET_assert(route == GNUNET_CONTAINER_heap_remove_node(route->hn));
-  GNUNET_assert(
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Destroying route from %s to %s of connection %s\n",
+       GNUNET_i2s (GCP_get_id (route->prev.hop)),
+       GNUNET_i2s2 (GCP_get_id (route->next.hop)),
+       GNUNET_sh2s (&route->cid.connection_of_tunnel));
+  GNUNET_assert (route == GNUNET_CONTAINER_heap_remove_node (route->hn));
+  GNUNET_assert (
     GNUNET_YES ==
-    GNUNET_CONTAINER_multishortmap_remove(routes,
-                                          &route->cid.connection_of_tunnel,
-                                          route));
-  GNUNET_STATISTICS_set(stats,
-                        "# routes",
-                        GNUNET_CONTAINER_multishortmap_size(routes),
-                        GNUNET_NO);
-  destroy_direction(&route->prev);
-  destroy_direction(&route->next);
-  GNUNET_free(route);
+    GNUNET_CONTAINER_multishortmap_remove (routes,
+                                           &route->cid.connection_of_tunnel,
+                                           route));
+  GNUNET_STATISTICS_set (stats,
+                         "# routes",
+                         GNUNET_CONTAINER_multishortmap_size (routes),
+                         GNUNET_NO);
+  destroy_direction (&route->prev);
+  destroy_direction (&route->next);
+  GNUNET_free (route);
 }
 
 
@@ -546,30 +549,30 @@ destroy_route(struct CadetRoute *route)
  * @param peer2 another one of the peers where a link is broken
  */
 static void
-send_broken(struct RouteDirection *target,
-            const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid,
-            const struct GNUNET_PeerIdentity *peer1,
-            const struct GNUNET_PeerIdentity *peer2)
+send_broken (struct RouteDirection *target,
+             const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid,
+             const struct GNUNET_PeerIdentity *peer1,
+             const struct GNUNET_PeerIdentity *peer2)
 {
   struct GNUNET_MQ_Envelope *env;
   struct GNUNET_CADET_ConnectionBrokenMessage *bm;
 
   if (NULL == target->mqm)
     return; /* Can't send notification, connection is down! */
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Notifying %s about BROKEN route at %s-%s of connection %s\n",
-      GCP_2s(target->hop),
-      GNUNET_i2s(peer1),
-      GNUNET_i2s2(peer2),
-      GNUNET_sh2s(&cid->connection_of_tunnel));
-
-  env = GNUNET_MQ_msg(bm, GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Notifying %s about BROKEN route at %s-%s of connection %s\n",
+       GCP_2s (target->hop),
+       GNUNET_i2s (peer1),
+       GNUNET_i2s2 (peer2),
+       GNUNET_sh2s (&cid->connection_of_tunnel));
+
+  env = GNUNET_MQ_msg (bm, GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN);
   bm->cid = *cid;
   if (NULL != peer1)
     bm->peer1 = *peer1;
   if (NULL != peer2)
     bm->peer2 = *peer2;
-  GCP_request_mq_cancel(target->mqm, env);
+  GCP_request_mq_cancel (target->mqm, env);
   target->mqm = NULL;
 }
 
@@ -582,31 +585,31 @@ send_broken(struct RouteDirection *target,
  * @param cls NULL
  */
 static void
-timeout_cb(void *cls)
+timeout_cb (void *cls)
 {
   struct CadetRoute *r;
   struct GNUNET_TIME_Relative linger;
   struct GNUNET_TIME_Absolute exp;
 
   timeout_task = NULL;
-  linger = GNUNET_TIME_relative_multiply(keepalive_period, 3);
-  while (NULL != (r = GNUNET_CONTAINER_heap_peek(route_heap)))
+  linger = GNUNET_TIME_relative_multiply (keepalive_period, 3);
+  while (NULL != (r = GNUNET_CONTAINER_heap_peek (route_heap)))
+  {
+    exp = GNUNET_TIME_absolute_add (r->last_use, linger);
+    if (0 != GNUNET_TIME_absolute_get_remaining (exp).rel_value_us)
     {
-      exp = GNUNET_TIME_absolute_add(r->last_use, linger);
-      if (0 != GNUNET_TIME_absolute_get_remaining(exp).rel_value_us)
-        {
-          /* Route not yet timed out, wait until it does. */
-          timeout_task = GNUNET_SCHEDULER_add_at(exp, &timeout_cb, NULL);
-          return;
-        }
-      GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-                 "Sending BROKEN due to timeout (%s was last use, %s linger)\n",
-                 GNUNET_STRINGS_absolute_time_to_string(r->last_use),
-                 GNUNET_STRINGS_relative_time_to_string(linger, GNUNET_YES));
-      send_broken(&r->prev, &r->cid, NULL, NULL);
-      send_broken(&r->next, &r->cid, NULL, NULL);
-      destroy_route(r);
+      /* Route not yet timed out, wait until it does. */
+      timeout_task = GNUNET_SCHEDULER_add_at (exp, &timeout_cb, NULL);
+      return;
     }
+    GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+                "Sending BROKEN due to timeout (%s was last use, %s linger)\n",
+                GNUNET_STRINGS_absolute_time_to_string (r->last_use),
+                GNUNET_STRINGS_relative_time_to_string (linger, GNUNET_YES));
+    send_broken (&r->prev, &r->cid, NULL, NULL);
+    send_broken (&r->next, &r->cid, NULL, NULL);
+    destroy_route (r);
+  }
   /* No more routes left, so no need for a #timeout_task */
 }
 
@@ -624,32 +627,32 @@ timeout_cb(void *cls)
  *                                 and the last envelope was discarded
  */
 static void
-dir_ready_cb(void *cls, int ready)
+dir_ready_cb (void *cls, int ready)
 {
   struct RouteDirection *dir = cls;
   struct CadetRoute *route = dir->my_route;
   struct RouteDirection *odir;
 
   if (GNUNET_YES == ready)
+  {
+    struct GNUNET_MQ_Envelope *env;
+
+    dir->is_ready = GNUNET_YES;
+    if (NULL != (env = dir->env_head))
     {
-      struct GNUNET_MQ_Envelope *env;
-
-      dir->is_ready = GNUNET_YES;
-      if (NULL != (env = dir->env_head))
-        {
-          GNUNET_MQ_dll_remove(&dir->env_head, &dir->env_tail, env);
-          cur_buffers--;
-          GNUNET_STATISTICS_set(stats, "# buffer use", cur_buffers, GNUNET_NO);
-          lower_rung(dir);
-          dir->is_ready = GNUNET_NO;
-          GCP_send(dir->mqm, env);
-        }
-      return;
+      GNUNET_MQ_dll_remove (&dir->env_head, &dir->env_tail, env);
+      cur_buffers--;
+      GNUNET_STATISTICS_set (stats, "# buffer use", cur_buffers, GNUNET_NO);
+      lower_rung (dir);
+      dir->is_ready = GNUNET_NO;
+      GCP_send (dir->mqm, env);
     }
+    return;
+  }
   odir = (dir == &route->next) ? &route->prev : &route->next;
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG, "Sending BROKEN due to MQ going down\n");
-  send_broken(&route->next, &route->cid, GCP_get_id(odir->hop), &my_full_id);
-  destroy_route(route);
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending BROKEN due to MQ going down\n");
+  send_broken (&route->next, &route->cid, GCP_get_id (odir->hop), &my_full_id);
+  destroy_route (route);
 }
 
 
@@ -661,16 +664,16 @@ dir_ready_cb(void *cls, int ready)
  * @param hop next hop on in the @a dir
  */
 static void
-dir_init(struct RouteDirection *dir,
-         struct CadetRoute *route,
-         struct CadetPeer *hop)
+dir_init (struct RouteDirection *dir,
+          struct CadetRoute *route,
+          struct CadetPeer *hop)
 {
   dir->hop = hop;
   dir->my_route = route;
-  dir->mqm = GCP_request_mq(hop, &dir_ready_cb, dir);
-  GNUNET_CONTAINER_DLL_insert(rung_head->rd_head, rung_head->rd_tail, dir);
+  dir->mqm = GCP_request_mq (hop, &dir_ready_cb, dir);
+  GNUNET_CONTAINER_DLL_insert (rung_head->rd_head, rung_head->rd_tail, dir);
   dir->rung = rung_head;
-  GNUNET_assert(GNUNET_YES == dir->is_ready);
+  GNUNET_assert (GNUNET_YES == dir->is_ready);
 }
 
 
@@ -685,7 +688,7 @@ dir_init(struct RouteDirection *dir,
  *        or NULL.
  */
 static void
-send_broken_without_mqm(
+send_broken_without_mqm (
   struct CadetPeer *target,
   const struct GNUNET_CADET_ConnectionTunnelIdentifier *cid,
   const struct GNUNET_PeerIdentity *failure_at)
@@ -693,12 +696,12 @@ send_broken_without_mqm(
   struct GNUNET_MQ_Envelope *env;
   struct GNUNET_CADET_ConnectionBrokenMessage *bm;
 
-  env = GNUNET_MQ_msg(bm, GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN);
+  env = GNUNET_MQ_msg (bm, GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN);
   bm->cid = *cid;
   bm->peer1 = my_full_id;
   if (NULL != failure_at)
     bm->peer2 = *failure_at;
-  GCP_send_ooo(target, env);
+  GCP_send_ooo (target, env);
 }
 
 
@@ -709,193 +712,194 @@ send_broken_without_mqm(
  * @param msg Message itself.
  */
 static void
-handle_connection_create(
+handle_connection_create (
   void *cls,
   const struct GNUNET_CADET_ConnectionCreateMessage *msg)
 {
   struct CadetPeer *sender = cls;
   struct CadetPeer *next;
   const struct GNUNET_PeerIdentity *pids =
-    (const struct GNUNET_PeerIdentity *)&msg[1];
+    (const struct GNUNET_PeerIdentity *) &msg[1];
   struct CadetRoute *route;
-  uint16_t size = ntohs(msg->header.size) - sizeof(*msg);
+  uint16_t size = ntohs (msg->header.size) - sizeof(*msg);
   unsigned int path_length;
   unsigned int off;
 
   path_length = size / sizeof(struct GNUNET_PeerIdentity);
   if (0 == path_length)
-    {
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Dropping CADET_CONNECTION_CREATE with empty path\n");
-      GNUNET_break_op(0);
-      return;
-    }
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Handling CADET_CONNECTION_CREATE from %s for CID %s with %u hops\n",
-      GCP_2s(sender),
-      GNUNET_sh2s(&msg->cid.connection_of_tunnel),
-      path_length);
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Dropping CADET_CONNECTION_CREATE with empty path\n");
+    GNUNET_break_op (0);
+    return;
+  }
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Handling CADET_CONNECTION_CREATE from %s for CID %s with %u hops\n",
+       GCP_2s (sender),
+       GNUNET_sh2s (&msg->cid.connection_of_tunnel),
+       path_length);
   /* Check for loops */
   {
     struct GNUNET_CONTAINER_MultiPeerMap *map;
 
-    map = GNUNET_CONTAINER_multipeermap_create(path_length * 2, GNUNET_YES);
-    GNUNET_assert(NULL != map);
+    map = GNUNET_CONTAINER_multipeermap_create (path_length * 2, GNUNET_YES);
+    GNUNET_assert (NULL != map);
     for (unsigned int i = 0; i < path_length; i++)
+    {
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "CADET_CONNECTION_CREATE has peer %s at offset %u\n",
+           GNUNET_i2s (&pids[i]),
+           i);
+      if (GNUNET_SYSERR == GNUNET_CONTAINER_multipeermap_put (
+            map,
+            &pids[i],
+            NULL,
+            GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY))
       {
-        LOG(GNUNET_ERROR_TYPE_DEBUG,
-            "CADET_CONNECTION_CREATE has peer %s at offset %u\n",
-            GNUNET_i2s(&pids[i]),
-            i);
-        if (GNUNET_SYSERR == GNUNET_CONTAINER_multipeermap_put(
-              map,
-              &pids[i],
-              NULL,
-              GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY))
-          {
-            /* bogus request */
-            GNUNET_CONTAINER_multipeermap_destroy(map);
-            LOG(GNUNET_ERROR_TYPE_DEBUG,
-                "Dropping CADET_CONNECTION_CREATE with cyclic path\n");
-            GNUNET_break_op(0);
-            return;
-          }
+        /* bogus request */
+        GNUNET_CONTAINER_multipeermap_destroy (map);
+        LOG (GNUNET_ERROR_TYPE_DEBUG,
+             "Dropping CADET_CONNECTION_CREATE with cyclic path\n");
+        GNUNET_break_op (0);
+        return;
       }
-    GNUNET_CONTAINER_multipeermap_destroy(map);
+    }
+    GNUNET_CONTAINER_multipeermap_destroy (map);
   }
   /* Initiator is at offset 0, find us */
   for (off = 1; off < path_length; off++)
-    if (0 == GNUNET_memcmp(&my_full_id, &pids[off]))
+    if (0 == GNUNET_memcmp (&my_full_id, &pids[off]))
       break;
   if (off == path_length)
-    {
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Dropping CADET_CONNECTION_CREATE without us in the path\n");
-      GNUNET_break_op(0);
-      return;
-    }
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Dropping CADET_CONNECTION_CREATE without us in the path\n");
+    GNUNET_break_op (0);
+    return;
+  }
   /* Check previous hop */
-  if (sender != GCP_get(&pids[off - 1], GNUNET_NO))
-    {
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Dropping CADET_CONNECTION_CREATE without sender at previous hop in the path\n");
-      GNUNET_break_op(0);
-      return;
-    }
-  if (NULL != (route = get_route(&msg->cid)))
-    {
-      /* Duplicate CREATE, pass it on, previous one might have been lost! */
-
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Passing on duplicate CADET_CONNECTION_CREATE message on connection %s\n",
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-      route_message(sender,
-                    &msg->cid,
-                    &msg->header,
-                    GNUNET_MQ_PRIO_CRITICAL_CONTROL |
-                    GNUNET_MQ_PREF_LOW_LATENCY);
-      return;
-    }
+  if (sender != GCP_get (&pids[off - 1], GNUNET_NO))
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Dropping CADET_CONNECTION_CREATE without sender at previous hop in the path\n");
+    GNUNET_break_op (0);
+    return;
+  }
+  if (NULL != (route = get_route (&msg->cid)))
+  {
+    /* Duplicate CREATE, pass it on, previous one might have been lost! */
+
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Passing on duplicate CADET_CONNECTION_CREATE message on connection %s\n",
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+    route_message (sender,
+                   &msg->cid,
+                   &msg->header,
+                   GNUNET_MQ_PRIO_CRITICAL_CONTROL
+                   | GNUNET_MQ_PREF_LOW_LATENCY);
+    return;
+  }
   if (off == path_length - 1)
+  {
+    /* We are the destination, create connection */
+    struct CadetConnection *cc;
+    struct CadetPeerPath *path;
+    struct CadetPeer *origin;
+
+    cc = GCC_lookup (&msg->cid);
+    if (NULL != cc)
     {
-      /* We are the destination, create connection */
-      struct CadetConnection *cc;
-      struct CadetPeerPath *path;
-      struct CadetPeer *origin;
-
-      cc = GCC_lookup(&msg->cid);
-      if (NULL != cc)
-        {
-          LOG(GNUNET_ERROR_TYPE_DEBUG,
-              "Received duplicate CADET_CONNECTION_CREATE message on connection %s\n",
-              GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-          GCC_handle_duplicate_create(cc);
-          return;
-        }
-
-      origin = GCP_get(&pids[0], GNUNET_YES);
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "I am destination for CADET_CONNECTION_CREATE message from %s for connection %s, building inverse path\n",
-          GCP_2s(origin),
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-      path = GCPP_get_path_from_route(path_length - 1, pids);
-      if (GNUNET_OK !=
-          GCT_add_inbound_connection(GCP_get_tunnel(origin, GNUNET_YES),
-                                     &msg->cid,
-                                     path))
-        {
-          /* Send back BROKEN: duplicate connection on the same path,
-             we will use the other one. */
-          LOG(GNUNET_ERROR_TYPE_DEBUG,
-              "Received CADET_CONNECTION_CREATE from %s for %s, but %s already has a connection. Sending BROKEN\n",
-              GCP_2s(sender),
-              GNUNET_sh2s(&msg->cid.connection_of_tunnel),
-              GCPP_2s(path));
-          send_broken_without_mqm(sender, &msg->cid, NULL);
-          return;
-        }
-      return;
-    }
-  /* We are merely a hop on the way, check if we can support the route */
-  next = GCP_get(&pids[off + 1], GNUNET_NO);
-  if ((NULL == next) || (GNUNET_NO == GCP_has_core_connection(next)))
-    {
-      /* unworkable, send back BROKEN notification */
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Received CADET_CONNECTION_CREATE from %s for %s. Next hop %s:%u is down. Sending BROKEN\n",
-          GCP_2s(sender),
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel),
-          GNUNET_i2s(&pids[off + 1]),
-          off + 1);
-      send_broken_without_mqm(sender, &msg->cid, &pids[off + 1]);
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "Received duplicate CADET_CONNECTION_CREATE message on connection %s\n",
+           GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+      GCC_handle_duplicate_create (cc);
       return;
     }
-  if (max_routes <= GNUNET_CONTAINER_multishortmap_size(routes))
+
+    origin = GCP_get (&pids[0], GNUNET_YES);
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "I am destination for CADET_CONNECTION_CREATE message from %s for connection %s, building inverse path\n",
+         GCP_2s (origin),
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+    path = GCPP_get_path_from_route (path_length - 1, pids);
+    if (GNUNET_OK !=
+        GCT_add_inbound_connection (GCP_get_tunnel (origin, GNUNET_YES),
+                                    &msg->cid,
+                                    path))
     {
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Received CADET_CONNECTION_CREATE from %s for %s. We have reached our route limit. Sending BROKEN\n",
-          GCP_2s(sender),
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-      send_broken_without_mqm(sender, &msg->cid, &pids[off - 1]);
+      /* Send back BROKEN: duplicate connection on the same path,
+         we will use the other one. */
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "Received CADET_CONNECTION_CREATE from %s for %s, but %s already has a connection. Sending BROKEN\n",
+           GCP_2s (sender),
+           GNUNET_sh2s (&msg->cid.connection_of_tunnel),
+           GCPP_2s (path));
+      send_broken_without_mqm (sender, &msg->cid, NULL);
       return;
     }
+    return;
+  }
+  /* We are merely a hop on the way, check if we can support the route */
+  next = GCP_get (&pids[off + 1], GNUNET_NO);
+  if ((NULL == next) || (GNUNET_NO == GCP_has_core_connection (next)))
+  {
+    /* unworkable, send back BROKEN notification */
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Received CADET_CONNECTION_CREATE from %s for %s. Next hop %s:%u is down. Sending BROKEN\n",
+         GCP_2s (sender),
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel),
+         GNUNET_i2s (&pids[off + 1]),
+         off + 1);
+    send_broken_without_mqm (sender, &msg->cid, &pids[off + 1]);
+    return;
+  }
+  if (max_routes <= GNUNET_CONTAINER_multishortmap_size (routes))
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Received CADET_CONNECTION_CREATE from %s for %s. We have reached our route limit. Sending BROKEN\n",
+         GCP_2s (sender),
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+    send_broken_without_mqm (sender, &msg->cid, &pids[off - 1]);
+    return;
+  }
 
   /* Workable route, create routing entry */
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Received CADET_CONNECTION_CREATE from %s for %s. Next hop %s:%u is up. Creating route\n",
-      GCP_2s(sender),
-      GNUNET_sh2s(&msg->cid.connection_of_tunnel),
-      GNUNET_i2s(&pids[off + 1]),
-      off + 1);
-  route = GNUNET_new(struct CadetRoute);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received CADET_CONNECTION_CREATE from %s for %s. Next hop %s:%u is up. Creating route\n",
+       GCP_2s (sender),
+       GNUNET_sh2s (&msg->cid.connection_of_tunnel),
+       GNUNET_i2s (&pids[off + 1]),
+       off + 1);
+  route = GNUNET_new (struct CadetRoute);
   route->cid = msg->cid;
-  route->last_use = GNUNET_TIME_absolute_get();
-  dir_init(&route->prev, route, sender);
-  dir_init(&route->next, route, next);
-  GNUNET_assert(GNUNET_OK ==
-                GNUNET_CONTAINER_multishortmap_put(
-                  routes,
-                  &route->cid.connection_of_tunnel,
-                  route,
-                  GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY));
-  GNUNET_STATISTICS_set(stats,
-                        "# routes",
-                        GNUNET_CONTAINER_multishortmap_size(routes),
-                        GNUNET_NO);
-  route->hn = GNUNET_CONTAINER_heap_insert(route_heap,
-                                           route,
-                                           route->last_use.abs_value_us);
+  route->last_use = GNUNET_TIME_absolute_get ();
+  dir_init (&route->prev, route, sender);
+  dir_init (&route->next, route, next);
+  GNUNET_assert (GNUNET_OK ==
+                 GNUNET_CONTAINER_multishortmap_put (
+                   routes,
+                   &route->cid.connection_of_tunnel,
+                   route,
+                   GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY));
+  GNUNET_STATISTICS_set (stats,
+                         "# routes",
+                         GNUNET_CONTAINER_multishortmap_size (routes),
+                         GNUNET_NO);
+  route->hn = GNUNET_CONTAINER_heap_insert (route_heap,
+                                            route,
+                                            route->last_use.abs_value_us);
   if (NULL == timeout_task)
     timeout_task =
-      GNUNET_SCHEDULER_add_delayed(GNUNET_TIME_relative_multiply(keepalive_period,
-                                                                 3),
-                                   &timeout_cb,
-                                   NULL);
+      GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_relative_multiply (
+                                      keepalive_period,
+                                      3),
+                                    &timeout_cb,
+                                    NULL);
   /* also pass CREATE message along to next hop */
-  route_message(sender,
-                &msg->cid,
-                &msg->header,
-                GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
+  route_message (sender,
+                 &msg->cid,
+                 &msg->header,
+                 GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
 }
 
 
@@ -906,7 +910,7 @@ handle_connection_create(
  * @param msg Message itself.
  */
 static void
-handle_connection_create_ack(
+handle_connection_create_ack (
   void *cls,
   const struct GNUNET_CADET_ConnectionCreateAckMessage *msg)
 {
@@ -914,31 +918,31 @@ handle_connection_create_ack(
   struct CadetConnection *cc;
 
   /* First, check if ACK belongs to a connection that ends here. */
-  cc = GCC_lookup(&msg->cid);
+  cc = GCC_lookup (&msg->cid);
   if (NULL != cc)
+  {
+    /* verify ACK came from the right direction */
+    unsigned int len;
+    struct CadetPeerPath *path = GCC_get_path (cc, &len);
+
+    if (peer != GCPP_get_peer_at_offset (path, 0))
     {
-      /* verify ACK came from the right direction */
-      unsigned int len;
-      struct CadetPeerPath *path = GCC_get_path(cc, &len);
-
-      if (peer != GCPP_get_peer_at_offset(path, 0))
-        {
-          /* received ACK from unexpected direction, ignore! */
-          GNUNET_break_op(0);
-          return;
-        }
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Received CONNECTION_CREATE_ACK for connection %s.\n",
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-      GCC_handle_connection_create_ack(cc);
+      /* received ACK from unexpected direction, ignore! */
+      GNUNET_break_op (0);
       return;
     }
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Received CONNECTION_CREATE_ACK for connection %s.\n",
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+    GCC_handle_connection_create_ack (cc);
+    return;
+  }
 
   /* We're just an intermediary peer, route the message along its path */
-  route_message(peer,
-                &msg->cid,
-                &msg->header,
-                GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
+  route_message (peer,
+                 &msg->cid,
+                 &msg->header,
+                 GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
 }
 
 
@@ -950,7 +954,7 @@ handle_connection_create_ack(
  * @deprecated duplicate logic with #handle_destroy(); dedup!
  */
 static void
-handle_connection_broken(
+handle_connection_broken (
   void *cls,
   const struct GNUNET_CADET_ConnectionBrokenMessage *msg)
 {
@@ -959,36 +963,36 @@ handle_connection_broken(
   struct CadetRoute *route;
 
   /* First, check if message belongs to a connection that ends here. */
-  cc = GCC_lookup(&msg->cid);
+  cc = GCC_lookup (&msg->cid);
   if (NULL != cc)
+  {
+    /* verify message came from the right direction */
+    unsigned int len;
+    struct CadetPeerPath *path = GCC_get_path (cc, &len);
+
+    if (peer != GCPP_get_peer_at_offset (path, 0))
     {
-      /* verify message came from the right direction */
-      unsigned int len;
-      struct CadetPeerPath *path = GCC_get_path(cc, &len);
-
-      if (peer != GCPP_get_peer_at_offset(path, 0))
-        {
-          /* received message from unexpected direction, ignore! */
-          GNUNET_break_op(0);
-          return;
-        }
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Received CONNECTION_BROKEN for connection %s. Destroying it.\n",
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-      GCC_destroy_without_core(cc);
-
-      /* FIXME: also destroy the path up to the specified link! */
+      /* received message from unexpected direction, ignore! */
+      GNUNET_break_op (0);
       return;
     }
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Received CONNECTION_BROKEN for connection %s. Destroying it.\n",
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+    GCC_destroy_without_core (cc);
+
+    /* FIXME: also destroy the path up to the specified link! */
+    return;
+  }
 
   /* We're just an intermediary peer, route the message along its path */
-  route_message(peer,
-                &msg->cid,
-                &msg->header,
-                GNUNET_MQ_PREF_LOW_LATENCY | GNUNET_MQ_PRIO_CRITICAL_CONTROL);
-  route = get_route(&msg->cid);
+  route_message (peer,
+                 &msg->cid,
+                 &msg->header,
+                 GNUNET_MQ_PREF_LOW_LATENCY | GNUNET_MQ_PRIO_CRITICAL_CONTROL);
+  route = get_route (&msg->cid);
   if (NULL != route)
-    destroy_route(route);
+    destroy_route (route);
   /* FIXME: also destroy paths we MAY have up to the specified link! */
 }
 
@@ -1000,7 +1004,7 @@ handle_connection_broken(
  * @param msg Message itself.
  */
 static void
-handle_connection_destroy(
+handle_connection_destroy (
   void *cls,
   const struct GNUNET_CADET_ConnectionDestroyMessage *msg)
 {
@@ -1009,38 +1013,38 @@ handle_connection_destroy(
   struct CadetRoute *route;
 
   /* First, check if message belongs to a connection that ends here. */
-  cc = GCC_lookup(&msg->cid);
+  cc = GCC_lookup (&msg->cid);
   if (NULL != cc)
+  {
+    /* verify message came from the right direction */
+    unsigned int len;
+    struct CadetPeerPath *path = GCC_get_path (cc, &len);
+
+    if (peer != GCPP_get_peer_at_offset (path, 0))
     {
-      /* verify message came from the right direction */
-      unsigned int len;
-      struct CadetPeerPath *path = GCC_get_path(cc, &len);
-
-      if (peer != GCPP_get_peer_at_offset(path, 0))
-        {
-          /* received message from unexpected direction, ignore! */
-          GNUNET_break_op(0);
-          return;
-        }
-      LOG(GNUNET_ERROR_TYPE_DEBUG,
-          "Received CONNECTION_DESTROY for connection %s. Destroying connection.\n",
-          GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-
-      GCC_destroy_without_core(cc);
+      /* received message from unexpected direction, ignore! */
+      GNUNET_break_op (0);
       return;
     }
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Received CONNECTION_DESTROY for connection %s. Destroying connection.\n",
+         GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+
+    GCC_destroy_without_core (cc);
+    return;
+  }
 
   /* We're just an intermediary peer, route the message along its path */
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Received CONNECTION_DESTROY for connection %s. Destroying route.\n",
-      GNUNET_sh2s(&msg->cid.connection_of_tunnel));
-  route_message(peer,
-                &msg->cid,
-                &msg->header,
-                GNUNET_MQ_PREF_LOW_LATENCY | GNUNET_MQ_PRIO_CRITICAL_CONTROL);
-  route = get_route(&msg->cid);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received CONNECTION_DESTROY for connection %s. Destroying route.\n",
+       GNUNET_sh2s (&msg->cid.connection_of_tunnel));
+  route_message (peer,
+                 &msg->cid,
+                 &msg->header,
+                 GNUNET_MQ_PREF_LOW_LATENCY | GNUNET_MQ_PRIO_CRITICAL_CONTROL);
+  route = get_route (&msg->cid);
   if (NULL != route)
-    destroy_route(route);
+    destroy_route (route);
 }
 
 
@@ -1051,41 +1055,41 @@ handle_connection_destroy(
  * @param msg Message itself.
  */
 static void
-handle_tunnel_kx(void *cls,
-                 const struct GNUNET_CADET_TunnelKeyExchangeMessage *msg)
+handle_tunnel_kx (void *cls,
+                  const struct GNUNET_CADET_TunnelKeyExchangeMessage *msg)
 {
   struct CadetPeer *peer = cls;
   struct CadetConnection *cc;
 
   /* First, check if message belongs to a connection that ends here. */
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "Routing KX with ephemeral %s on CID %s\n",
-      GNUNET_e2s(&msg->ephemeral_key),
-      GNUNET_sh2s(&msg->cid.connection_of_tunnel));
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Routing KX with ephemeral %s on CID %s\n",
+       GNUNET_e2s (&msg->ephemeral_key),
+       GNUNET_sh2s (&msg->cid.connection_of_tunnel));
 
 
-  cc = GCC_lookup(&msg->cid);
+  cc = GCC_lookup (&msg->cid);
   if (NULL != cc)
+  {
+    /* verify message came from the right direction */
+    unsigned int len;
+    struct CadetPeerPath *path = GCC_get_path (cc, &len);
+
+    if (peer != GCPP_get_peer_at_offset (path, 0))
     {
-      /* verify message came from the right direction */
-      unsigned int len;
-      struct CadetPeerPath *path = GCC_get_path(cc, &len);
-
-      if (peer != GCPP_get_peer_at_offset(path, 0))
-        {
-          /* received message from unexpected direction, ignore! */
-          GNUNET_break_op(0);
-          return;
-        }
-      GCC_handle_kx(cc, msg);
+      /* received message from unexpected direction, ignore! */
+      GNUNET_break_op (0);
       return;
     }
+    GCC_handle_kx (cc, msg);
+    return;
+  }
 
   /* We're just an intermediary peer, route the message along its path */
-  route_message(peer,
-                &msg->cid,
-                &msg->header,
-                GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
+  route_message (peer,
+                 &msg->cid,
+                 &msg->header,
+                 GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
 }
 
 
@@ -1096,7 +1100,7 @@ handle_tunnel_kx(void *cls,
  * @param msg Message itself.
  */
 static void
-handle_tunnel_kx_auth(
+handle_tunnel_kx_auth (
   void *cls,
   const struct GNUNET_CADET_TunnelKeyExchangeAuthMessage *msg)
 {
@@ -1104,28 +1108,28 @@ handle_tunnel_kx_auth(
   struct CadetConnection *cc;
 
   /* First, check if message belongs to a connection that ends here. */
-  cc = GCC_lookup(&msg->kx.cid);
+  cc = GCC_lookup (&msg->kx.cid);
   if (NULL != cc)
+  {
+    /* verify message came from the right direction */
+    unsigned int len;
+    struct CadetPeerPath *path = GCC_get_path (cc, &len);
+
+    if (peer != GCPP_get_peer_at_offset (path, 0))
     {
-      /* verify message came from the right direction */
-      unsigned int len;
-      struct CadetPeerPath *path = GCC_get_path(cc, &len);
-
-      if (peer != GCPP_get_peer_at_offset(path, 0))
-        {
-          /* received message from unexpected direction, ignore! */
-          GNUNET_break_op(0);
-          return;
-        }
-      GCC_handle_kx_auth(cc, msg);
+      /* received message from unexpected direction, ignore! */
+      GNUNET_break_op (0);
       return;
     }
+    GCC_handle_kx_auth (cc, msg);
+    return;
+  }
 
   /* We're just an intermediary peer, route the message along its path */
-  route_message(peer,
-                &msg->kx.cid,
-                &msg->kx.header,
-                GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
+  route_message (peer,
+                 &msg->kx.cid,
+                 &msg->kx.header,
+                 GNUNET_MQ_PRIO_CRITICAL_CONTROL | GNUNET_MQ_PREF_LOW_LATENCY);
 }
 
 
@@ -1138,8 +1142,8 @@ handle_tunnel_kx_auth(
  * @return #GNUNET_YES if size is correct, #GNUNET_NO otherwise.
  */
 static int
-check_tunnel_encrypted(void *cls,
-                       const struct GNUNET_CADET_TunnelEncryptedMessage *msg)
+check_tunnel_encrypted (void *cls,
+                        const struct GNUNET_CADET_TunnelEncryptedMessage *msg)
 {
   return GNUNET_YES;
 }
@@ -1152,31 +1156,31 @@ check_tunnel_encrypted(void *cls,
  * @param msg Message itself.
  */
 static void
-handle_tunnel_encrypted(void *cls,
-                        const struct GNUNET_CADET_TunnelEncryptedMessage *msg)
+handle_tunnel_encrypted (void *cls,
+                         const struct GNUNET_CADET_TunnelEncryptedMessage *msg)
 {
   struct CadetPeer *peer = cls;
   struct CadetConnection *cc;
 
   /* First, check if message belongs to a connection that ends here. */
-  cc = GCC_lookup(&msg->cid);
+  cc = GCC_lookup (&msg->cid);
   if (NULL != cc)
+  {
+    /* verify message came from the right direction */
+    unsigned int len;
+    struct CadetPeerPath *path = GCC_get_path (cc, &len);
+
+    if (peer != GCPP_get_peer_at_offset (path, 0))
     {
-      /* verify message came from the right direction */
-      unsigned int len;
-      struct CadetPeerPath *path = GCC_get_path(cc, &len);
-
-      if (peer != GCPP_get_peer_at_offset(path, 0))
-        {
-          /* received message from unexpected direction, ignore! */
-          GNUNET_break_op(0);
-          return;
-        }
-      GCC_handle_encrypted(cc, msg);
+      /* received message from unexpected direction, ignore! */
+      GNUNET_break_op (0);
       return;
     }
+    GCC_handle_encrypted (cc, msg);
+    return;
+  }
   /* We're just an intermediary peer, route the message along its path */
-  route_message(peer, &msg->cid, &msg->header, GNUNET_MQ_PRIO_BEST_EFFORT);
+  route_message (peer, &msg->cid, &msg->header, GNUNET_MQ_PRIO_BEST_EFFORT);
 }
 
 
@@ -1193,14 +1197,14 @@ handle_tunnel_encrypted(void *cls,
  * @param my_identity ID of this peer, NULL if we failed
  */
 static void
-core_init_cb(void *cls, const struct GNUNET_PeerIdentity *my_identity)
+core_init_cb (void *cls, const struct GNUNET_PeerIdentity *my_identity)
 {
   if (NULL == my_identity)
-    {
-      GNUNET_break(0);
-      return;
-    }
-  GNUNET_break(0 == GNUNET_memcmp(my_identity, &my_full_id));
+  {
+    GNUNET_break (0);
+    return;
+  }
+  GNUNET_break (0 == GNUNET_memcmp (my_identity, &my_full_id));
 }
 
 
@@ -1211,17 +1215,17 @@ core_init_cb(void *cls, const struct GNUNET_PeerIdentity *my_identity)
  * @param peer peer identity this notification is about
  */
 static void *
-core_connect_cb(void *cls,
-                const struct GNUNET_PeerIdentity *peer,
-                struct GNUNET_MQ_Handle *mq)
+core_connect_cb (void *cls,
+                 const struct GNUNET_PeerIdentity *peer,
+                 struct GNUNET_MQ_Handle *mq)
 {
   struct CadetPeer *cp;
 
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "CORE connection to peer %s was established.\n",
-      GNUNET_i2s(peer));
-  cp = GCP_get(peer, GNUNET_YES);
-  GCP_set_mq(cp, mq);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "CORE connection to peer %s was established.\n",
+       GNUNET_i2s (peer));
+  cp = GCP_get (peer, GNUNET_YES);
+  GCP_set_mq (cp, mq);
   return cp;
 }
 
@@ -1233,16 +1237,16 @@ core_connect_cb(void *cls,
  * @param peer peer identity this notification is about
  */
 static void
-core_disconnect_cb(void *cls,
-                   const struct GNUNET_PeerIdentity *peer,
-                   void *peer_cls)
+core_disconnect_cb (void *cls,
+                    const struct GNUNET_PeerIdentity *peer,
+                    void *peer_cls)
 {
   struct CadetPeer *cp = peer_cls;
 
-  LOG(GNUNET_ERROR_TYPE_DEBUG,
-      "CORE connection to peer %s went down.\n",
-      GNUNET_i2s(peer));
-  GCP_set_mq(cp, NULL);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "CORE connection to peer %s went down.\n",
+       GNUNET_i2s (peer));
+  GCP_set_mq (cp, NULL);
 }
 
 
@@ -1252,57 +1256,57 @@ core_disconnect_cb(void *cls,
  * @param c Configuration.
  */
 void
-GCO_init(const struct GNUNET_CONFIGURATION_Handle *c)
+GCO_init (const struct GNUNET_CONFIGURATION_Handle *c)
 {
   struct GNUNET_MQ_MessageHandler handlers[] =
-  { GNUNET_MQ_hd_var_size(connection_create,
-                          GNUNET_MESSAGE_TYPE_CADET_CONNECTION_CREATE,
-                          struct GNUNET_CADET_ConnectionCreateMessage,
-                          NULL),
-    GNUNET_MQ_hd_fixed_size(connection_create_ack,
-                            GNUNET_MESSAGE_TYPE_CADET_CONNECTION_CREATE_ACK,
-                            struct GNUNET_CADET_ConnectionCreateAckMessage,
-                            NULL),
-    GNUNET_MQ_hd_fixed_size(connection_broken,
-                            GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN,
-                            struct GNUNET_CADET_ConnectionBrokenMessage,
-                            NULL),
-    GNUNET_MQ_hd_fixed_size(connection_destroy,
-                            GNUNET_MESSAGE_TYPE_CADET_CONNECTION_DESTROY,
-                            struct GNUNET_CADET_ConnectionDestroyMessage,
-                            NULL),
-    GNUNET_MQ_hd_fixed_size(tunnel_kx,
-                            GNUNET_MESSAGE_TYPE_CADET_TUNNEL_KX,
-                            struct GNUNET_CADET_TunnelKeyExchangeMessage,
-                            NULL),
-    GNUNET_MQ_hd_fixed_size(tunnel_kx_auth,
-                            GNUNET_MESSAGE_TYPE_CADET_TUNNEL_KX_AUTH,
-                            struct GNUNET_CADET_TunnelKeyExchangeAuthMessage,
-                            NULL),
-    GNUNET_MQ_hd_var_size(tunnel_encrypted,
-                          GNUNET_MESSAGE_TYPE_CADET_TUNNEL_ENCRYPTED,
-                          struct GNUNET_CADET_TunnelEncryptedMessage,
-                          NULL),
-    GNUNET_MQ_handler_end() };
-
-  if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_number(c,
-                                                         "CADET",
-                                                         "MAX_ROUTES",
-                                                         &max_routes))
+  { GNUNET_MQ_hd_var_size (connection_create,
+                           GNUNET_MESSAGE_TYPE_CADET_CONNECTION_CREATE,
+                           struct GNUNET_CADET_ConnectionCreateMessage,
+                           NULL),
+    GNUNET_MQ_hd_fixed_size (connection_create_ack,
+                             GNUNET_MESSAGE_TYPE_CADET_CONNECTION_CREATE_ACK,
+                             struct GNUNET_CADET_ConnectionCreateAckMessage,
+                             NULL),
+    GNUNET_MQ_hd_fixed_size (connection_broken,
+                             GNUNET_MESSAGE_TYPE_CADET_CONNECTION_BROKEN,
+                             struct GNUNET_CADET_ConnectionBrokenMessage,
+                             NULL),
+    GNUNET_MQ_hd_fixed_size (connection_destroy,
+                             GNUNET_MESSAGE_TYPE_CADET_CONNECTION_DESTROY,
+                             struct GNUNET_CADET_ConnectionDestroyMessage,
+                             NULL),
+    GNUNET_MQ_hd_fixed_size (tunnel_kx,
+                             GNUNET_MESSAGE_TYPE_CADET_TUNNEL_KX,
+                             struct GNUNET_CADET_TunnelKeyExchangeMessage,
+                             NULL),
+    GNUNET_MQ_hd_fixed_size (tunnel_kx_auth,
+                             GNUNET_MESSAGE_TYPE_CADET_TUNNEL_KX_AUTH,
+                             struct GNUNET_CADET_TunnelKeyExchangeAuthMessage,
+                             NULL),
+    GNUNET_MQ_hd_var_size (tunnel_encrypted,
+                           GNUNET_MESSAGE_TYPE_CADET_TUNNEL_ENCRYPTED,
+                           struct GNUNET_CADET_TunnelEncryptedMessage,
+                           NULL),
+    GNUNET_MQ_handler_end () };
+
+  if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_number (c,
+                                                          "CADET",
+                                                          "MAX_ROUTES",
+                                                          &max_routes))
     max_routes = 5000;
-  if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_number(c,
-                                                         "CADET",
-                                                         "MAX_MSGS_QUEUE",
-                                                         &max_buffers))
+  if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_number (c,
+                                                          "CADET",
+                                                          "MAX_MSGS_QUEUE",
+                                                          &max_buffers))
     max_buffers = 10000;
-  routes = GNUNET_CONTAINER_multishortmap_create(1024, GNUNET_NO);
-  route_heap = GNUNET_CONTAINER_heap_create(GNUNET_CONTAINER_HEAP_ORDER_MIN);
-  core = GNUNET_CORE_connect(c,
-                             NULL,
-                             &core_init_cb,
-                             &core_connect_cb,
-                             &core_disconnect_cb,
-                             handlers);
+  routes = GNUNET_CONTAINER_multishortmap_create (1024, GNUNET_NO);
+  route_heap = GNUNET_CONTAINER_heap_create (GNUNET_CONTAINER_HEAP_ORDER_MIN);
+  core = GNUNET_CORE_connect (c,
+                              NULL,
+                              &core_init_cb,
+                              &core_connect_cb,
+                              &core_disconnect_cb,
+                              handlers);
 }
 
 
@@ -1310,23 +1314,23 @@ GCO_init(const struct GNUNET_CONFIGURATION_Handle *c)
  * Shut down the CORE subsystem.
  */
 void
-GCO_shutdown()
+GCO_shutdown ()
 {
   if (NULL != core)
-    {
-      GNUNET_CORE_disconnect(core);
-      core = NULL;
-    }
-  GNUNET_assert(0 == GNUNET_CONTAINER_multishortmap_size(routes));
-  GNUNET_CONTAINER_multishortmap_destroy(routes);
+  {
+    GNUNET_CORE_disconnect (core);
+    core = NULL;
+  }
+  GNUNET_assert (0 == GNUNET_CONTAINER_multishortmap_size (routes));
+  GNUNET_CONTAINER_multishortmap_destroy (routes);
   routes = NULL;
-  GNUNET_CONTAINER_heap_destroy(route_heap);
+  GNUNET_CONTAINER_heap_destroy (route_heap);
   route_heap = NULL;
   if (NULL != timeout_task)
-    {
-      GNUNET_SCHEDULER_cancel(timeout_task);
-      timeout_task = NULL;
-    }
+  {
+    GNUNET_SCHEDULER_cancel (timeout_task);
+    timeout_task = NULL;
+  }
 }
 
 /* end of gnunet-cadet-service_core.c */