-makefile for new test_stream_local (commented)
[oweals/gnunet.git] / src / transport / gnunet-service-transport_neighbours.c
index 6011ad2492336470199f99782f669849013b177e..9b8fafe33230c1ac6e33875c9d8e8228da024c92 100644 (file)
 #define QUOTA_VIOLATION_DROP_THRESHOLD 10
 
 /**
- * How often do we send KEEPALIVE messages to each of our neighbours?
- * (idle timeout is 5 minutes or 300 seconds, so with 90s interval we
- * send 3 keepalives in each interval, so 3 messages would need to be
+ * How often do we send KEEPALIVE messages to each of our neighbours and measure
+ * the latency with this neighbour?
+ * (idle timeout is 5 minutes or 300 seconds, so with 30s interval we
+ * send 10 keepalives in each interval, so 10 messages would need to be
  * lost in a row for a disconnect).
  */
-#define KEEPALIVE_FREQUENCY GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 90)
+#define KEEPALIVE_FREQUENCY GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 30)
 
 
-#define ATS_RESPONSE_TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 3)
+#define ATS_RESPONSE_TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 5)
 
+#define FAST_RECONNECT_RATE GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_MILLISECONDS, 100)
+
+#define FAST_RECONNECT_TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 1)
 
 #define SETUP_CONNECTION_TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 15)
 
+#define  TEST_NEW_CODE GNUNET_NO
 
 /**
  * Entry in neighbours.
  */
 struct NeighbourMapEntry;
 
+GNUNET_NETWORK_STRUCT_BEGIN
+
 /**
  * Message a peer sends to another to indicate its
  * preference for communicating via a particular
@@ -122,7 +129,7 @@ struct SessionDisconnectMessage
    * Public key of the sender.
    */
   struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded public_key;
-  
+
   /**
    * Signature of the peer that sends us the disconnect.  Only
    * valid if the timestamp is AFTER the timestamp from the
@@ -131,7 +138,7 @@ struct SessionDisconnectMessage
   struct GNUNET_CRYPTO_RsaSignature signature;
 
 };
-
+GNUNET_NETWORK_STRUCT_END
 
 /**
  * For each neighbour we keep a list of messages
@@ -184,22 +191,48 @@ struct MessageQueue
 
 };
 
+
 enum State
 {
-    /* fresh peer or completely disconnected */
-    S_NOT_CONNECTED = 0,
-    /* sent CONNECT message to other peer, waiting for CONNECT_ACK */
-    S_CONNECT_SENT = 1,
-    /* received CONNECT message to other peer, sending CONNECT_ACK */
-    S_CONNECT_RECV = 4,
-    /* sent CONNECT_ACK message to other peer, wait for ACK or payload */
-    S_CONNECT_RECV_ACK_SENT = 8,
-    /* received ACK or payload */
-    S_CONNECTED = 16,
-    /* Disconnect in progress */
-    S_DISCONNECT = 32
+  /**
+   * fresh peer or completely disconnected
+   */
+  S_NOT_CONNECTED,
+
+  /**
+   * sent CONNECT message to other peer, waiting for CONNECT_ACK
+   */
+  S_CONNECT_SENT,
+
+  /**
+   * received CONNECT message to other peer, sending CONNECT_ACK
+   */
+  S_CONNECT_RECV,
+
+  /**
+   * received ACK or payload
+   */
+  S_CONNECTED,
+
+  /**
+   * connection ended, fast reconnect
+   */
+  S_FAST_RECONNECT,
+
+  /**
+   * Disconnect in progress
+   */
+  S_DISCONNECT
+};
+
+enum Address_State
+{
+  USED,
+  UNUSED,
+  FRESH,
 };
 
+
 /**
  * Entry in neighbours.
  */
@@ -218,11 +251,6 @@ struct NeighbourMapEntry
    */
   struct MessageQueue *messages_tail;
 
-  /**
-   * Performance data for the peer.
-   */
-  //struct GNUNET_ATS_Information *ats;
-
   /**
    * Are we currently trying to send a message? If so, which one?
    */
@@ -234,19 +262,15 @@ struct NeighbourMapEntry
   struct Session *session;
 
   /**
-   * Name of the plugin we currently use.
+   * Address we currently use.
    */
-  char *plugin_name;
+  struct GNUNET_HELLO_Address *address;
 
   /**
-   * Address used for communicating with the peer, NULL for inbound connections.
+   * Address we currently use.
    */
-  void *addr;
+  struct GNUNET_HELLO_Address *fast_reconnect_address;
 
-  /**
-   * Number of bytes in 'addr'.
-   */
-  size_t addrlen;
 
   /**
    * Identity of this neighbour.
@@ -290,18 +314,34 @@ struct NeighbourMapEntry
    */
   struct GNUNET_TIME_Absolute connect_ts;
 
+  /**
+   * When did we sent the last keep-alive message?
+   */
+  struct GNUNET_TIME_Absolute keep_alive_sent;
+
+  /**
+   * Latest calculated latency value
+   */
+  struct GNUNET_TIME_Relative latency;
+
   /**
    * Timeout for ATS
    * We asked ATS for a new address for this peer
    */
   GNUNET_SCHEDULER_TaskIdentifier ats_suggest;
 
+  /**
+   * Delay for ATS request
+   */
+  GNUNET_SCHEDULER_TaskIdentifier ats_request;
+
   /**
    * Task the resets the peer state after due to an pending
    * unsuccessful connection setup
    */
   GNUNET_SCHEDULER_TaskIdentifier state_reset;
 
+
   /**
    * How often has the other peer (recently) violated the inbound
    * traffic limit?  Incremented by 10 per violation, decremented by 1
@@ -316,6 +356,20 @@ struct NeighbourMapEntry
    */
   int state;
 
+  /**
+   * Did we sent an KEEP_ALIVE message and are we expecting a response?
+   */
+  int expect_latency_response;
+
+  /**
+   * Was the address used successfully
+   */
+  int address_state;
+
+  /**
+   * Fast reconnect attempts for identical address
+   */
+  unsigned int fast_reconnect_attempts;
 };
 
 
@@ -325,7 +379,7 @@ struct NeighbourMapEntry
 static struct GNUNET_CONTAINER_MultiHashMap *neighbours;
 
 /**
- * Closure for connect_notify_cb and disconnect_notify_cb
+ * Closure for connect_notify_cb, disconnect_notify_cb and address_change_cb
  */
 static void *callback_cls;
 
@@ -339,6 +393,11 @@ static GNUNET_TRANSPORT_NotifyConnect connect_notify_cb;
  */
 static GNUNET_TRANSPORT_NotifyDisconnect disconnect_notify_cb;
 
+/**
+ * Function to call when we changed an active address of a neighbour.
+ */
+static GNUNET_TRANSPORT_PeerIterateCallback address_change_cb;
+
 /**
  * counter for connected neighbours
  */
@@ -356,10 +415,18 @@ lookup_neighbour (const struct GNUNET_PeerIdentity *pid)
   return GNUNET_CONTAINER_multihashmap_get (neighbours, &pid->hashPubKey);
 }
 
+/**
+ * Disconnect from the given neighbour, clean up the record.
+ *
+ * @param n neighbour to disconnect from
+ */
+static void
+disconnect_neighbour (struct NeighbourMapEntry *n);
+
 #define change_state(n, state, ...) change (n, state, __LINE__)
 
 static int
-is_connecting (struct NeighbourMapEntry * n)
+is_connecting (struct NeighbourMapEntry *n)
 {
   if ((n->state > S_NOT_CONNECTED) && (n->state < S_CONNECTED))
     return GNUNET_YES;
@@ -367,7 +434,7 @@ is_connecting (struct NeighbourMapEntry * n)
 }
 
 static int
-is_connected (struct NeighbourMapEntry * n)
+is_connected (struct NeighbourMapEntry *n)
 {
   if (n->state == S_CONNECTED)
     return GNUNET_YES;
@@ -375,7 +442,7 @@ is_connected (struct NeighbourMapEntry * n)
 }
 
 static int
-is_disconnecting (struct NeighbourMapEntry * n)
+is_disconnecting (struct NeighbourMapEntry *n)
 {
   if (n->state == S_DISCONNECT)
     return GNUNET_YES;
@@ -385,223 +452,229 @@ is_disconnecting (struct NeighbourMapEntry * n)
 static const char *
 print_state (int state)
 {
-  switch (state) {
-    case S_CONNECTED:
-        return "S_CONNECTED";
-      break;
-    case S_CONNECT_RECV:
-      return "S_CONNECT_RECV";
-      break;
-    case S_CONNECT_RECV_ACK_SENT:
-      return"S_CONNECT_RECV_ACK_SENT";
-      break;
-    case S_CONNECT_SENT:
-      return "S_CONNECT_SENT";
-      break;
-    case S_DISCONNECT:
-      return "S_DISCONNECT";
-      break;
-    case S_NOT_CONNECTED:
-      return "S_NOT_CONNECTED";
-      break;
-    default:
-      GNUNET_break (0);
-      break;
+  switch (state)
+  {
+  case S_CONNECTED:
+    return "S_CONNECTED";
+    break;
+  case S_CONNECT_RECV:
+    return "S_CONNECT_RECV";
+    break;
+  case S_CONNECT_SENT:
+    return "S_CONNECT_SENT";
+    break;
+  case S_DISCONNECT:
+    return "S_DISCONNECT";
+    break;
+  case S_NOT_CONNECTED:
+    return "S_NOT_CONNECTED";
+    break;
+  case S_FAST_RECONNECT:
+    return "S_FAST_RECONNECT";
+    break;
+  default:
+    GNUNET_break (0);
+    break;
   }
   return NULL;
 }
 
 static int
-change (struct NeighbourMapEntry * n, int state, int line);
+change (struct NeighbourMapEntry *n, int state, int line);
 
 static void
-ats_suggest_cancel (void *cls,
-    const struct GNUNET_SCHEDULER_TaskContext *tc);
+ats_suggest_cancel (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc);
+
 
 static void
-reset_task (void *cls,
-            const struct GNUNET_SCHEDULER_TaskContext *tc)
+reset_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
-  struct NeighbourMapEntry * n = cls;
+  struct NeighbourMapEntry *n = cls;
+
+  if (n == NULL)
+    return;
 
   n->state_reset = GNUNET_SCHEDULER_NO_TASK;
+  if (n->state == S_CONNECTED)
+    return;
 
 #if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-      "Connection to peer `%s' %s failed in state `%s', resetting connection attempt \n",
-      GNUNET_i2s (&n->id), GST_plugins_a2s(n->plugin_name, n->addr, n->addrlen), print_state(n->state));
-#endif
   GNUNET_STATISTICS_update (GST_stats,
-                            gettext_noop ("# failed connection attempts due to timeout"),
-                            1,
+                            gettext_noop
+                            ("# failed connection attempts due to timeout"), 1,
                             GNUNET_NO);
+#endif
 
   /* resetting state */
+
+  if (n->state == S_FAST_RECONNECT)
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Fast reconnect time out, disconnecting peer `%s'\n",
+                GNUNET_i2s (&n->id));
+    disconnect_neighbour(n);
+    return;
+  }
+
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "State for neighbour `%s' %X changed from `%s' to `%s' in line %u\n",
+              GNUNET_i2s (&n->id), n, print_state(n->state), "S_NOT_CONNECTED", __LINE__);
+
   n->state = S_NOT_CONNECTED;
 
   /* destroying address */
-  GNUNET_ATS_address_destroyed (GST_ats,
-                                &n->id,
-                                n->plugin_name,
-                                n->addr,
-                                n->addrlen,
-                                NULL);
+  if (n->address != NULL)
+  {
+    GNUNET_assert (strlen (n->address->transport_name) > 0);
+    GNUNET_ATS_address_destroyed (GST_ats, n->address, n->session);
+  }
 
   /* request new address */
   if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
-    GNUNET_SCHEDULER_cancel(n->ats_suggest);
-  n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel, n);
-  GNUNET_ATS_suggest_address(GST_ats, &n->id);
+    GNUNET_SCHEDULER_cancel (n->ats_suggest);
+  n->ats_suggest =
+      GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel,
+                                    n);
+  GNUNET_ATS_suggest_address (GST_ats, &n->id);
 }
 
 static int
-change (struct NeighbourMapEntry * n, int state, int line)
+change (struct NeighbourMapEntry *n, int state, int line)
 {
-  char * old = strdup(print_state(n->state));
-  char * new = strdup(print_state(state));
-
+  int previous_state;
   /* allowed transitions */
   int allowed = GNUNET_NO;
-  switch (n->state) {
+
+  previous_state = n->state;
+
+  switch (n->state)
+  {
   case S_NOT_CONNECTED:
     if ((state == S_CONNECT_RECV) || (state == S_CONNECT_SENT) ||
         (state == S_DISCONNECT))
-    {
       allowed = GNUNET_YES;
-
-      /* Schedule reset task */
-      if ((state == S_CONNECT_RECV) || (state == S_CONNECT_SENT) )
-      {
-        GNUNET_assert (n->state_reset == GNUNET_SCHEDULER_NO_TASK);
-        n->state_reset = GNUNET_SCHEDULER_add_delayed (SETUP_CONNECTION_TIMEOUT, &reset_task, n);
-      }
-
-      break;
-    }
     break;
   case S_CONNECT_RECV:
-    if ((state == S_NOT_CONNECTED) || (state == S_DISCONNECT) ||
-        (state == S_CONNECTED) || /* FIXME SENT -> RECV ISSUE!*/ (state == S_CONNECT_SENT))
-    {
-      if ((state == S_CONNECTED) || (state == S_DISCONNECT) || (state == S_NOT_CONNECTED))
-      {
-#if DEBUG_TRANSPORT
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-            "Removed reset task for peer `%s' %s failed in state transition `%s' -> `%s' \n",
-            GNUNET_i2s (&n->id), GST_plugins_a2s(n->plugin_name, n->addr, n->addrlen), print_state(n->state), print_state(state));
-#endif
-        GNUNET_assert (n->state_reset != GNUNET_SCHEDULER_NO_TASK);
-        GNUNET_SCHEDULER_cancel (n->state_reset);
-        n->state_reset = GNUNET_SCHEDULER_NO_TASK;
-      }
-
-      allowed = GNUNET_YES;
-      break;
-    }
+    allowed = GNUNET_YES;
     break;
   case S_CONNECT_SENT:
-    if ((state == S_NOT_CONNECTED) || (state == S_CONNECTED) ||
-        (state == S_DISCONNECT) || /* FIXME SENT -> RECV ISSUE!*/ (state == S_CONNECT_RECV))
-    {
-      if ((state == S_CONNECTED) || (state == S_DISCONNECT) || (state == S_NOT_CONNECTED))
-      {
-#if DEBUG_TRANSPORT
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-            "Removed reset task for peer `%s' %s failed in state transition `%s' -> `%s' \n",
-            GNUNET_i2s (&n->id), GST_plugins_a2s(n->plugin_name, n->addr, n->addrlen), print_state(n->state), print_state(state));
-#endif
-        GNUNET_assert (n->state_reset != GNUNET_SCHEDULER_NO_TASK);
-        GNUNET_SCHEDULER_cancel (n->state_reset);
-        n->state_reset = GNUNET_SCHEDULER_NO_TASK;
-      }
-
-      allowed = GNUNET_YES;
-      break;
-    }
+    allowed = GNUNET_YES;
     break;
   case S_CONNECTED:
-    if (state == S_DISCONNECT)
-    {
+    if ((state == S_DISCONNECT) || (state == S_FAST_RECONNECT))
       allowed = GNUNET_YES;
-      break;
-    }
     break;
   case S_DISCONNECT:
-    /*
-    if (state == S_NOT_CONNECTED)
-    {
+    break;
+  case S_FAST_RECONNECT:
+    if ((state == S_CONNECTED) || (state == S_DISCONNECT))
       allowed = GNUNET_YES;
-      break;
-    }*/
     break;
   default:
     GNUNET_break (0);
     break;
-
   }
-
   if (allowed == GNUNET_NO)
   {
+    char *old = GNUNET_strdup (print_state (n->state));
+    char *new = GNUNET_strdup (print_state (state));
+
     GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-        "Illegal state transition from `%s' to `%s' in line %u \n",
-        old, new, line);
+                "Illegal state transition from `%s' to `%s' in line %u \n", old,
+                new, line);
     GNUNET_break (0);
     GNUNET_free (old);
     GNUNET_free (new);
     return GNUNET_SYSERR;
   }
+  {
+    char *old = GNUNET_strdup (print_state (n->state));
+    char *new = GNUNET_strdup (print_state (state));
 
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "State for neighbour `%s' %X changed from `%s' to `%s' in line %u\n",
+                GNUNET_i2s (&n->id), n, old, new, line);
+    GNUNET_free (old);
+    GNUNET_free (new);
+  }
   n->state = state;
+
+  switch (n->state)
+  {
+  case S_FAST_RECONNECT:
+  case S_CONNECT_RECV:
+  case S_CONNECT_SENT:
+    if (n->state_reset != GNUNET_SCHEDULER_NO_TASK)
+      GNUNET_SCHEDULER_cancel (n->state_reset);
+    n->state_reset =
+        GNUNET_SCHEDULER_add_delayed (SETUP_CONNECTION_TIMEOUT, &reset_task, n);
+    break;
+  case S_CONNECTED:
+  case S_NOT_CONNECTED:
+  case S_DISCONNECT:
+    if (GNUNET_SCHEDULER_NO_TASK != n->state_reset)
+    {
 #if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "State for neighbour `%s' %X changed from `%s' to `%s' in line %u\n",
-      GNUNET_i2s (&n->id), n, old, new, line);
+      char *old = GNUNET_strdup (print_state (n->state));
+      char *new = GNUNET_strdup (print_state (state));
+
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                  "Removed reset task for peer `%s' %s failed in state transition `%s' -> `%s' \n",
+                  GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), old, new);
+      GNUNET_free (old);
+      GNUNET_free (new);
 #endif
-  GNUNET_free (old);
-  GNUNET_free (new);
+      GNUNET_assert (n->state_reset != GNUNET_SCHEDULER_NO_TASK);
+      GNUNET_SCHEDULER_cancel (n->state_reset);
+      n->state_reset = GNUNET_SCHEDULER_NO_TASK;
+    }
+    break;
+
+  default:
+    GNUNET_assert (0);
+  }
+
+  if (NULL != address_change_cb)
+  {
+    if (n->state == S_CONNECTED)
+      address_change_cb (callback_cls, &n->id, n->address);
+    else if (previous_state == S_CONNECTED)
+      address_change_cb (callback_cls, &n->id, NULL);
+  }
+
   return GNUNET_OK;
 }
 
 static ssize_t
-send_with_plugin ( const struct GNUNET_PeerIdentity * target,
-    const char *msgbuf,
-    size_t msgbuf_size,
-    uint32_t priority,
-    struct GNUNET_TIME_Relative timeout,
-    struct Session * session,
-    const char * plugin_name,
-    const void *addr,
-    size_t addrlen,
-    int force_address,
-    GNUNET_TRANSPORT_TransmitContinuation cont,
-    void *cont_cls)
-
+send_with_session (struct NeighbourMapEntry *n,
+                   const char *msgbuf, size_t msgbuf_size,
+                   uint32_t priority,
+                   struct GNUNET_TIME_Relative timeout,
+                   GNUNET_TRANSPORT_TransmitContinuation cont, void *cont_cls)
 {
   struct GNUNET_TRANSPORT_PluginFunctions *papi;
   size_t ret = GNUNET_SYSERR;
 
-  papi = GST_plugins_find (plugin_name);
+  GNUNET_assert (n != NULL);
+  GNUNET_assert (n->session != NULL);
+
+  papi = GST_plugins_find (n->address->transport_name);
   if (papi == NULL)
   {
     if (cont != NULL)
-      cont (cont_cls, target, GNUNET_SYSERR);
+      cont (cont_cls, &n->id, GNUNET_SYSERR);
     return GNUNET_SYSERR;
   }
 
   ret = papi->send (papi->cls,
-      target,
-      msgbuf, msgbuf_size,
-      0,
-      timeout,
-      session,
-      addr, addrlen,
-      GNUNET_YES,
-      cont, cont_cls);
-
-  if (ret == -1)
-  {
-    if (cont != NULL)
-      cont (cont_cls, target, GNUNET_SYSERR);
-  }
+                   n->session,
+                   msgbuf, msgbuf_size,
+                   0,
+                   timeout,
+                   cont, cont_cls);
+
+  if ((ret == -1) && (cont != NULL))
+      cont (cont_cls, &n->id, GNUNET_SYSERR);
   return ret;
 }
 
@@ -627,12 +700,13 @@ transmit_send_continuation (void *cls,
                             const struct GNUNET_PeerIdentity *receiver,
                             int success)
 {
-  struct MessageQueue *mq;
+  struct MessageQueue *mq = cls;
   struct NeighbourMapEntry *n;
+  struct NeighbourMapEntry *tmp;
 
-  mq = cls;
+  tmp = lookup_neighbour (receiver);
   n = mq->n;
-  if (NULL != n)
+  if ((NULL != n) && (tmp != NULL) && (tmp == n))
   {
     GNUNET_assert (n->is_active == mq);
     n->is_active = NULL;
@@ -689,7 +763,20 @@ try_transmission_to_peer (struct NeighbourMapEntry *n)
   if (NULL == mq)
     return;                     /* no more messages */
 
-  if (GST_plugins_find (n->plugin_name) == NULL)
+  if (n->address == NULL)
+  {
+#if DEBUG_TRANSPORT
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "No address for peer `%s'\n",
+                GNUNET_i2s (&n->id));
+#endif
+    GNUNET_CONTAINER_DLL_remove (n->messages_head, n->messages_tail, mq);
+    transmit_send_continuation (mq, &n->id, GNUNET_SYSERR);
+    GNUNET_assert (n->transmission_task == GNUNET_SCHEDULER_NO_TASK);
+    n->transmission_task = GNUNET_SCHEDULER_add_now (&transmission_task, n);
+    return;
+  }
+
+  if (GST_plugins_find (n->address->transport_name) == NULL)
   {
     GNUNET_break (0);
     return;
@@ -698,7 +785,7 @@ try_transmission_to_peer (struct NeighbourMapEntry *n)
   n->is_active = mq;
   mq->n = n;
 
-  if  ((n->session == NULL) && (n->addr == NULL) && (n->addrlen == 0))
+  if ((n->address->address_length == 0) && (n->session == NULL))
   {
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "No address for peer `%s'\n",
                 GNUNET_i2s (&n->id));
@@ -708,12 +795,11 @@ try_transmission_to_peer (struct NeighbourMapEntry *n)
     return;
   }
 
-  ret = send_with_plugin (&n->id,
-                          mq->message_buf, mq->message_buf_size, 0,
-                          timeout,
-                          n->session, n->plugin_name, n->addr, n->addrlen,
-                          GNUNET_YES,
-                          &transmit_send_continuation, mq);
+  ret = send_with_session(n,
+              mq->message_buf, mq->message_buf_size,
+              0, timeout,
+              &transmit_send_continuation, mq);
+
   if (ret == -1)
   {
     /* failure, but 'send' would not call continuation in this case,
@@ -734,7 +820,8 @@ static void
 transmission_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct NeighbourMapEntry *n = cls;
-  GNUNET_assert (NULL != lookup_neighbour(&n->id));
+
+  GNUNET_assert (NULL != lookup_neighbour (&n->id));
   n->transmission_task = GNUNET_SCHEDULER_NO_TASK;
   try_transmission_to_peer (n);
 }
@@ -746,73 +833,81 @@ transmission_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
  * @param cls closure for callbacks
  * @param connect_cb function to call if we connect to a peer
  * @param disconnect_cb function to call if we disconnect from a peer
+ * @param peer_address_cb function to call if we change an active address
+ *                   of a neighbour
  */
 void
-GST_neighbours_start (void *cls, GNUNET_TRANSPORT_NotifyConnect connect_cb,
-                      GNUNET_TRANSPORT_NotifyDisconnect disconnect_cb)
+GST_neighbours_start (void *cls,
+                      GNUNET_TRANSPORT_NotifyConnect connect_cb,
+                      GNUNET_TRANSPORT_NotifyDisconnect disconnect_cb,
+                      GNUNET_TRANSPORT_PeerIterateCallback peer_address_cb)
 {
   callback_cls = cls;
   connect_notify_cb = connect_cb;
   disconnect_notify_cb = disconnect_cb;
+  address_change_cb = peer_address_cb;
   neighbours = GNUNET_CONTAINER_multihashmap_create (NEIGHBOUR_TABLE_SIZE);
 }
 
 
 static void
-send_disconnect_cont (void *cls,
-    const struct GNUNET_PeerIdentity * target,
-    int result)
+send_disconnect_cont (void *cls, const struct GNUNET_PeerIdentity *target,
+                      int result)
 {
 #if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending DISCONNECT message to peer `%4s': %i\n",
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Sending DISCONNECT message to peer `%4s': %i\n",
               GNUNET_i2s (target), result);
 #endif
 }
 
 
 static int
-send_disconnect (const struct GNUNET_PeerIdentity * target,
-                const char *plugin_name,
-                const char *sender_address, uint16_t sender_address_len,
-                struct Session *session)
+send_disconnect (struct NeighbourMapEntry * n)
 {
   size_t ret;
   struct SessionDisconnectMessage disconnect_msg;
 
 #if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sending DISCONNECT message to peer `%4s'\n",
-              GNUNET_i2s (target));
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Sending DISCONNECT message to peer `%4s'\n",
+              GNUNET_i2s (&n->id));
 #endif
 
   disconnect_msg.header.size = htons (sizeof (struct SessionDisconnectMessage));
-  disconnect_msg.header.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_DISCONNECT);
+  disconnect_msg.header.type =
+      htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_DISCONNECT);
   disconnect_msg.reserved = htonl (0);
-  disconnect_msg.purpose.size = htonl (sizeof (struct GNUNET_CRYPTO_RsaSignaturePurpose) +
-                                       sizeof (struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded) +
-                                       sizeof (struct GNUNET_TIME_AbsoluteNBO) );
-  disconnect_msg.purpose.purpose = htonl (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_DISCONNECT);
-  disconnect_msg.timestamp = GNUNET_TIME_absolute_hton (GNUNET_TIME_absolute_get ());
+  disconnect_msg.purpose.size =
+      htonl (sizeof (struct GNUNET_CRYPTO_RsaSignaturePurpose) +
+             sizeof (struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded) +
+             sizeof (struct GNUNET_TIME_AbsoluteNBO));
+  disconnect_msg.purpose.purpose =
+      htonl (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_DISCONNECT);
+  disconnect_msg.timestamp =
+      GNUNET_TIME_absolute_hton (GNUNET_TIME_absolute_get ());
   disconnect_msg.public_key = GST_my_public_key;
   GNUNET_assert (GNUNET_OK ==
                  GNUNET_CRYPTO_rsa_sign (GST_my_private_key,
                                          &disconnect_msg.purpose,
                                          &disconnect_msg.signature));
 
-  ret = send_with_plugin(target,
-                        (const char *) &disconnect_msg, sizeof (disconnect_msg),
-                        UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
-                        session, plugin_name, sender_address,  sender_address_len,
-                        GNUNET_YES, &send_disconnect_cont, NULL);
+  ret = send_with_session (n,
+            (const char *) &disconnect_msg, sizeof (disconnect_msg),
+            UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
+            &send_disconnect_cont, NULL);
 
   if (ret == GNUNET_SYSERR)
     return GNUNET_SYSERR;
 
   GNUNET_STATISTICS_update (GST_stats,
-                            gettext_noop ("# peers disconnected due to external request"), 1,
+                            gettext_noop
+                            ("# peers disconnected due to external request"), 1,
                             GNUNET_NO);
   return GNUNET_OK;
 }
 
+
 /**
  * Disconnect from the given neighbour, clean up the record.
  *
@@ -822,24 +917,46 @@ static void
 disconnect_neighbour (struct NeighbourMapEntry *n)
 {
   struct MessageQueue *mq;
-  int was_connected = is_connected(n);
+  int previous_state;
+
+  previous_state = n->state;
+
+  if (is_disconnecting (n))
+    return;
 
   /* send DISCONNECT MESSAGE */
-  if (is_connected(n) || is_connecting(n))
+  if (previous_state == S_CONNECTED)
   {
-    if (GNUNET_OK == send_disconnect(&n->id, n->plugin_name, n->addr, n->addrlen, n->session))
+    if (GNUNET_OK == send_disconnect (n))
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Sent DISCONNECT_MSG to `%s'\n",
                   GNUNET_i2s (&n->id));
     else
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Could not send DISCONNECT_MSG to `%s'\n",
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                  "Could not send DISCONNECT_MSG to `%s'\n",
                   GNUNET_i2s (&n->id));
   }
 
-
-  if (is_disconnecting(n))
-    return;
   change_state (n, S_DISCONNECT);
 
+  if (previous_state == S_CONNECTED)
+  {
+    GNUNET_assert (NULL != n->address);
+    if (n->address_state == USED)
+    {
+      GST_validation_set_address_use (n->address, n->session, GNUNET_NO, __LINE__);
+      GNUNET_ATS_address_in_use (GST_ats, n->address, n->session, GNUNET_NO);
+      n->address_state = UNUSED;
+    }
+  }
+
+  if (n->address != NULL)
+  {
+    struct GNUNET_TRANSPORT_PluginFunctions *papi;
+
+    papi = GST_plugins_find (n->address->transport_name);
+    if (papi != NULL)
+      papi->disconnect (papi->cls, &n->id);
+  }
   while (NULL != (mq = n->messages_head))
   {
     GNUNET_CONTAINER_DLL_remove (n->messages_head, n->messages_tail, mq);
@@ -852,17 +969,32 @@ disconnect_neighbour (struct NeighbourMapEntry *n)
     n->is_active->n = NULL;
     n->is_active = NULL;
   }
-  if (was_connected)
+
+  switch (previous_state)
   {
-    GNUNET_assert (GNUNET_SCHEDULER_NO_TASK != n->keepalive_task);
-    GNUNET_SCHEDULER_cancel (n->keepalive_task);
-    n->keepalive_task = GNUNET_SCHEDULER_NO_TASK;  
+  case S_CONNECTED:
     GNUNET_assert (neighbours_connected > 0);
     neighbours_connected--;
+    GNUNET_assert (GNUNET_SCHEDULER_NO_TASK != n->keepalive_task);
+    GNUNET_SCHEDULER_cancel (n->keepalive_task);
+    n->keepalive_task = GNUNET_SCHEDULER_NO_TASK;
+    n->expect_latency_response = GNUNET_NO;
     GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# peers connected"), -1,
                               GNUNET_NO);
     disconnect_notify_cb (callback_cls, &n->id);
+    break;
+  case S_FAST_RECONNECT:
+    GNUNET_STATISTICS_update (GST_stats,
+                              gettext_noop ("# fast reconnects failed"), 1,
+                              GNUNET_NO);
+    disconnect_notify_cb (callback_cls, &n->id);
+    break;
+  default:
+    break;
   }
+
+  GNUNET_ATS_suggest_address_cancel (GST_ats, &n->id);
+
   GNUNET_assert (GNUNET_YES ==
                  GNUNET_CONTAINER_multihashmap_remove (neighbours,
                                                        &n->id.hashPubKey, n));
@@ -871,6 +1003,12 @@ disconnect_neighbour (struct NeighbourMapEntry *n)
     GNUNET_SCHEDULER_cancel (n->ats_suggest);
     n->ats_suggest = GNUNET_SCHEDULER_NO_TASK;
   }
+  if (n->ats_request != GNUNET_SCHEDULER_NO_TASK)
+  {
+    GNUNET_SCHEDULER_cancel (n->ats_request);
+    n->ats_request = GNUNET_SCHEDULER_NO_TASK;
+  }
+
   if (GNUNET_SCHEDULER_NO_TASK != n->timeout_task)
   {
     GNUNET_SCHEDULER_cancel (n->timeout_task);
@@ -881,16 +1019,15 @@ disconnect_neighbour (struct NeighbourMapEntry *n)
     GNUNET_SCHEDULER_cancel (n->transmission_task);
     n->transmission_task = GNUNET_SCHEDULER_NO_TASK;
   }
-  if (NULL != n->plugin_name)
+  if (NULL != n->fast_reconnect_address)
   {
-    GNUNET_free (n->plugin_name);
-    n->plugin_name = NULL;
+    GNUNET_HELLO_address_free (n->fast_reconnect_address);
+    n->fast_reconnect_address = NULL;
   }
-  if (NULL != n->addr)
+  if (NULL != n->address)
   {
-    GNUNET_free (n->addr);
-    n->addr = NULL;
-    n->addrlen = 0;
+    GNUNET_HELLO_address_free (n->address);
+    n->address = NULL;
   }
   n->session = NULL;
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Deleting peer `%4s', %X\n",
@@ -912,9 +1049,12 @@ neighbour_timeout_task (void *cls,
   struct NeighbourMapEntry *n = cls;
 
   n->timeout_task = GNUNET_SCHEDULER_NO_TASK;
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Peer`%4s' disconnected due to timeout\n",
+              GNUNET_i2s (&n->id));
 
   GNUNET_STATISTICS_update (GST_stats,
-                            gettext_noop ("# peers disconnected due to timeout"), 1,
+                            gettext_noop
+                            ("# peers disconnected due to timeout"), 1,
                             GNUNET_NO);
   disconnect_neighbour (n);
 }
@@ -928,27 +1068,36 @@ neighbour_timeout_task (void *cls,
  */
 static void
 neighbour_keepalive_task (void *cls,
-                         const struct GNUNET_SCHEDULER_TaskContext *tc)
+                          const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct NeighbourMapEntry *n = cls;
   struct GNUNET_MessageHeader m;
+  int ret;
 
-  n->keepalive_task = GNUNET_SCHEDULER_add_delayed (KEEPALIVE_FREQUENCY,
-                                                   &neighbour_keepalive_task,
-                                                   n);
-  GNUNET_assert (is_connected(n));
-  GNUNET_STATISTICS_update (GST_stats,
-                           gettext_noop ("# keepalives sent"), 1,
-                           GNUNET_NO);
+  GNUNET_assert (S_CONNECTED == n->state);
+  n->keepalive_task =
+      GNUNET_SCHEDULER_add_delayed (KEEPALIVE_FREQUENCY,
+                                    &neighbour_keepalive_task, n);
+
+  GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# keepalives sent"), 1,
+                            GNUNET_NO);
   m.size = htons (sizeof (struct GNUNET_MessageHeader));
   m.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_KEEPALIVE);
 
-  send_with_plugin(&n->id, (const void *) &m,
-                   sizeof (m),
-                   UINT32_MAX /* priority */ ,
-                   GNUNET_TIME_UNIT_FOREVER_REL,
-                   n->session, n->plugin_name, n->addr, n->addrlen,
-                   GNUNET_YES, NULL, NULL);
+  ret = send_with_session (n,
+            (const void *) &m, sizeof (m),
+            UINT32_MAX /* priority */ ,
+            GNUNET_TIME_UNIT_FOREVER_REL,
+            NULL, NULL);
+
+  n->expect_latency_response = GNUNET_NO;
+  n->keep_alive_sent = GNUNET_TIME_absolute_get_zero ();
+  if (ret != GNUNET_SYSERR)
+  {
+    n->expect_latency_response = GNUNET_YES;
+    n->keep_alive_sent = GNUNET_TIME_absolute_get ();
+  }
+
 }
 
 
@@ -968,97 +1117,149 @@ disconnect_all_neighbours (void *cls, const GNUNET_HashCode * key, void *value)
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Disconnecting peer `%4s', %s\n",
               GNUNET_i2s (&n->id), "SHUTDOWN_TASK");
 #endif
-  if (is_connected(n))
+  if (S_CONNECTED == n->state)
     GNUNET_STATISTICS_update (GST_stats,
-                             gettext_noop ("# peers disconnected due to global disconnect"), 1,
-                             GNUNET_NO);
+                              gettext_noop
+                              ("# peers disconnected due to global disconnect"),
+                              1, GNUNET_NO);
   disconnect_neighbour (n);
   return GNUNET_OK;
 }
 
 
 static void
-ats_suggest_cancel (void *cls,
-    const struct GNUNET_SCHEDULER_TaskContext *tc)
+ats_suggest_cancel (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct NeighbourMapEntry *n = cls;
 
   n->ats_suggest = GNUNET_SCHEDULER_NO_TASK;
 
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              " ATS did not suggested address to connect to peer `%s'\n",
+              "ATS did not suggested address to connect to peer `%s'\n",
               GNUNET_i2s (&n->id));
 
-  disconnect_neighbour(n);
-}
+  GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# ATS address suggestions failed"), 1,
+                            GNUNET_NO);
 
 
+  disconnect_neighbour (n);
+}
+
 /**
  * Cleanup the neighbours subsystem.
  */
 void
 GST_neighbours_stop ()
 {
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 
   GNUNET_CONTAINER_multihashmap_iterate (neighbours, &disconnect_all_neighbours,
                                          NULL);
   GNUNET_CONTAINER_multihashmap_destroy (neighbours);
-  GNUNET_assert (neighbours_connected == 0);
+//  GNUNET_assert (neighbours_connected == 0);
   neighbours = NULL;
   callback_cls = NULL;
   connect_notify_cb = NULL;
   disconnect_notify_cb = NULL;
+  address_change_cb = NULL;
 }
 
+struct ContinutionContext
+{
+  struct GNUNET_HELLO_Address *address;
 
-/**
- * We tried to send a SESSION_CONNECT message to another peer.  If this
- * succeeded, we change the state.  If it failed, we should tell
- * ATS to not use this address anymore (until it is re-validated).
- *
- * @param cls the 'struct NeighbourMapEntry'
+  struct Session *session;
+};
+
+static void
+send_outbound_quota (const struct GNUNET_PeerIdentity *target,
+                     struct GNUNET_BANDWIDTH_Value32NBO quota)
+{
+  struct QuotaSetMessage q_msg;
+
+#if DEBUG_TRANSPORT
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Sending outbound quota of %u Bps for peer `%s' to all clients\n",
+              ntohl (quota.value__), GNUNET_i2s (target));
+#endif
+  q_msg.header.size = htons (sizeof (struct QuotaSetMessage));
+  q_msg.header.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SET_QUOTA);
+  q_msg.quota = quota;
+  q_msg.peer = (*target);
+  GST_clients_broadcast (&q_msg.header, GNUNET_NO);
+}
+
+/**
+ * We tried to send a SESSION_CONNECT message to another peer.  If this
+ * succeeded, we change the state.  If it failed, we should tell
+ * ATS to not use this address anymore (until it is re-validated).
+ *
+ * @param cls the 'struct GNUNET_HELLO_Address' of the address that was tried
+ * @param target peer to send the message to
  * @param success GNUNET_OK on success
  */
 static void
-send_connect_continuation (void *cls,
-      const struct GNUNET_PeerIdentity * target,
-      int success)
-
+send_connect_continuation (void *cls, const struct GNUNET_PeerIdentity *target,
+                           int success)
 {
-  struct NeighbourMapEntry *n = cls;
-
-  GNUNET_assert (n != NULL);
-  GNUNET_assert (!is_connected(n));
+  struct ContinutionContext *cc = cls;
+  struct NeighbourMapEntry *n = lookup_neighbour (&cc->address->peer);
 
-  if (is_disconnecting(n))
-    return; /* neighbour is going away */
   if (GNUNET_YES != success)
   {
+    GNUNET_assert (strlen (cc->address->transport_name) > 0);
+    GNUNET_ATS_address_destroyed (GST_ats, cc->address, cc->session);
+  }
+  if ((NULL == neighbours) || (NULL == n) || (n->state == S_DISCONNECT))
+  {
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;
+  }
+
+  if ((GNUNET_YES == success) &&
+      ((n->state == S_NOT_CONNECTED) || (n->state == S_CONNECT_SENT)))
+  {
+    change_state (n, S_CONNECT_SENT);
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;
+  }
+
+  if ((GNUNET_NO == success) &&
+      ((n->state == S_NOT_CONNECTED) || (n->state == S_CONNECT_SENT)))
+  {
 #if DEBUG_TRANSPORT
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Failed to send CONNECT_MSG to peer `%4s' with plugin `%s' address '%s' session %X, asking ATS for new address \n",
-              GNUNET_i2s (&n->id), n->plugin_name,
-              (n->addrlen == 0) ? "<inbound>" : GST_plugins_a2s (n->plugin_name,
-                                                                  n->addr,
-                                                                  n->addrlen),
-              n->session);
+                "Failed to send CONNECT_MSG to peer `%4s' with address '%s' session %p, asking ATS for new address \n",
+                GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), n->session);
 #endif
-
-    GNUNET_ATS_address_destroyed (GST_ats,
-                                 &n->id,
-                                 n->plugin_name, 
-                                 n->addr,
-                                 n->addrlen,
-                                 NULL);
-
-    if (n->ats_suggest!= GNUNET_SCHEDULER_NO_TASK)
-      GNUNET_SCHEDULER_cancel(n->ats_suggest);
-    n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel, n);
-    GNUNET_ATS_suggest_address(GST_ats, &n->id);
-    return;
+    change_state (n, S_NOT_CONNECTED);
+    if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
+      GNUNET_SCHEDULER_cancel (n->ats_suggest);
+    n->ats_suggest =
+        GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, &ats_suggest_cancel,
+                                      n);
+    GNUNET_ATS_suggest_address (GST_ats, &n->id);
   }
-  change_state(n, S_CONNECT_SENT);
+  GNUNET_HELLO_address_free (cc->address);
+  GNUNET_free (cc);
+}
+
+
+static void
+ats_request (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
+{
+  struct NeighbourMapEntry *n = cls;
+  n->ats_request = GNUNET_SCHEDULER_NO_TASK;
+
+  n->ats_suggest =
+    GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel,
+                                  n);
 }
 
 
@@ -1067,206 +1268,382 @@ send_connect_continuation (void *cls,
  * we should tell ATS to not use this address anymore (until it is re-validated).
  *
  * @param cls the 'struct NeighbourMapEntry'
+ * @param target peer to send the message to
  * @param success GNUNET_OK on success
  */
 static void
 send_switch_address_continuation (void *cls,
-      const struct GNUNET_PeerIdentity * target,
-      int success)
-
+                                  const struct GNUNET_PeerIdentity *target,
+                                  int success)
 {
-  struct NeighbourMapEntry *n = cls;
+  struct ContinutionContext *cc = cls;
+  struct NeighbourMapEntry *n;
 
-  GNUNET_assert (n != NULL);
-  if (is_disconnecting(n))
-    return; /* neighbour is going away */
+  if (neighbours == NULL)
+  {
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;                     /* neighbour is going away */
+  }
 
-  GNUNET_assert (n->state == S_CONNECTED);
+  n = lookup_neighbour (&cc->address->peer);
+  if ((n == NULL) || (is_disconnecting (n)))
+  {
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;                     /* neighbour is going away */
+  }
+
+  GNUNET_assert ((n->state == S_CONNECTED) || (n->state == S_FAST_RECONNECT));
   if (GNUNET_YES != success)
   {
-#if DEBUG_TRANSPORT
+
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Failed to switch connected peer `%s' to plugin `%s' address '%s' session %X, asking ATS for new address \n",
-              GNUNET_i2s (&n->id), n->plugin_name,
-              (n->addrlen == 0) ? "<inbound>" : GST_plugins_a2s (n->plugin_name,
-                                                                  n->addr,
-                                                                  n->addrlen),
-              n->session);
-#endif
+                "Failed to switch connected peer `%s' in state %s to address '%s' session %X, asking ATS for new address \n",
+                GNUNET_i2s (&n->id), print_state(n->state), GST_plugins_a2s (cc->address), cc->session);
 
-    GNUNET_ATS_address_destroyed (GST_ats,
-                                  &n->id,
-                                  n->plugin_name,
-                                  n->addr,
-                                  n->addrlen,
-                                  NULL);
+    GNUNET_assert (strlen (cc->address->transport_name) > 0);
+    GNUNET_ATS_address_destroyed (GST_ats, cc->address, cc->session);
 
     if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
-      GNUNET_SCHEDULER_cancel(n->ats_suggest);
-    n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel, n);
-    GNUNET_ATS_suggest_address(GST_ats, &n->id);
+    {
+      GNUNET_SCHEDULER_cancel (n->ats_suggest);
+      n->ats_suggest = GNUNET_SCHEDULER_NO_TASK;
+    }
+
+    if (n->state == S_FAST_RECONNECT)
+    {
+      if (GNUNET_SCHEDULER_NO_TASK == n->ats_request)
+      {
+        /* Throtteled ATS request for fast reconnect */
+        struct GNUNET_TIME_Relative delay = GNUNET_TIME_relative_multiply(FAST_RECONNECT_RATE, n->fast_reconnect_attempts);
+        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                    "Fast reconnect attempt %u failed, delay ATS request for %llu ms\n", n->fast_reconnect_attempts, delay.rel_value);
+        n->ats_request =
+            GNUNET_SCHEDULER_add_delayed (delay,
+                                          ats_request,
+                                          n);
+      }
+    }
+    else
+    {
+      /* Immediate ATS request for connected peers */
+      n->ats_suggest =
+        GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel,
+                                      n);
+      GNUNET_ATS_suggest_address (GST_ats, &n->id);
+    }
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
     return;
   }
+  /* Tell ATS that switching addresses was successful */
+  switch (n->state)
+  {
+  case S_CONNECTED:
+    if (n->address_state == FRESH)
+    {
+      GST_validation_set_address_use (cc->address, cc->session, GNUNET_YES,  __LINE__);
+      GNUNET_ATS_address_update (GST_ats, cc->address, cc->session, NULL, 0);
+      if (cc->session != n->session)
+        GNUNET_break (0);
+      GNUNET_ATS_address_in_use (GST_ats, cc->address, cc->session, GNUNET_YES);
+      n->address_state = USED;
+    }
+    break;
+  case S_FAST_RECONNECT:
+#if DEBUG_TRANSPORT
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Successful fast reconnect to peer `%s'\n",
+                GNUNET_i2s (&n->id));
+#endif
+    change_state (n, S_CONNECTED);
+    neighbours_connected++;
+    GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# peers connected"), 1,
+                              GNUNET_NO);
+
+    if (n->address_state == FRESH)
+    {
+      GST_validation_set_address_use (cc->address, cc->session, GNUNET_YES,  __LINE__);
+      GNUNET_ATS_address_update (GST_ats, cc->address, cc->session, NULL, 0);
+      GNUNET_ATS_address_in_use (GST_ats, cc->address, cc->session, GNUNET_YES);
+      n->address_state = USED;
+    }
+
+    if (n->keepalive_task == GNUNET_SCHEDULER_NO_TASK)
+      n->keepalive_task =
+          GNUNET_SCHEDULER_add_now (&neighbour_keepalive_task, n);
+
+    /* Updating quotas */
+    GST_neighbours_set_incoming_quota (&n->id, n->bandwidth_in);
+    send_outbound_quota (target, n->bandwidth_out);
+
+  default:
+    break;
+  }
+  GNUNET_HELLO_address_free (cc->address);
+  GNUNET_free (cc);
 }
 
+
 /**
  * We tried to send a SESSION_CONNECT message to another peer.  If this
  * succeeded, we change the state.  If it failed, we should tell
  * ATS to not use this address anymore (until it is re-validated).
  *
  * @param cls the 'struct NeighbourMapEntry'
+ * @param target peer to send the message to
  * @param success GNUNET_OK on success
  */
 static void
 send_connect_ack_continuation (void *cls,
-      const struct GNUNET_PeerIdentity * target,
-      int success)
-
+                               const struct GNUNET_PeerIdentity *target,
+                               int success)
 {
-  struct NeighbourMapEntry *n = cls;
+  struct ContinutionContext *cc = cls;
+  struct NeighbourMapEntry *n;
 
-  GNUNET_assert (n != NULL);
+  if (neighbours == NULL)
+  {
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;                     /* neighbour is going away */
+  }
+
+  n = lookup_neighbour (&cc->address->peer);
+  if ((n == NULL) || (is_disconnecting (n)))
+  {
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;                     /* neighbour is going away */
+  }
 
   if (GNUNET_YES == success)
-    return; /* sending successful */
+  {
+    GNUNET_HELLO_address_free (cc->address);
+    GNUNET_free (cc);
+    return;                     /* sending successful */
+  }
 
   /* sending failed, ask for next address  */
 #if DEBUG_TRANSPORT
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Failed to send CONNECT_MSG to peer `%4s' with plugin `%s' address '%s' session %X, asking ATS for new address \n",
-              GNUNET_i2s (&n->id), n->plugin_name,
-              (n->addrlen == 0) ? "<inbound>" : GST_plugins_a2s (n->plugin_name,
-                                                                  n->addr,
-                                                                  n->addrlen),
-              n->session);
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Failed to send CONNECT_MSG to peer `%4s' with address '%s' session %X, asking ATS for new address \n",
+              GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), n->session);
 #endif
-    change_state(n, S_NOT_CONNECTED);
-
-    GNUNET_ATS_address_destroyed (GST_ats,
-                                  &n->id,
-                                  n->plugin_name,
-                                  n->addr,
-                                  n->addrlen,
-                                  NULL);
+  change_state (n, S_NOT_CONNECTED);
+  GNUNET_assert (strlen (cc->address->transport_name) > 0);
+  GNUNET_ATS_address_destroyed (GST_ats, cc->address, cc->session);
 
-    if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
-      GNUNET_SCHEDULER_cancel(n->ats_suggest);
-    n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel, n);
-    GNUNET_ATS_suggest_address(GST_ats, &n->id);
+  if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
+    GNUNET_SCHEDULER_cancel (n->ats_suggest);
+  n->ats_suggest =
+      GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel,
+                                    n);
+  GNUNET_ATS_suggest_address (GST_ats, &n->id);
+  GNUNET_HELLO_address_free (cc->address);
+  GNUNET_free (cc);
 }
 
+
 /**
  * For an existing neighbour record, set the active connection to
- * the given address.
+ * use the given address.
  *
  * @param peer identity of the peer to switch the address for
- * @param plugin_name name of transport that delivered the PONG
  * @param address address of the other peer, NULL if other peer
  *                       connected to us
- * @param address_len number of bytes in address
  * @param session session to use (or NULL)
  * @param ats performance data
  * @param ats_count number of entries in ats
+ * @param bandwidth_in inbound quota to be used when connection is up
+ * @param bandwidth_out outbound quota to be used when connection is up
  * @return GNUNET_YES if we are currently connected, GNUNET_NO if the
  *         connection is not up (yet)
  */
 int
-GST_neighbours_switch_to_address_3way (const struct GNUNET_PeerIdentity *peer,
-                                  const char *plugin_name, const void *address,
-                                  size_t address_len, struct Session *session,
-                                  const struct GNUNET_ATS_Information
-                                  *ats, uint32_t ats_count,
-                                  struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in,
-                                  struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out)
+GST_neighbours_switch_to_address (const struct GNUNET_PeerIdentity *peer,
+                                       const struct GNUNET_HELLO_Address
+                                       *address,
+                                       struct Session *session,
+                                       const struct GNUNET_ATS_Information *ats,
+                                       uint32_t ats_count,
+                                       struct GNUNET_BANDWIDTH_Value32NBO
+                                       bandwidth_in,
+                                       struct GNUNET_BANDWIDTH_Value32NBO
+                                       bandwidth_out)
 {
   struct NeighbourMapEntry *n;
   struct SessionConnectMessage connect_msg;
+  struct ContinutionContext *cc;
   size_t msg_len;
   size_t ret;
 
-  GNUNET_assert (neighbours != NULL);
+  if (neighbours == NULL)
+  {
+    /* This can happen during shutdown */
+    return GNUNET_NO;
+  }
   n = lookup_neighbour (peer);
   if (NULL == n)
+    return GNUNET_NO;
+  if (n->state == S_DISCONNECT)
+  {
+    /* We are disconnecting, nothing to do here */
+    return GNUNET_NO;
+  }
+  GNUNET_assert (address->transport_name != NULL);
+  if ((session == NULL) && (0 == address->address_length))
   {
-    if (NULL == session)
-      GNUNET_ATS_address_destroyed (GST_ats,
-                                   peer,
-                                   plugin_name, address,
-                                   address_len, NULL);    
+    GNUNET_break_op (0);
+    /* FIXME: is this actually possible? When does this happen? */
+    if (strlen (address->transport_name) > 0)
+      GNUNET_ATS_address_destroyed (GST_ats, address, session);
+    GNUNET_ATS_suggest_address (GST_ats, peer);
     return GNUNET_NO;
   }
 
+  /* checks successful and neighbour != NULL */
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "ATS tells us to switch to address '%s' session %p for peer `%s' in state `%s'\n",
+              (address->address_length != 0) ? GST_plugins_a2s (address): "<inbound>",
+              session,
+              GNUNET_i2s (peer),
+              print_state (n->state));
+
   if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
   {
-    GNUNET_SCHEDULER_cancel(n->ats_suggest);
+    GNUNET_SCHEDULER_cancel (n->ats_suggest);
     n->ats_suggest = GNUNET_SCHEDULER_NO_TASK;
   }
 
-#if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "ATS tells us to switch to plugin `%s' address '%s' session %X for %s peer `%s'\n",
-              plugin_name,
-              (address_len == 0) ? "<inbound>" : GST_plugins_a2s (plugin_name,
-                                                                  address,
-                                                                  address_len),
-              session, (is_connected(n) ? "CONNECTED" : "NOT CONNECTED"),
-              GNUNET_i2s (peer));
-#endif
 
-  // do not switch addresses just update quotas
-  if (n != NULL)
+  if (n->state == S_FAST_RECONNECT)
   {
-    if ((is_connected(n)) && (address_len == n->addrlen))
+    /* Throtteled fast reconnect */
+
+    if (NULL == n->fast_reconnect_address)
     {
-      if ((0 == memcmp (address, n->addr, address_len)) &&
-          (n->session == session))
-      {
-        struct QuotaSetMessage q_msg;
+      n->fast_reconnect_address = GNUNET_HELLO_address_copy (address);
 
-#if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Sending outbound quota of %u Bps and inbound quota of % Bps for peer `%s' to all clients\n",
-              ntohl (n->bandwidth_out.value__),
-              ntohl (n->bandwidth_in.value__),
-              GNUNET_i2s (peer));
-#endif
+    }
+    else if (0 == GNUNET_HELLO_address_cmp(address, n->fast_reconnect_address))
+    {
+      n->fast_reconnect_attempts ++;
 
-        n->bandwidth_in = bandwidth_in;
-        n->bandwidth_out = bandwidth_out;
-        GST_neighbours_set_incoming_quota(&n->id, n->bandwidth_in);
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "FAST RECONNECT to peer `%s' and address '%s' with identical address attempt %u\n",
+                GNUNET_i2s (&n->id), GST_plugins_a2s (address), n->fast_reconnect_attempts);
+    }
+  }
+  else
+  {
+    n->fast_reconnect_attempts = 0;
+  }
 
-        q_msg.header.size = htons (sizeof (struct QuotaSetMessage));
-        q_msg.header.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SET_QUOTA);
-        q_msg.quota = n->bandwidth_out;
-        q_msg.peer = (*peer);
-        GST_clients_broadcast (&q_msg.header, GNUNET_NO);
-        return GNUNET_NO;
-      }
+  /* do not switch addresses just update quotas */
+  if ((n->state == S_CONNECTED) && (NULL != n->address) &&
+      (0 == GNUNET_HELLO_address_cmp (address, n->address)) &&
+      (n->session == session))
+  {
+    n->bandwidth_in = bandwidth_in;
+    n->bandwidth_out = bandwidth_out;
+    GST_neighbours_set_incoming_quota (&n->id, n->bandwidth_in);
+    send_outbound_quota (peer, n->bandwidth_out);
+    return GNUNET_NO;
+  }
+  if (n->state == S_CONNECTED)
+  {
+    /* mark old address as no longer used */
+    GNUNET_assert (NULL != n->address);
+    if (n->address_state == USED)
+    {
+      GST_validation_set_address_use (n->address, n->session, GNUNET_NO,  __LINE__);
+      GNUNET_ATS_address_in_use (GST_ats, n->address, n->session, GNUNET_NO);
+      n->address_state = UNUSED;
     }
   }
 
-  GNUNET_free_non_null (n->addr);
-  n->addr = GNUNET_malloc (address_len);
-  memcpy (n->addr, address, address_len);
+  /* set new address */
+  if (NULL != n->address)
+    GNUNET_HELLO_address_free (n->address);
+  n->address = GNUNET_HELLO_address_copy (address);
+  n->address_state = FRESH;
   n->bandwidth_in = bandwidth_in;
   n->bandwidth_out = bandwidth_out;
-  n->addrlen = address_len;
-  n->session = session;
-  GNUNET_free_non_null (n->plugin_name);
-  n->plugin_name = GNUNET_strdup (plugin_name);
   GNUNET_SCHEDULER_cancel (n->timeout_task);
   n->timeout_task =
       GNUNET_SCHEDULER_add_delayed (GNUNET_CONSTANTS_IDLE_CONNECTION_TIMEOUT,
                                     &neighbour_timeout_task, n);
 
-  if (n->state == S_DISCONNECT)
+  if (NULL != address_change_cb && n->state == S_CONNECTED)
+    address_change_cb (callback_cls, &n->id, n->address); 
+
+  /* Obtain an session for this address from plugin */
+  struct GNUNET_TRANSPORT_PluginFunctions *papi;
+  papi = GST_plugins_find (address->transport_name);
+
+  if (papi == NULL)
   {
-    /* We are disconnecting, nothing to do here */
+    /* we don't have the plugin for this address */
+    GNUNET_ATS_address_destroyed (GST_ats, n->address, NULL);
+
+    if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
+      GNUNET_SCHEDULER_cancel (n->ats_suggest);
+    n->ats_suggest =  GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT,
+                                      ats_suggest_cancel,
+                                      n);
+    GNUNET_ATS_suggest_address (GST_ats, &n->id);
+    GNUNET_HELLO_address_free (n->address);
+    n->address = NULL;
+    n->session = NULL;
     return GNUNET_NO;
   }
-  /* We are not connected/connecting and initiate a fresh connect */
-  if (n->state == S_NOT_CONNECTED)
+
+  if (session == NULL)
+  {
+    n->session = papi->get_session (papi->cls, address);
+    /* Session could not be initiated */
+    if (n->session == NULL)
+    {
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                  "Failed to obtain new session %p for peer `%s' and  address '%s'\n",
+                  n->session, GNUNET_i2s (&n->id), GST_plugins_a2s (n->address));
+
+      GNUNET_ATS_address_destroyed (GST_ats, n->address, NULL);
+
+      if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
+        GNUNET_SCHEDULER_cancel (n->ats_suggest);
+      n->ats_suggest =  GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT,
+                                        ats_suggest_cancel,
+                                        n);
+      GNUNET_ATS_suggest_address (GST_ats, &n->id);
+      GNUNET_HELLO_address_free (n->address);
+      n->address = NULL;
+      n->session = NULL;
+      return GNUNET_NO;
+    }
+
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Obtained new session %p for peer `%s' and  address '%s'\n",
+                 n->session, GNUNET_i2s (&n->id), GST_plugins_a2s (n->address));
+    /* Telling ATS about new session */
+    GNUNET_ATS_address_update (GST_ats, n->address, n->session, NULL, 0);
+  }
+  else
   {
+    n->session = session;
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Using existing session %p for peer `%s' and  address '%s'\n",
+                n->session,
+                GNUNET_i2s (&n->id),
+                (address->address_length != 0) ? GST_plugins_a2s (address): "<inbound>");
+  }
+
+  switch (n->state)
+  {
+  case S_NOT_CONNECTED:
+  case S_CONNECT_SENT:
     msg_len = sizeof (struct SessionConnectMessage);
     connect_msg.header.size = htons (msg_len);
     connect_msg.header.type =
@@ -1275,30 +1652,38 @@ GST_neighbours_switch_to_address_3way (const struct GNUNET_PeerIdentity *peer,
     connect_msg.timestamp =
         GNUNET_TIME_absolute_hton (GNUNET_TIME_absolute_get ());
 
-    ret = send_with_plugin (peer, (const char *) &connect_msg, msg_len, UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
-                            session, plugin_name, address, address_len,
-                            GNUNET_YES, &send_connect_continuation, n);
+    cc = GNUNET_malloc (sizeof (struct ContinutionContext));
+    cc->session = n->session;
+    cc->address = GNUNET_HELLO_address_copy (address);
+
+    ret = send_with_session (n,
+      (const char *) &connect_msg, msg_len,
+      UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
+      &send_connect_continuation, cc);
 
     return GNUNET_NO;
-  }
-  /* We received a CONNECT message and asked ATS for an address */
-  else if (n->state == S_CONNECT_RECV)
-  {
+  case S_CONNECT_RECV:
+    /* We received a CONNECT message and asked ATS for an address */
     msg_len = sizeof (struct SessionConnectMessage);
     connect_msg.header.size = htons (msg_len);
     connect_msg.header.type =
-      htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_CONNECT_ACK);
+        htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_CONNECT_ACK);
     connect_msg.reserved = htonl (0);
-    connect_msg.timestamp = GNUNET_TIME_absolute_hton (GNUNET_TIME_absolute_get ());
-
-    ret = send_with_plugin(&n->id, (const void *) &connect_msg, msg_len, UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
-                           session, plugin_name, address, address_len,
-                           GNUNET_YES, &send_connect_ack_continuation, n);
+    connect_msg.timestamp =
+        GNUNET_TIME_absolute_hton (GNUNET_TIME_absolute_get ());
+    cc = GNUNET_malloc (sizeof (struct ContinutionContext));
+    cc->session = n->session;
+    cc->address = GNUNET_HELLO_address_copy (address);
+
+    ret = send_with_session(n,
+                            (const void *) &connect_msg, msg_len,
+                            UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
+                            &send_connect_ack_continuation,
+                            cc);
     return GNUNET_NO;
-  }
-  /* connected peer is switching addresses */
-  else if (n->state == S_CONNECTED)
-  {
+  case S_CONNECTED:
+  case S_FAST_RECONNECT:
+    /* connected peer is switching addresses or tries fast reconnect */
     msg_len = sizeof (struct SessionConnectMessage);
     connect_msg.header.size = htons (msg_len);
     connect_msg.header.type =
@@ -1306,36 +1691,71 @@ GST_neighbours_switch_to_address_3way (const struct GNUNET_PeerIdentity *peer,
     connect_msg.reserved = htonl (0);
     connect_msg.timestamp =
         GNUNET_TIME_absolute_hton (GNUNET_TIME_absolute_get ());
-
-    ret = send_with_plugin (peer, (const char *) &connect_msg, msg_len, UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
-                            session, plugin_name, address, address_len,
-                            GNUNET_YES, &send_switch_address_continuation, n);
+    cc = GNUNET_malloc (sizeof (struct ContinutionContext));
+    cc->session = n->session;
+    cc->address = GNUNET_HELLO_address_copy (address);
+    ret = send_with_session(n,
+                            (const void *) &connect_msg, msg_len,
+                            UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
+                            &send_switch_address_continuation, cc);
     if (ret == GNUNET_SYSERR)
     {
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                "Failed to send CONNECT_MESSAGE to `%4s' using plugin `%s' address '%s' session %X\n",
-                GNUNET_i2s (peer), plugin_name,
-                (address_len == 0) ? "<inbound>" : GST_plugins_a2s (plugin_name,
-                                                                    address,
-                                                                    address_len),
-                session);
+                  "Failed to send CONNECT_MESSAGE to `%4s' using address '%s' session %X\n",
+                  GNUNET_i2s (peer), GST_plugins_a2s (address), session);
     }
     return GNUNET_NO;
+  default:
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                "Invalid connection state to switch addresses %u \n", n->state);
+    GNUNET_break_op (0);
+    return GNUNET_NO;
   }
-  else if (n->state == S_CONNECT_SENT)
-  {
-      //FIXME
-     return GNUNET_NO;
-  }
-  GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Invalid connection state to switch addresses %u \n", n->state);
-  GNUNET_break_op (0);
-  return GNUNET_NO;
 }
 
 
+/**
+ * Obtain current latency information for the given neighbour.
+ *
+ * @param peer
+ * @return observed latency of the address, FOREVER if the address was
+ *         never successfully validated
+ */
+struct GNUNET_TIME_Relative
+GST_neighbour_get_latency (const struct GNUNET_PeerIdentity *peer)
+{
+  struct NeighbourMapEntry *n;
+
+  n = lookup_neighbour (peer);
+  if ((NULL == n) || ((n->address == NULL) && (n->session == NULL)))
+    return GNUNET_TIME_UNIT_FOREVER_REL;
+
+  return n->latency;
+}
+
+/**
+ * Obtain current address information for the given neighbour.
+ *
+ * @param peer
+ * @return address currently used
+ */
+struct GNUNET_HELLO_Address *
+GST_neighbour_get_current_address (const struct GNUNET_PeerIdentity *peer)
+{
+  struct NeighbourMapEntry *n;
+
+  n = lookup_neighbour (peer);
+  if ((NULL == n) || ((n->address == NULL) && (n->session == NULL)))
+    return NULL;
+
+  return n->address;
+}
+
+
+
 /**
  * Create an entry in the neighbour map for the given peer
- * 
+ *
  * @param peer peer to create an entry for
  * @return new neighbour map entry
  */
@@ -1346,22 +1766,22 @@ setup_neighbour (const struct GNUNET_PeerIdentity *peer)
 
 #if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-             "Unknown peer `%s', creating new neighbour\n",
-             GNUNET_i2s (peer));
+              "Unknown peer `%s', creating new neighbour\n", GNUNET_i2s (peer));
 #endif
   n = GNUNET_malloc (sizeof (struct NeighbourMapEntry));
   n->id = *peer;
   n->state = S_NOT_CONNECTED;
+  n->latency = GNUNET_TIME_relative_get_forever ();
   GNUNET_BANDWIDTH_tracker_init (&n->in_tracker,
-                                GNUNET_CONSTANTS_DEFAULT_BW_IN_OUT,
-                                MAX_BANDWIDTH_CARRY_S);
+                                 GNUNET_CONSTANTS_DEFAULT_BW_IN_OUT,
+                                 MAX_BANDWIDTH_CARRY_S);
   n->timeout_task =
-    GNUNET_SCHEDULER_add_delayed (GNUNET_CONSTANTS_IDLE_CONNECTION_TIMEOUT,
-                                 &neighbour_timeout_task, n);
+      GNUNET_SCHEDULER_add_delayed (GNUNET_CONSTANTS_IDLE_CONNECTION_TIMEOUT,
+                                    &neighbour_timeout_task, n);
   GNUNET_assert (GNUNET_OK ==
-                GNUNET_CONTAINER_multihashmap_put (neighbours,
-                                                   &n->id.hashPubKey, n,
-                                                   GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY));
+                 GNUNET_CONTAINER_multihashmap_put (neighbours,
+                                                    &n->id.hashPubKey, n,
+                                                    GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY));
   return n;
 }
 
@@ -1376,21 +1796,28 @@ GST_neighbours_try_connect (const struct GNUNET_PeerIdentity *target)
 {
   struct NeighbourMapEntry *n;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 #if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Trying to connect to peer `%s'\n",
               GNUNET_i2s (target));
 #endif
-  GNUNET_assert (0 !=
-                 memcmp (target, &GST_my_identity,
-                         sizeof (struct GNUNET_PeerIdentity)));
+  if (0 ==
+      memcmp (target, &GST_my_identity, sizeof (struct GNUNET_PeerIdentity)))
+  {
+    /* my own hello */
+    return;
+  }
   n = lookup_neighbour (target);
 
   if (NULL != n)
   {
-    if ((is_connected(n)) || (is_connecting(n)))
-      return;                     /* already connecting or connected */
-    if (is_disconnecting(n))
+    if ((S_CONNECTED == n->state) || (is_connecting (n)))
+      return;                   /* already connecting or connected */
+    if (is_disconnecting (n))
       change_state (n, S_NOT_CONNECTED);
   }
 
@@ -1403,7 +1830,7 @@ GST_neighbours_try_connect (const struct GNUNET_PeerIdentity *target)
               GNUNET_i2s (&n->id));
 #endif
 
-   GNUNET_ATS_suggest_address (GST_ats, &n->id);
+  GNUNET_ATS_suggest_address (GST_ats, &n->id);
 }
 
 /**
@@ -1417,16 +1844,19 @@ GST_neighbours_test_connected (const struct GNUNET_PeerIdentity *target)
 {
   struct NeighbourMapEntry *n;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return GNUNET_NO;
+  }
 
   n = lookup_neighbour (target);
 
-  if ((NULL == n) || (!is_connected(n)))
+  if ((NULL == n) || (S_CONNECTED != n->state))
     return GNUNET_NO;           /* not connected */
   return GNUNET_YES;
 }
 
-
 /**
  * A session was terminated. Take note.
  *
@@ -1439,11 +1869,14 @@ GST_neighbours_session_terminated (const struct GNUNET_PeerIdentity *peer,
 {
   struct NeighbourMapEntry *n;
 
-  GNUNET_assert (neighbours != NULL);
+  if (neighbours == NULL)
+  {
+    /* This can happen during shutdown */
+    return;
+  }
 
 #if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Session %X to peer `%s' ended \n",
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Session %X to peer `%s' ended \n",
               session, GNUNET_i2s (peer));
 #endif
 
@@ -1452,25 +1885,61 @@ GST_neighbours_session_terminated (const struct GNUNET_PeerIdentity *peer,
     return;
   if (session != n->session)
     return;                     /* doesn't affect us */
+  if (n->state == S_CONNECTED)
+  {
+    if (n->address_state == USED)
+    {
+      GST_validation_set_address_use (n->address, n->session, GNUNET_NO,  __LINE__);
+      GNUNET_ATS_address_in_use (GST_ats, n->address, n->session, GNUNET_NO);
+      n->address_state = UNUSED;
+
+    }
+  }
 
+  if (NULL != n->address)
+  {
+    GNUNET_HELLO_address_free (n->address);
+    n->address = NULL;
+  }
   n->session = NULL;
-  GNUNET_free (n->addr);
-  n->addr = NULL;
-  n->addrlen = 0;
 
   /* not connected anymore anyway, shouldn't matter */
-  if ((!is_connected(n)) && (!is_connecting(n)))
+  if (S_CONNECTED != n->state)
     return;
 
+  if (n->keepalive_task != GNUNET_SCHEDULER_NO_TASK)
+  {
+    GNUNET_SCHEDULER_cancel (n->keepalive_task);
+    n->keepalive_task = GNUNET_SCHEDULER_NO_TASK;
+    n->expect_latency_response = GNUNET_NO;
+  }
+
+  /* connected, try fast reconnect */
+  /* statistics "transport" : "# peers connected" -= 1
+   * neighbours_connected -= 1
+   * BUT: no disconnect_cb to notify clients about disconnect
+   */
+
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Trying fast reconnect to peer `%s'\n",
+              GNUNET_i2s (peer));
+
+  GNUNET_assert (neighbours_connected > 0);
+  change_state (n, S_FAST_RECONNECT);
+  neighbours_connected--;
+  GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# peers connected"), -1,
+                            GNUNET_NO);
+
+
   /* We are connected, so ask ATS to switch addresses */
   GNUNET_SCHEDULER_cancel (n->timeout_task);
-  n->timeout_task =
-      GNUNET_SCHEDULER_add_delayed (GNUNET_CONSTANTS_DISCONNECT_SESSION_TIMEOUT,
+  n->timeout_task = GNUNET_SCHEDULER_add_delayed (GNUNET_CONSTANTS_DISCONNECT_SESSION_TIMEOUT,
                                     &neighbour_timeout_task, n);
   /* try QUICKLY to re-establish a connection, reduce timeout! */
   if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
-    GNUNET_SCHEDULER_cancel(n->ats_suggest);
-  n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel, n);
+    GNUNET_SCHEDULER_cancel (n->ats_suggest);
+  n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT,
+                                    &ats_suggest_cancel,
+                                    n);
   GNUNET_ATS_suggest_address (GST_ats, peer);
 }
 
@@ -1493,10 +1962,14 @@ GST_neighbours_send (const struct GNUNET_PeerIdentity *target, const void *msg,
   struct NeighbourMapEntry *n;
   struct MessageQueue *mq;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 
   n = lookup_neighbour (target);
-  if ((n == NULL) || (!is_connected(n)))
+  if ((n == NULL) || (!is_connected (n)))
   {
     GNUNET_STATISTICS_update (GST_stats,
                               gettext_noop
@@ -1507,7 +1980,7 @@ GST_neighbours_send (const struct GNUNET_PeerIdentity *target, const void *msg,
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                   "Could not send message to peer `%s': unknown neighbour",
                   GNUNET_i2s (target));
-    else if (!is_connected(n))
+    else if (!is_connected (n))
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                   "Could not send message to peer `%s': not connected\n",
                   GNUNET_i2s (target));
@@ -1517,16 +1990,16 @@ GST_neighbours_send (const struct GNUNET_PeerIdentity *target, const void *msg,
     return;
   }
 
-  if ((n->session == NULL) && (n->addr == NULL) && (n->addrlen ==0))
+  if ((n->session == NULL) && (n->address == NULL))
   {
     GNUNET_STATISTICS_update (GST_stats,
                               gettext_noop
                               ("# messages not sent (no such peer or not connected)"),
                               1, GNUNET_NO);
 #if DEBUG_TRANSPORT
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  "Could not send message to peer `%s': no address available\n",
-                  GNUNET_i2s (target));
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Could not send message to peer `%s': no address available\n",
+                GNUNET_i2s (target));
 #endif
 
     if (NULL != cont)
@@ -1574,7 +2047,11 @@ GST_neighbours_calculate_receive_delay (const struct GNUNET_PeerIdentity
   struct NeighbourMapEntry *n;
   struct GNUNET_TIME_Relative ret;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return GNUNET_TIME_UNIT_FOREVER_REL;
+  }
 
   n = lookup_neighbour (sender);
   if (n == NULL)
@@ -1584,14 +2061,14 @@ GST_neighbours_calculate_receive_delay (const struct GNUNET_PeerIdentity
     if (NULL == n)
     {
       GNUNET_STATISTICS_update (GST_stats,
-                               gettext_noop
-                               ("# messages discarded due to lack of neighbour record"),
-                               1, GNUNET_NO);
+                                gettext_noop
+                                ("# messages discarded due to lack of neighbour record"),
+                                1, GNUNET_NO);
       *do_forward = GNUNET_NO;
       return GNUNET_TIME_UNIT_ZERO;
     }
   }
-  if (!is_connected(n))
+  if (!is_connected (n))
   {
     *do_forward = GNUNET_SYSERR;
     return GNUNET_TIME_UNIT_ZERO;
@@ -1657,7 +2134,11 @@ GST_neighbours_keepalive (const struct GNUNET_PeerIdentity *neighbour)
 {
   struct NeighbourMapEntry *n;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 
   n = lookup_neighbour (neighbour);
   if (NULL == n)
@@ -1672,6 +2153,99 @@ GST_neighbours_keepalive (const struct GNUNET_PeerIdentity *neighbour)
   n->timeout_task =
       GNUNET_SCHEDULER_add_delayed (GNUNET_CONSTANTS_IDLE_CONNECTION_TIMEOUT,
                                     &neighbour_timeout_task, n);
+
+  /* send reply to measure latency */
+  if (S_CONNECTED != n->state)
+    return;
+
+  struct GNUNET_MessageHeader m;
+
+  m.size = htons (sizeof (struct GNUNET_MessageHeader));
+  m.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_KEEPALIVE_RESPONSE);
+
+  send_with_session(n,
+      (const void *) &m, sizeof (m),
+      UINT32_MAX,
+      GNUNET_TIME_UNIT_FOREVER_REL,
+      NULL, NULL);
+}
+
+/**
+ * We received a KEEP_ALIVE_RESPONSE message and use this to calculate latency
+ * to this peer
+ *
+ * @param neighbour neighbour to keep alive
+ * @param ats performance data
+ * @param ats_count number of entries in ats
+ */
+void
+GST_neighbours_keepalive_response (const struct GNUNET_PeerIdentity *neighbour,
+                                   const struct GNUNET_ATS_Information *ats,
+                                   uint32_t ats_count)
+{
+  struct NeighbourMapEntry *n;
+  struct GNUNET_ATS_Information *ats_new;
+  uint32_t latency;
+
+  if (neighbours == NULL)
+  {
+    // This can happen during shutdown
+    return;
+  }
+
+  n = lookup_neighbour (neighbour);
+  if ((NULL == n) || (n->state != S_CONNECTED))
+  {
+    GNUNET_STATISTICS_update (GST_stats,
+                              gettext_noop
+                              ("# KEEPALIVE_RESPONSE messages discarded (not connected)"),
+                              1, GNUNET_NO);
+    return;
+  }
+  if (n->expect_latency_response != GNUNET_YES)
+  {
+    GNUNET_STATISTICS_update (GST_stats,
+                              gettext_noop
+                              ("# KEEPALIVE_RESPONSE messages discarded (not expected)"),
+                              1, GNUNET_NO);
+    return;
+  }
+  n->expect_latency_response = GNUNET_NO;
+
+  GNUNET_assert (n->keep_alive_sent.abs_value !=
+                 GNUNET_TIME_absolute_get_zero ().abs_value);
+  n->latency =
+      GNUNET_TIME_absolute_get_difference (n->keep_alive_sent,
+                                           GNUNET_TIME_absolute_get ());
+#if DEBUG_TRANSPORT
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Latency for peer `%s' is %llu ms\n",
+              GNUNET_i2s (&n->id), n->latency.rel_value);
+#endif
+
+
+  if (n->latency.rel_value == GNUNET_TIME_relative_get_forever ().rel_value)
+  {
+    GNUNET_ATS_address_update (GST_ats, n->address, n->session, ats, ats_count);
+  }
+  else
+  {
+    ats_new =
+        GNUNET_malloc (sizeof (struct GNUNET_ATS_Information) *
+                       (ats_count + 1));
+    memcpy (ats_new, ats, sizeof (struct GNUNET_ATS_Information) * ats_count);
+
+    /* add latency */
+    ats_new[ats_count].type = htonl (GNUNET_ATS_QUALITY_NET_DELAY);
+    if (n->latency.rel_value > UINT32_MAX)
+      latency = UINT32_MAX;
+    else
+      latency = n->latency.rel_value;
+    ats_new[ats_count].value = htonl (latency);
+
+    GNUNET_ATS_address_update (GST_ats, n->address, n->session, ats_new,
+                               ats_count + 1);
+    GNUNET_free (ats_new);
+  }
 }
 
 
@@ -1687,7 +2261,11 @@ GST_neighbours_set_incoming_quota (const struct GNUNET_PeerIdentity *neighbour,
 {
   struct NeighbourMapEntry *n;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 
   n = lookup_neighbour (neighbour);
   if (n == NULL)
@@ -1698,6 +2276,11 @@ GST_neighbours_set_incoming_quota (const struct GNUNET_PeerIdentity *neighbour,
                               1, GNUNET_NO);
     return;
   }
+#if DEBUG_TRANSPORT
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Setting inbound quota of %u Bps for peer `%s' to all clients\n",
+              ntohl (quota.value__), GNUNET_i2s (&n->id));
+#endif
   GNUNET_BANDWIDTH_tracker_update_quota (&n->in_tracker, quota);
   if (0 != ntohl (quota.value__))
     return;
@@ -1705,10 +2288,10 @@ GST_neighbours_set_incoming_quota (const struct GNUNET_PeerIdentity *neighbour,
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Disconnecting peer `%4s' due to `%s'\n",
               GNUNET_i2s (&n->id), "SET_QUOTA");
 #endif
-  if (is_connected(n))
+  if (is_connected (n))
     GNUNET_STATISTICS_update (GST_stats,
-                             gettext_noop ("# disconnects due to quota of 0"), 1,
-                             GNUNET_NO);
+                              gettext_noop ("# disconnects due to quota of 0"),
+                              1, GNUNET_NO);
   disconnect_neighbour (n);
 }
 
@@ -1744,10 +2327,10 @@ neighbours_iterate (void *cls, const GNUNET_HashCode * key, void *value)
   struct IteratorContext *ic = cls;
   struct NeighbourMapEntry *n = value;
 
-  if (!is_connected(n))
+  if (!is_connected (n))
     return GNUNET_OK;
 
-  ic->cb (ic->cb_cls, &n->id, NULL, 0, n->plugin_name, n->addr, n->addrlen);
+  ic->cb (ic->cb_cls, &n->id, NULL, 0, n->address);
   return GNUNET_OK;
 }
 
@@ -1763,7 +2346,11 @@ GST_neighbours_iterate (GST_NeighbourIterator cb, void *cb_cls)
 {
   struct IteratorContext ic;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 
   ic.cb = cb;
   ic.cb_cls = cb_cls;
@@ -1780,19 +2367,15 @@ GST_neighbours_force_disconnect (const struct GNUNET_PeerIdentity *target)
 {
   struct NeighbourMapEntry *n;
 
-  GNUNET_assert (neighbours != NULL);
+  // This can happen during shutdown
+  if (neighbours == NULL)
+  {
+    return;
+  }
 
   n = lookup_neighbour (target);
   if (NULL == n)
     return;                     /* not active */
-  if (is_connected(n))
-  {
-    send_disconnect(&n->id, n->plugin_name, n->addr, n->addrlen, n->session);
-
-    n = lookup_neighbour (target);
-    if (NULL == n)
-      return;                     /* gone already */
-  }
   disconnect_neighbour (n);
 }
 
@@ -1800,13 +2383,15 @@ GST_neighbours_force_disconnect (const struct GNUNET_PeerIdentity *target)
 /**
  * We received a disconnect message from the given peer,
  * validate and process.
- * 
+ *
  * @param peer sender of the message
  * @param msg the disconnect message
  */
 void
-GST_neighbours_handle_disconnect_message (const struct GNUNET_PeerIdentity *peer,
-                                         const struct GNUNET_MessageHeader *msg)
+GST_neighbours_handle_disconnect_message (const struct GNUNET_PeerIdentity
+                                          *peer,
+                                          const struct GNUNET_MessageHeader
+                                          *msg)
 {
   struct NeighbourMapEntry *n;
   const struct SessionDisconnectMessage *sdm;
@@ -1814,18 +2399,20 @@ GST_neighbours_handle_disconnect_message (const struct GNUNET_PeerIdentity *peer
 
 #if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-      "Received DISCONNECT message from peer `%s'\n", GNUNET_i2s (peer));
+              "Received DISCONNECT message from peer `%s'\n",
+              GNUNET_i2s (peer));
 #endif
 
   if (ntohs (msg->size) != sizeof (struct SessionDisconnectMessage))
   {
     // GNUNET_break_op (0);
     GNUNET_STATISTICS_update (GST_stats,
-                             gettext_noop ("# disconnect messages ignored (old format)"), 1,
-                             GNUNET_NO);
+                              gettext_noop
+                              ("# disconnect messages ignored (old format)"), 1,
+                              GNUNET_NO);
     return;
   }
-  sdm = (const struct SessionDisconnectMessage) msg;
+  sdm = (const struct SessionDisconnectMessage *) msg;
   n = lookup_neighbour (peer);
   if (NULL == n)
     return;                     /* gone already */
@@ -1833,21 +2420,20 @@ GST_neighbours_handle_disconnect_message (const struct GNUNET_PeerIdentity *peer
       n->connect_ts.abs_value)
   {
     GNUNET_STATISTICS_update (GST_stats,
-                             gettext_noop ("# disconnect messages ignored (timestamp)"), 1,
-                             GNUNET_NO);
+                              gettext_noop
+                              ("# disconnect messages ignored (timestamp)"), 1,
+                              GNUNET_NO);
     return;
   }
   GNUNET_CRYPTO_hash (&sdm->public_key,
-                     sizeof (struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded),
-                     &hc);
-  if (0 != memcmp (peer,
-                  &hc,
-                  sizeof (struct GNUNET_PeerIdentity)))
+                      sizeof (struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded),
+                      &hc);
+  if (0 != memcmp (peer, &hc, sizeof (struct GNUNET_PeerIdentity)))
   {
     GNUNET_break_op (0);
     return;
   }
-  if (ntohl (sdm->purpose.size) != 
+  if (ntohl (sdm->purpose.size) !=
       sizeof (struct GNUNET_CRYPTO_RsaSignaturePurpose) +
       sizeof (struct GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded) +
       sizeof (struct GNUNET_TIME_AbsoluteNBO))
@@ -1856,10 +2442,9 @@ GST_neighbours_handle_disconnect_message (const struct GNUNET_PeerIdentity *peer
     return;
   }
   if (GNUNET_OK !=
-      GNUNET_CRYPTO_rsa_verify (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_DISCONNECT,
-                               &sdm->purpose,
-                               &sdm->signature,
-                               &sdm->public_key))
+      GNUNET_CRYPTO_rsa_verify
+      (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_DISCONNECT, &sdm->purpose,
+       &sdm->signature, &sdm->public_key))
   {
     GNUNET_break_op (0);
     return;
@@ -1867,155 +2452,165 @@ GST_neighbours_handle_disconnect_message (const struct GNUNET_PeerIdentity *peer
   GST_neighbours_force_disconnect (peer);
 }
 
+
 /**
  * We received a 'SESSION_CONNECT_ACK' message from the other peer.
  * Consider switching to it.
  *
  * @param message possibly a 'struct SessionConnectMessage' (check format)
  * @param peer identity of the peer to switch the address for
- * @param plugin_name name of transport that delivered the PONG
  * @param address address of the other peer, NULL if other peer
  *                       connected to us
- * @param address_len number of bytes in address
  * @param session session to use (or NULL)
  * @param ats performance data
  * @param ats_count number of entries in ats
 */
+ */
 void
 GST_neighbours_handle_connect_ack (const struct GNUNET_MessageHeader *message,
-                               const struct GNUNET_PeerIdentity *peer,
-                               const char *plugin_name,
-                               const char *sender_address, uint16_t sender_address_len,
-                               struct Session *session,
-                               const struct GNUNET_ATS_Information *ats,
-                               uint32_t ats_count)
+                                   const struct GNUNET_PeerIdentity *peer,
+                                   const struct GNUNET_HELLO_Address *address,
+                                   struct Session *session,
+                                   const struct GNUNET_ATS_Information *ats,
+                                   uint32_t ats_count)
 {
   const struct SessionConnectMessage *scm;
-  struct QuotaSetMessage q_msg;
   struct GNUNET_MessageHeader msg;
   struct NeighbourMapEntry *n;
   size_t msg_len;
   size_t ret;
-  int was_connected;
 
-#if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-      "Received CONNECT_ACK message from peer `%s'\n", GNUNET_i2s (peer));
-#endif
+              "Received CONNECT_ACK message from peer `%s'\n",
+              GNUNET_i2s (peer));
 
   if (ntohs (message->size) != sizeof (struct SessionConnectMessage))
   {
     GNUNET_break_op (0);
     return;
   }
-
   scm = (const struct SessionConnectMessage *) message;
   GNUNET_break_op (ntohl (scm->reserved) == 0);
   n = lookup_neighbour (peer);
   if (NULL == n)
-    n = setup_neighbour (peer);
-/*
-  if (n->state != S_CONNECT_SENT)
   {
-    GNUNET_break (0);
-    send_disconnect(&n->id, n->plugin_name, n->addr, n->addrlen, n->session);
+    /* we did not send 'CONNECT' -- at least not recently */
+    GNUNET_STATISTICS_update (GST_stats,
+                              gettext_noop
+                              ("# unexpected CONNECT_ACK messages (no peer)"),
+                              1, GNUNET_NO);
     return;
   }
-*/
+
+  /* Additional check
+   *
+   * ((n->state != S_CONNECT_RECV) && (n->address != NULL)):
+   *
+   * We also received an CONNECT message, switched from SENDT to RECV and
+   * ATS already suggested us an address after a successful blacklist check
+   */
+
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Received CONNECT_ACK message from peer `%s' in state `%s'\n",
+              GNUNET_i2s (peer),
+              print_state(n->state));
+
+  if ((n->address != NULL) && (n->state == S_CONNECTED))
+  {
+    /* After fast reconnect: send ACK (ACK) even when we are connected */
+    msg_len = sizeof (msg);
+    msg.size = htons (msg_len);
+    msg.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_ACK);
+
+    ret = send_with_session(n,
+              (const char *) &msg, msg_len,
+              UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
+              NULL, NULL);
+
+    if (ret == GNUNET_SYSERR)
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                  "Failed to send SESSION_ACK to `%4s' using address '%s' session %X\n",
+                  GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), n->session);
+    return;
+  }
+
+  if ((n->state != S_CONNECT_SENT) &&
+      ((n->state != S_CONNECT_RECV) && (n->address != NULL)))
+  {
+    GNUNET_STATISTICS_update (GST_stats,
+                              gettext_noop
+                              ("# unexpected CONNECT_ACK messages"), 1,
+                              GNUNET_NO);
+    return;
+  }
+  if (n->state != S_CONNECTED)
+    change_state (n, S_CONNECTED);
+
   if (NULL != session)
+  {
     GNUNET_log_from (GNUNET_ERROR_TYPE_DEBUG | GNUNET_ERROR_TYPE_BULK,
                      "transport-ats",
                      "Giving ATS session %p of plugin %s for peer %s\n",
-                     session,
-                     plugin_name,
-                     GNUNET_i2s (peer));
-  GNUNET_ATS_address_update (GST_ats,
-                             peer,
-                             plugin_name, sender_address, sender_address_len,
-                             session, ats, ats_count);
-
-  was_connected = is_connected(n);
-  if (!is_connected(n))
-    change_state (n, S_CONNECTED);
+                     session, address->transport_name, GNUNET_i2s (peer));
+  }
+  GNUNET_ATS_address_update (GST_ats, address, session, ats, ats_count);
+  GNUNET_assert (NULL != n->address);
 
-#if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-              "Setting inbound quota of %u for peer `%s' to \n",
-              ntohl (n->bandwidth_in.value__), GNUNET_i2s (&n->id));
-#endif
-  GST_neighbours_set_incoming_quota(&n->id, n->bandwidth_in);
+  if ((n->address_state == FRESH) && (0 == GNUNET_HELLO_address_cmp(address, n->address)))
+  {
+    GST_validation_set_address_use (n->address, n->session, GNUNET_YES,  __LINE__);
+    GNUNET_ATS_address_in_use (GST_ats, n->address, n->session, GNUNET_YES);
+    n->address_state = USED;
+  }
+
+  GST_neighbours_set_incoming_quota (&n->id, n->bandwidth_in);
 
-  /* send ACK (ACK)*/
-  msg_len =  sizeof (msg);
+  /* send ACK (ACK) */
+  msg_len = sizeof (msg);
   msg.size = htons (msg_len);
   msg.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SESSION_ACK);
 
-  ret = send_with_plugin (&n->id, (const char *) &msg, msg_len, UINT32_MAX,
-                          GNUNET_TIME_UNIT_FOREVER_REL,
-                          n->session, n->plugin_name, n->addr, n->addrlen,
-                          GNUNET_YES, NULL, NULL);
+  ret = send_with_session(n,
+            (const char *) &msg, msg_len,
+            UINT32_MAX, GNUNET_TIME_UNIT_FOREVER_REL,
+            NULL, NULL);
 
   if (ret == GNUNET_SYSERR)
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Failed to send SESSION_ACK to `%4s' using plugin `%s' address '%s' session %X\n",
-              GNUNET_i2s (&n->id), n->plugin_name,
-              (n->addrlen == 0) ? "<inbound>" : GST_plugins_a2s (n->plugin_name,
-                                                                 n->addr,
-                                                                 n->addrlen),
-              n->session);
+                "Failed to send SESSION_ACK to `%4s' using address '%s' session %X\n",
+                GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), n->session);
 
 
-  if (!was_connected)
-  {
-    if (n->keepalive_task == GNUNET_SCHEDULER_NO_TASK)
-      n->keepalive_task = GNUNET_SCHEDULER_add_delayed (KEEPALIVE_FREQUENCY,
-                                                        &neighbour_keepalive_task,
-                                                        n);
-
-    neighbours_connected++;
-    GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# peers connected"), 1,
-                              GNUNET_NO);
-#if DEBUG_TRANSPORT
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Notify about connect of `%4s' using plugin `%s' address '%s' session %X LINE %u\n",
-              GNUNET_i2s (&n->id), n->plugin_name,
-              (n->addrlen == 0) ? "<inbound>" : GST_plugins_a2s (n->plugin_name,
-                                                                 n->addr,
-                                                                 n->addrlen),
-              n->session, __LINE__);
-#endif
-    connect_notify_cb (callback_cls, &n->id, ats, ats_count);
-  }
+  if (n->keepalive_task == GNUNET_SCHEDULER_NO_TASK)
+    n->keepalive_task = GNUNET_SCHEDULER_add_now (&neighbour_keepalive_task, n);
 
+  neighbours_connected++;
+  GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# peers connected"), 1,
+                            GNUNET_NO);
 #if DEBUG_TRANSPORT
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                "Sending outbound quota of %u Bps for peer `%s' to all clients\n",
-                ntohl (n->bandwidth_out.value__), GNUNET_i2s (peer));
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Notify about connect of `%4s' using address '%s' session %X LINE %u\n",
+              GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), n->session,
+              __LINE__);
 #endif
-    q_msg.header.size = htons (sizeof (struct QuotaSetMessage));
-    q_msg.header.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SET_QUOTA);
-    q_msg.quota = n->bandwidth_out;
-    q_msg.peer = (*peer);
-    GST_clients_broadcast (&q_msg.header, GNUNET_NO);
+  connect_notify_cb (callback_cls, &n->id, ats, ats_count);
+  send_outbound_quota (peer, n->bandwidth_out);
 
 }
 
+
 void
 GST_neighbours_handle_ack (const struct GNUNET_MessageHeader *message,
-    const struct GNUNET_PeerIdentity *peer,
-    const char *plugin_name,
-    const char *sender_address, uint16_t sender_address_len,
-    struct Session *session,
-    const struct GNUNET_ATS_Information *ats,
-    uint32_t ats_count)
+                           const struct GNUNET_PeerIdentity *peer,
+                           const struct GNUNET_HELLO_Address *address,
+                           struct Session *session,
+                           const struct GNUNET_ATS_Information *ats,
+                           uint32_t ats_count)
 {
   struct NeighbourMapEntry *n;
-  struct QuotaSetMessage q_msg;
-  int was_connected;
 
 #if DEBUG_TRANSPORT
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-      "Received ACK message from peer `%s'\n", GNUNET_i2s (peer));
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Received ACK message from peer `%s'\n",
+              GNUNET_i2s (peer));
 #endif
 
   if (ntohs (message->size) != sizeof (struct GNUNET_MessageHeader))
@@ -2023,68 +2618,53 @@ GST_neighbours_handle_ack (const struct GNUNET_MessageHeader *message,
     GNUNET_break_op (0);
     return;
   }
-
   n = lookup_neighbour (peer);
   if (NULL == n)
   {
-    send_disconnect(peer, plugin_name, sender_address, sender_address_len, session);
     GNUNET_break (0);
+    return;
   }
-
-  if (is_connected(n))
+  if (S_CONNECTED == n->state)
     return;
-
+  if (!is_connecting (n))
+  {
+    GNUNET_STATISTICS_update (GST_stats,
+                              gettext_noop ("# unexpected ACK messages"), 1,
+                              GNUNET_NO);
+    return;
+  }
+  change_state (n, S_CONNECTED);
   if (NULL != session)
     GNUNET_log_from (GNUNET_ERROR_TYPE_DEBUG | GNUNET_ERROR_TYPE_BULK,
                      "transport-ats",
                      "Giving ATS session %p of plugin %s for peer %s\n",
-                     session,
-                     plugin_name,
-                     GNUNET_i2s (peer));
-  GNUNET_ATS_address_update (GST_ats,
-                             peer,
-                             plugin_name, sender_address, sender_address_len,
-                             session, ats, ats_count);
-
-  was_connected = is_connected(n);
-  change_state (n, S_CONNECTED);
+                     session, address->transport_name, GNUNET_i2s (peer));
+  GNUNET_ATS_address_update (GST_ats, address, session, ats, ats_count);
+  GNUNET_assert (n->address != NULL);
 
-  GST_neighbours_set_incoming_quota(&n->id, n->bandwidth_in);
+  if ((n->address_state == FRESH) && (0 == GNUNET_HELLO_address_cmp(address, n->address)))
+  {
+    GST_validation_set_address_use (n->address, n->session, GNUNET_YES,  __LINE__);
+    GNUNET_ATS_address_in_use (GST_ats, n->address, n->session, GNUNET_YES);
+    n->address_state = USED;
+  }
 
-  if (n->keepalive_task == GNUNET_SCHEDULER_NO_TASK)
-    n->keepalive_task = GNUNET_SCHEDULER_add_delayed (KEEPALIVE_FREQUENCY,
-                                                      &neighbour_keepalive_task,
-                                                      n);
 
-  if (!was_connected)
-  {
   neighbours_connected++;
   GNUNET_STATISTICS_update (GST_stats, gettext_noop ("# peers connected"), 1,
                             GNUNET_NO);
 
-#if DEBUG_TRANSPORT
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Notify about connect of `%4s' using plugin `%s' address '%s' session %X LINE %u\n",
-              GNUNET_i2s (&n->id), n->plugin_name,
-              (n->addrlen == 0) ? "<inbound>" : GST_plugins_a2s (n->plugin_name,
-                                                                 n->addr,
-                                                                 n->addrlen),
-              n->session, __LINE__);
-#endif
-  connect_notify_cb (callback_cls, &n->id, ats, ats_count);
-  }
+  GST_neighbours_set_incoming_quota (&n->id, n->bandwidth_in);
+  if (n->keepalive_task == GNUNET_SCHEDULER_NO_TASK)
+    n->keepalive_task = GNUNET_SCHEDULER_add_now (&neighbour_keepalive_task, n);
 #if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Sending outbound quota of %u Bps for peer `%s' to all clients\n",
-              ntohl (n->bandwidth_out.value__), GNUNET_i2s (peer));
+              "Notify about connect of `%4s' using address '%s' session %X LINE %u\n",
+              GNUNET_i2s (&n->id), GST_plugins_a2s (n->address), n->session,
+              __LINE__);
 #endif
-
-  q_msg.header.size = htons (sizeof (struct QuotaSetMessage));
-  q_msg.header.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_SET_QUOTA);
-  q_msg.quota = n->bandwidth_out;
-  q_msg.peer = (*peer);
-  GST_clients_broadcast (&q_msg.header, GNUNET_NO);
-
+  connect_notify_cb (callback_cls, &n->id, ats, ats_count);
+  send_outbound_quota (peer, n->bandwidth_out);
 }
 
 struct BlackListCheckContext
@@ -2095,11 +2675,7 @@ struct BlackListCheckContext
 
   struct Session *session;
 
-  char *sender_address;
-
-  uint16_t sender_address_len;
-
-  char *plugin_name;
+  struct GNUNET_HELLO_Address *address;
 
   struct GNUNET_TIME_Absolute ts;
 };
@@ -2107,20 +2683,23 @@ struct BlackListCheckContext
 
 static void
 handle_connect_blacklist_cont (void *cls,
-                               const struct GNUNET_PeerIdentity
-                               * peer, int result)
+                               const struct GNUNET_PeerIdentity *peer,
+                               int result)
 {
   struct NeighbourMapEntry *n;
-  struct BlackListCheckContext * bcc = cls;
+  struct BlackListCheckContext *bcc = cls;
 
 #if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-      "Blacklist check due to CONNECT message: `%s'\n", GNUNET_i2s (peer), (result == GNUNET_OK) ? "ALLOWED" : "FORBIDDEN");
+              "Blacklist check due to CONNECT message: `%s'\n",
+              GNUNET_i2s (peer),
+              (result == GNUNET_OK) ? "ALLOWED" : "FORBIDDEN");
 #endif
 
   /* not allowed */
   if (GNUNET_OK != result)
   {
+    GNUNET_HELLO_address_free (bcc->address);
     GNUNET_free (bcc);
     return;
   }
@@ -2134,29 +2713,30 @@ handle_connect_blacklist_cont (void *cls,
     if (NULL != bcc->session)
       GNUNET_log_from (GNUNET_ERROR_TYPE_DEBUG | GNUNET_ERROR_TYPE_BULK,
                        "transport-ats",
-                       "Giving ATS session %p of plugin %s address `%s' for peer %s\n",
-                       bcc->session,
-                       bcc->plugin_name,
-                       GST_plugins_a2s (bcc->plugin_name, bcc->sender_address, bcc->sender_address_len),
+                       "Giving ATS session %p of address `%s' for peer %s\n",
+                       bcc->session, GST_plugins_a2s (bcc->address),
                        GNUNET_i2s (peer));
-    GNUNET_ATS_address_update (GST_ats,
-                               peer,
-                               bcc->plugin_name, bcc->sender_address, bcc->sender_address_len,
-                               bcc->session, bcc->ats, bcc->ats_count);
+    /* Tell ATS about the session, so ATS can suggest it if it likes it. */
+
+    GNUNET_ATS_address_update (GST_ats, bcc->address, bcc->session, bcc->ats,
+                               bcc->ats_count);
     n->connect_ts = bcc->ts;
   }
 
+  GNUNET_HELLO_address_free (bcc->address);
   GNUNET_free (bcc);
-/*
-  if (n->state != S_NOT_CONNECTED)
-    return;*/
-  change_state (n, S_CONNECT_RECV);
+
+  if (n->state != S_CONNECT_RECV)
+    change_state (n, S_CONNECT_RECV);
+
 
   /* Ask ATS for an address to connect via that address */
   if (n->ats_suggest != GNUNET_SCHEDULER_NO_TASK)
-    GNUNET_SCHEDULER_cancel(n->ats_suggest);
-  n->ats_suggest = GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel, n);
-  GNUNET_ATS_suggest_address(GST_ats, peer);
+    GNUNET_SCHEDULER_cancel (n->ats_suggest);
+  n->ats_suggest =
+      GNUNET_SCHEDULER_add_delayed (ATS_RESPONSE_TIMEOUT, ats_suggest_cancel,
+                                    n);
+  GNUNET_ATS_suggest_address (GST_ats, peer);
 }
 
 /**
@@ -2165,30 +2745,27 @@ handle_connect_blacklist_cont (void *cls,
  *
  * @param message possibly a 'struct SessionConnectMessage' (check format)
  * @param peer identity of the peer to switch the address for
- * @param plugin_name name of transport that delivered the PONG
  * @param address address of the other peer, NULL if other peer
  *                       connected to us
- * @param address_len number of bytes in address
  * @param session session to use (or NULL)
  * @param ats performance data
  * @param ats_count number of entries in ats (excluding 0-termination)
 */
+ */
 void
 GST_neighbours_handle_connect (const struct GNUNET_MessageHeader *message,
-                              const struct GNUNET_PeerIdentity *peer,
-                              const char *plugin_name,
-                              const char *sender_address, uint16_t sender_address_len,
-                              struct Session *session,
-                              const struct GNUNET_ATS_Information *ats,
-                              uint32_t ats_count)
+                               const struct GNUNET_PeerIdentity *peer,
+                               const struct GNUNET_HELLO_Address *address,
+                               struct Session *session,
+                               const struct GNUNET_ATS_Information *ats,
+                               uint32_t ats_count)
 {
   const struct SessionConnectMessage *scm;
-  struct NeighbourMapEntry * n;
-  struct BlackListCheckContext * bcc = NULL;
+  struct BlackListCheckContext *bcc = NULL;
+  struct NeighbourMapEntry *n;
 
 #if DEBUG_TRANSPORT
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-      "Received CONNECT message from peer `%s'\n", GNUNET_i2s (peer));
+              "Received CONNECT message from peer `%s'\n", GNUNET_i2s (peer));
 #endif
 
   if (ntohs (message->size) != sizeof (struct SessionConnectMessage))
@@ -2200,40 +2777,32 @@ GST_neighbours_handle_connect (const struct GNUNET_MessageHeader *message,
   scm = (const struct SessionConnectMessage *) message;
   GNUNET_break_op (ntohl (scm->reserved) == 0);
 
-  n = lookup_neighbour(peer);
-  if (n != NULL)
+  GNUNET_ATS_address_update (GST_ats, address, session, ats, ats_count);
+
+  n = lookup_neighbour (peer);
+  if ((n != NULL) && ((S_CONNECTED == n->state) || (S_FAST_RECONNECT == n->state)))
   {
-    /* connected peer switches addresses */
-    if (is_connected(n))
-    {
-      GNUNET_ATS_address_update(GST_ats, peer, plugin_name, sender_address, sender_address_len, session, ats, ats_count);
-      return;
-    }
+    /* connected peer switches addresses or is trying to do a fast reconnect*/
+    return;
   }
 
-  /* we are not connected to this peer */
-  /* do blacklist check*/
-  bcc = GNUNET_malloc (sizeof (struct BlackListCheckContext) +
-            sizeof (struct GNUNET_ATS_Information) * ats_count +
-            sender_address_len +
-            strlen (plugin_name)+1);
 
+  /* we are not connected to this peer */
+  /* do blacklist check */
+  bcc =
+      GNUNET_malloc (sizeof (struct BlackListCheckContext) +
+                     sizeof (struct GNUNET_ATS_Information) * (ats_count + 1));
   bcc->ts = GNUNET_TIME_absolute_ntoh (scm->timestamp);
-
-  bcc->ats_count = ats_count;
-  bcc->sender_address_len = sender_address_len;
+  bcc->ats_count = ats_count + 1;
+  bcc->address = GNUNET_HELLO_address_copy (address);
   bcc->session = session;
-
   bcc->ats = (struct GNUNET_ATS_Information *) &bcc[1];
-  memcpy (bcc->ats, ats,sizeof (struct GNUNET_ATS_Information) * ats_count );
-
-  bcc->sender_address = (char *) &bcc->ats[ats_count];
-  memcpy (bcc->sender_address, sender_address , sender_address_len);
-
-  bcc->plugin_name = &bcc->sender_address[sender_address_len];
-  strcpy (bcc->plugin_name, plugin_name);
-
-  GST_blacklist_test_allowed (peer, plugin_name, handle_connect_blacklist_cont, bcc);
+  memcpy (bcc->ats, ats, sizeof (struct GNUNET_ATS_Information) * ats_count);
+  bcc->ats[ats_count].type = htonl (GNUNET_ATS_QUALITY_NET_DELAY);
+  bcc->ats[ats_count].value =
+      htonl ((uint32_t) GST_neighbour_get_latency (peer).rel_value);
+  GST_blacklist_test_allowed (peer, address->transport_name,
+                              &handle_connect_blacklist_cont, bcc);
 }