-fixed duplicate ntfy_tmt_rdy
[oweals/gnunet.git] / src / rps / gnunet-service-rps.c
index b5333ae282a1fb09b6166d2110902d33bcad2d55..fd398b863c02fd60f1c8c18ad18850a1387a92eb 100644 (file)
@@ -14,8 +14,8 @@
 
      You should have received a copy of the GNU General Public License
      along with GNUnet; see the file COPYING.  If not, write to the
-     Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-     Boston, MA 02111-1307, USA.
+     Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+     Boston, MA 02110-1301, USA.
 */
 
 /**
 #include "platform.h"
 #include "gnunet_util_lib.h"
 #include "gnunet_cadet_service.h"
+#include "gnunet_peerinfo_service.h"
 #include "gnunet_nse_service.h"
 #include "rps.h"
+#include "rps-test_util.h"
 
 #include "gnunet-service-rps_sampler.h"
 
 
 // TODO align message structs
 
-// (TODO api -- possibility of getting weak random peer immideately)
+// TODO connect to friends
 
-// TODO malicious peer
+// TODO store peers somewhere persistent
 
-// TODO connect to friends
+// TODO blacklist? (-> mal peer detection on top of brahms)
 
-// TODO store peers somewhere
+// TODO API request_cancel
 
-// TODO ignore list?
+// TODO don't pass peermap all the time
 
 // hist_size_init, hist_size_max
 
@@ -77,7 +79,7 @@ get_rand_peer_ignore_list (const struct GNUNET_PeerIdentity *peer_list, unsigned
 /**
  * Struct used to store the context of a connected client.
  */
-struct client_ctx
+struct ClientContext
 {
   /**
    * The message queue to communicate with the client.
@@ -98,7 +100,14 @@ enum PeerFlags
   /**
    * We set this bit when we can be sure the other peer is/was live.
    */
-  VALID                = 0x10
+  VALID                = 0x10,
+
+  /**
+   * We set this bit when we are going to destroy the channel to this peer.
+   * When cleanup_channel is called, we know that we wanted to destroy it.
+   * Otherwise the channel to the other peer was destroyed.
+   */
+  TO_DESTROY           = 0x20,
 };
 
 
@@ -132,7 +141,7 @@ struct PeerOutstandingOp
 struct PeerContext
 {
   /**
-   * In own gossip/sampler list, in other's gossip/sampler list
+   * Flags indicating status of peer
    */
   uint32_t peer_flags;
 
@@ -210,15 +219,22 @@ static struct GNUNET_CONTAINER_MultiPeerMap *peer_map;
 
 
 /**
- * The gossiped list of peers.
+ * Name to log view to
  */
-static struct GNUNET_PeerIdentity *gossip_list;
+static char *file_name_view_log;
 
 /**
- * Size of the gossiped list
+ * The "local view" containing peers we learned from gossip and history
  */
-//static unsigned int gossip_list_size;
-static uint32_t gossip_list_size;
+static struct GNUNET_CONTAINER_MultiPeerMap *view;
+
+/**
+ * An array containing the peers of the local view.
+ *
+ * This is created every time we send a pull reply if it has changed since the
+ * last pull reply we sent.
+ */
+static struct GNUNET_PeerIdentity *view_array;
 
 
 /**
@@ -231,22 +247,19 @@ static unsigned int sampler_size_client_need;
  * The size of sampler we need to be able to satisfy the Brahms protocol's
  * need of random peers.
  *
- * This is directly taken as the #gossip_list_size on update of the
- * #gossip_list
- *
  * This is one minimum size the sampler grows to.
  */
 static unsigned int sampler_size_est_need;
 
 
 /**
- * Percentage of total peer number in the gossip list
+ * Percentage of total peer number in the view
  * to send random PUSHes to
  */
 static float alpha;
 
 /**
- * Percentage of total peer number in the gossip list
+ * Percentage of total peer number in the view
  * to send random PULLs to
  */
 static float beta;
@@ -306,6 +319,16 @@ static struct GNUNET_NSE_Handle *nse;
  */
 static struct GNUNET_CADET_Handle *cadet_handle;
 
+/**
+ * Handler to PEERINFO.
+ */
+static struct GNUNET_PEERINFO_Handle *peerinfo_handle;
+
+/**
+ * Handle for cancellation of iteration over peers.
+ */
+struct GNUNET_PEERINFO_NotifyContext *peerinfo_notify_handle;
+
 
 /**
  * Request counter.
@@ -340,20 +363,27 @@ static struct GNUNET_TIME_Relative  request_rate;
 
 
 /**
- * List with the peers we sent requests to.
- */
-struct GNUNET_PeerIdentity *pending_pull_reply_list;
-
-/**
- * Size of #pending_pull_reply_list.
+ * Number of history update tasks.
  */
-uint32_t pending_pull_reply_list_size;
+uint32_t num_hist_update_tasks;
 
 
 /**
- * Number of history update tasks.
+ * Closure used to pass the client and the id to the callback
+ * that replies to a client's request
  */
-uint32_t num_hist_update_tasks;
+struct ReplyCls
+{
+  /**
+   * The identifier of the request
+   */
+  uint32_t id;
+
+  /**
+   * The client handle to send the reply to
+   */
+  struct GNUNET_SERVER_Client *client;
+};
 
 
 #ifdef ENABLE_MALICIOUS
@@ -363,6 +393,7 @@ uint32_t num_hist_update_tasks;
  * 0 Don't act malicious at all - Default
  * 1 Try to maximise representation
  * 2 Try to partition the network
+ * 3 Combined attack
  */
 uint32_t mal_type = 0;
 
@@ -371,6 +402,12 @@ uint32_t mal_type = 0;
  */
 static struct GNUNET_PeerIdentity *mal_peers = NULL;
 
+/**
+ * Hashmap of malicious peers used as set.
+ * Used to more efficiently check whether we know that peer.
+ */
+static struct GNUNET_CONTAINER_MultiPeerMap *mal_peer_set = NULL;
+
 /**
  * Number of other malicious peers
  */
@@ -391,19 +428,31 @@ struct AttackedPeer
   /**
    * PeerID
    */
-  struct GNUNET_PeerIdentity *peer_id;
+  struct GNUNET_PeerIdentity peer_id;
 };
 
 /**
  * If type is 2 this is the DLL of attacked peers
  */
-//static struct AttackedPeer *att_peers_head = NULL;
-//static struct AttackedPeer *att_peers_tail = NULL;
+static struct AttackedPeer *att_peers_head = NULL;
+static struct AttackedPeer *att_peers_tail = NULL;
+
+/**
+ * This index is used to point to an attacked peer to
+ * implement the round-robin-ish way to select attacked peers.
+ */
+static struct AttackedPeer *att_peer_index = NULL;
+
+/**
+ * Hashmap of attacked peers used as set.
+ * Used to more efficiently check whether we know that peer.
+ */
+static struct GNUNET_CONTAINER_MultiPeerMap *att_peer_set = NULL;
 
 /**
  * Number of attacked peers
  */
-//static uint32_t num_attacked_peers = 0;
+static uint32_t num_attacked_peers = 0;
 
 
 /**
@@ -450,11 +499,6 @@ static uint32_t push_limit = 10000;
  */
 #define unset_peer_flag(peer_ctx, mask) (peer_ctx->peer_flags &= (~mask))
 
-/**
- * Compute the minimum of two ints
- */
-#define min(x, y) ((x < y) ? x : y)
-
 /**
  * Clean the send channel of a peer
  */
@@ -569,7 +613,7 @@ get_rand_peer_ignore_list (const struct GNUNET_PeerIdentity *peer_list,
 
   /**;
    * Choose the r_index of the peer we want to return
-   * at random from the interval of the gossip list
+   * at random from the interval of the view
    */
   r_index = GNUNET_CRYPTO_random_u64 (GNUNET_CRYPTO_QUALITY_STRONG,
                                       tmp_size);
@@ -589,7 +633,7 @@ get_rand_peer_ignore_list (const struct GNUNET_PeerIdentity *peer_list,
 
     /**;
      * Choose the r_index of the peer we want to return
-     * at random from the interval of the gossip list
+     * at random from the interval of the view
      */
     r_index = GNUNET_CRYPTO_random_u64 (GNUNET_CRYPTO_QUALITY_STRONG,
                                         tmp_size);
@@ -635,15 +679,37 @@ get_peer_ctx (struct GNUNET_CONTAINER_MultiPeerMap *peer_map,
 
 
 /**
- * Put random peer from sampler into the gossip list as history update.
+ * Put random peer from sampler into the view as history update.
  */
   void
 hist_update (void *cls, struct GNUNET_PeerIdentity *ids, uint32_t num_peers)
 {
-  GNUNET_assert (1 == num_peers);
+  unsigned int i;
 
-  if (gossip_list_size < sampler_size_est_need)
-    GNUNET_array_append (gossip_list, gossip_list_size, *ids);
+  for (i = 0; i < GNUNET_MIN (
+       sampler_size_est_need - GNUNET_CONTAINER_multipeermap_size (view),
+       num_peers); i++)
+  {
+    if (GNUNET_OK != GNUNET_CONTAINER_multipeermap_put (view,
+          &ids[i],
+          NULL,
+          GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_FAST))
+        {
+          LOG (GNUNET_ERROR_TYPE_WARNING,
+               "Failed to put peer in peermap. (hist_update)\n");
+        }
+
+    /* Might want to check that we really updated the view */
+    if (NULL != view_array)
+    {
+      GNUNET_free (view_array);
+      view_array = NULL;
+    }
+
+    to_file (file_name_view_log,
+             "+%s\t(hist)",
+             GNUNET_i2s_full (ids));
+  }
 
   if (0 < num_hist_update_tasks)
     num_hist_update_tasks--;
@@ -734,6 +800,7 @@ get_channel (struct GNUNET_CONTAINER_MultiPeerMap *peer_map,
   struct PeerContext *peer_ctx;
 
   peer_ctx = get_peer_ctx (peer_map, peer);
+
   if (NULL == peer_ctx->send_channel)
   {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
@@ -747,12 +814,6 @@ get_channel (struct GNUNET_CONTAINER_MultiPeerMap *peer_map,
                                    GNUNET_RPS_CADET_PORT,
                                    GNUNET_CADET_OPTION_RELIABLE);
 
-    // do I have to explicitly put it in the peer_map?
-    (void) GNUNET_CONTAINER_multipeermap_put
-      (peer_map,
-       peer,
-       peer_ctx,
-       GNUNET_CONTAINER_MULTIHASHMAPOPTION_REPLACE);
   }
   return peer_ctx->send_channel;
 }
@@ -772,6 +833,8 @@ get_mq (struct GNUNET_CONTAINER_MultiPeerMap *peer_map,
 
   peer_ctx = get_peer_ctx (peer_map, peer_id);
 
+  GNUNET_assert (NULL == peer_ctx->is_live_task);
+
   if (NULL == peer_ctx->mq)
   {
     (void) get_channel (peer_map, peer_id);
@@ -792,12 +855,14 @@ get_mq (struct GNUNET_CONTAINER_MultiPeerMap *peer_map,
 void
 check_peer_live (struct PeerContext *peer_ctx)
 {
-  (void) get_channel (peer_map, &peer_ctx->peer_id);
   LOG (GNUNET_ERROR_TYPE_DEBUG,
        "Get informed about peer %s getting live\n",
        GNUNET_i2s (&peer_ctx->peer_id));
-  if (NULL == peer_ctx->is_live_task)
+
+  if (NULL == peer_ctx->is_live_task &&
+      NULL == peer_ctx->send_channel)
   {
+    (void) get_channel (peer_map, &peer_ctx->peer_id);
     peer_ctx->is_live_task =
         GNUNET_CADET_notify_transmit_ready (peer_ctx->send_channel,
                                             GNUNET_NO,
@@ -806,11 +871,12 @@ check_peer_live (struct PeerContext *peer_ctx)
                                             cadet_ntfy_tmt_rdy_cb,
                                             peer_ctx);
   }
-  else
-  {
+  else if (NULL != peer_ctx->is_live_task)
     LOG (GNUNET_ERROR_TYPE_DEBUG,
          "Already waiting for notification\n");
-  }
+  else if (NULL != peer_ctx->send_channel)
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Already have established channel to peer\n");
 }
 
 
@@ -852,8 +918,6 @@ insert_in_pull_list (void *cls, const struct GNUNET_PeerIdentity *peer)
 {
   if (GNUNET_NO == in_arr (pull_list, pull_list_size, peer))
     GNUNET_array_append (pull_list, pull_list_size, *peer);
-
-  peer_clean (peer);
 }
 
 /**
@@ -872,29 +936,42 @@ insert_in_pull_list_scheduled (const struct PeerContext *peer_ctx)
 
 
 /**
- * Insert PeerID in #gossip_list
+ * Insert PeerID in #view
  *
  * Called once we know a peer is live.
  */
   void
-insert_in_gossip_list (void *cls, const struct GNUNET_PeerIdentity *peer)
+insert_in_view (void *cls, const struct GNUNET_PeerIdentity *peer)
 {
-  if (GNUNET_NO == in_arr (gossip_list, gossip_list_size, peer))
-    GNUNET_array_append (gossip_list, gossip_list_size, *peer);
+  if (GNUNET_YES != GNUNET_CONTAINER_multipeermap_put (view,
+        peer,
+        NULL,
+        GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_FAST))
+  {
+    LOG (GNUNET_ERROR_TYPE_WARNING,
+        "Failed to put peer into view. (insert_in_view)\n");
+  }
+
+  /* Might want to check whether we really modified the view */
+  if (NULL != view_array)
+  {
+    GNUNET_free (view_array);
+    view_array = NULL;
+  }
 
   (void) get_channel (peer_map, peer);
 }
 
 /**
- * Check whether #insert_in_gossip_list was already scheduled
+ * Check whether #insert_in_view was already scheduled
  */
   int
-insert_in_gossip_list_scheduled (const struct PeerContext *peer_ctx)
+insert_in_view_scheduled (const struct PeerContext *peer_ctx)
 {
   unsigned int i;
 
   for ( i = 0 ; i < peer_ctx->num_outstanding_ops ; i++ )
-    if (insert_in_gossip_list == peer_ctx->outstanding_ops[i].op)
+    if (insert_in_view == peer_ctx->outstanding_ops[i].op)
       return GNUNET_YES;
   return GNUNET_NO;
 }
@@ -1021,6 +1098,138 @@ est_request_rate()
 }
 
 
+/**
+ * Add all peers in @a peer_array to @a peer_map used as set.
+ *
+ * @param peer_array array containing the peers
+ * @param num_peers number of peers in @peer_array
+ * @param peer_map the peermap to use as set
+ */
+static void
+add_peer_array_to_set (const struct GNUNET_PeerIdentity *peer_array,
+                       unsigned int num_peers,
+                       struct GNUNET_CONTAINER_MultiPeerMap *peer_map)
+{
+  unsigned int i;
+  if (NULL == peer_map)
+  {
+    LOG (GNUNET_ERROR_TYPE_WARNING,
+         "Trying to add peers to an empty peermap.\n");
+    return;
+  }
+
+  for (i = 0; i < num_peers; i++)
+  {
+    GNUNET_CONTAINER_multipeermap_put (peer_map,
+                                       &peer_array[i],
+                                       NULL,
+                                       GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_FAST);
+  }
+}
+
+
+/**
+ * Send a PULL REPLY to @a peer_id
+ *
+ * @param peer_id the peer to send the reply to.
+ * @param peer_ids the peers to send to @a peer_id
+ * @param num_peer_ids the number of peers to send to @a peer_id
+ */
+static void
+send_pull_reply (const struct GNUNET_PeerIdentity *peer_id,
+                 const struct GNUNET_PeerIdentity *peer_ids,
+                 unsigned int num_peer_ids)
+{
+  uint32_t send_size;
+  struct GNUNET_MQ_Handle *mq;
+  struct GNUNET_MQ_Envelope *ev;
+  struct GNUNET_RPS_P2P_PullReplyMessage *out_msg;
+
+  /* Compute actual size */
+  send_size = sizeof (struct GNUNET_RPS_P2P_PullReplyMessage) +
+              num_peer_ids * sizeof (struct GNUNET_PeerIdentity);
+
+  if (GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE < send_size)
+    /* Compute number of peers to send
+     * If too long, simply truncate */
+    // TODO select random ones via permutation
+    //      or even better: do good protocol design
+    send_size =
+      (GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE -
+       sizeof (struct GNUNET_RPS_P2P_PullReplyMessage)) /
+       sizeof (struct GNUNET_PeerIdentity);
+  else
+    send_size = num_peer_ids;
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+      "PULL REQUEST from peer %s received, going to send %u peers\n",
+      GNUNET_i2s (peer_id), send_size);
+
+  mq = get_mq (peer_map, peer_id);
+
+  ev = GNUNET_MQ_msg_extra (out_msg,
+                            send_size * sizeof (struct GNUNET_PeerIdentity),
+                            GNUNET_MESSAGE_TYPE_RPS_PP_PULL_REPLY);
+  out_msg->num_peers = htonl (send_size);
+  memcpy (&out_msg[1], peer_ids,
+         send_size * sizeof (struct GNUNET_PeerIdentity));
+
+  GNUNET_MQ_send (mq, ev);
+}
+
+
+/**
+ * This function is called on new peer_ids from 'external' sources
+ * (client seed, cadet get_peers(), ...)
+ *
+ * @param peer_id the new peer_id
+ */
+static void
+new_peer_id (const struct GNUNET_PeerIdentity *peer_id)
+{
+  struct PeerOutstandingOp out_op;
+  struct PeerContext *peer_ctx;
+
+  if (NULL != peer_id &&
+      0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity, peer_id))
+  {
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+        "Got peer_id %s (at %p, view size: %u)\n",
+        GNUNET_i2s (peer_id),
+        peer_id,
+        GNUNET_CONTAINER_multipeermap_size (view));
+
+    peer_ctx = get_peer_ctx (peer_map, peer_id);
+    if (GNUNET_YES != get_peer_flag (peer_ctx, VALID))
+    {
+      if (GNUNET_NO == insert_in_sampler_scheduled (peer_ctx))
+      {
+        out_op.op = insert_in_sampler;
+        out_op.op_cls = NULL;
+        GNUNET_array_append (peer_ctx->outstanding_ops,
+                             peer_ctx->num_outstanding_ops,
+                             out_op);
+      }
+
+      if (GNUNET_NO == insert_in_view_scheduled (peer_ctx))
+      {
+        out_op.op = insert_in_view;
+        out_op.op_cls = NULL;
+        GNUNET_array_append (peer_ctx->outstanding_ops,
+                             peer_ctx->num_outstanding_ops,
+                             out_op);
+      }
+
+      /* Trigger livelyness test on peer */
+      check_peer_live (peer_ctx);
+    }
+    // else...?
+
+    // send push/pull to each of those peers?
+  }
+}
+
+
 /***********************************************************************
  * /Util functions
 ***********************************************************************/
@@ -1032,7 +1241,7 @@ est_request_rate()
 /**
  * Function called by NSE.
  *
- * Updates sizes of sampler list and gossip list and adapt those lists
+ * Updates sizes of sampler list and view and adapt those lists
  * accordingly.
  */
   void
@@ -1070,16 +1279,17 @@ nse_callback (void *cls, struct GNUNET_TIME_Absolute timestamp,
  * Sends those to the requesting client.
  */
 void client_respond (void *cls,
-    struct GNUNET_PeerIdentity *ids, uint32_t num_peers)
+    struct GNUNET_PeerIdentity *peer_ids, uint32_t num_peers)
 {
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "sampler returned %" PRIX32 " peers\n", num_peers);
   struct GNUNET_MQ_Envelope *ev;
   struct GNUNET_RPS_CS_ReplyMessage *out_msg;
-  struct GNUNET_SERVER_Client *client;
+  struct ReplyCls *reply_cls = (struct ReplyCls *) cls;
   uint32_t size_needed;
-  struct client_ctx *cli_ctx;
+  struct ClientContext *cli_ctx;
 
-  client = (struct GNUNET_SERVER_Client *) cls;
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "sampler returned %" PRIu32 " peers\n",
+       num_peers);
 
   size_needed = sizeof (struct GNUNET_RPS_CS_ReplyMessage) +
                 num_peers * sizeof (struct GNUNET_PeerIdentity);
@@ -1090,19 +1300,22 @@ void client_respond (void *cls,
                             num_peers * sizeof (struct GNUNET_PeerIdentity),
                             GNUNET_MESSAGE_TYPE_RPS_CS_REPLY);
   out_msg->num_peers = htonl (num_peers);
+  out_msg->id = htonl (reply_cls->id);
 
   memcpy (&out_msg[1],
-      ids,
-      num_peers * sizeof (struct GNUNET_PeerIdentity));
-  GNUNET_free (ids);
+          peer_ids,
+          num_peers * sizeof (struct GNUNET_PeerIdentity));
+  GNUNET_free (peer_ids);
 
-  cli_ctx = GNUNET_SERVER_client_get_user_context (client, struct client_ctx);
+  cli_ctx = GNUNET_SERVER_client_get_user_context (reply_cls->client, struct ClientContext);
   if (NULL == cli_ctx) {
-    cli_ctx = GNUNET_new (struct client_ctx);
-    cli_ctx->mq = GNUNET_MQ_queue_for_server_client (client);
-    GNUNET_SERVER_client_set_user_context (client, cli_ctx);
+    cli_ctx = GNUNET_new (struct ClientContext);
+    cli_ctx->mq = GNUNET_MQ_queue_for_server_client (reply_cls->client);
+    GNUNET_SERVER_client_set_user_context (reply_cls->client, cli_ctx);
   }
 
+  GNUNET_free (reply_cls);
+
   GNUNET_MQ_send (cli_ctx->mq, ev);
 }
 
@@ -1116,12 +1329,13 @@ void client_respond (void *cls,
  */
 static void
 handle_client_request (void *cls,
-            struct GNUNET_SERVER_Client *client,
-            const struct GNUNET_MessageHeader *message)
+                       struct GNUNET_SERVER_Client *client,
+                       const struct GNUNET_MessageHeader *message)
 {
   struct GNUNET_RPS_CS_RequestMessage *msg;
   uint32_t num_peers;
   uint32_t size_needed;
+  struct ReplyCls *reply_cls;
   uint32_t i;
 
   msg = (struct GNUNET_RPS_CS_RequestMessage *) message;
@@ -1139,10 +1353,18 @@ handle_client_request (void *cls,
   for (i = 0 ; i < num_peers ; i++)
     est_request_rate();
 
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "Client requested %" PRIX32 " random peer(s).\n", num_peers);
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Client requested %" PRIu32 " random peer(s).\n",
+       num_peers);
+
+  reply_cls = GNUNET_new (struct ReplyCls);
+  reply_cls->id = ntohl (msg->id);
+  reply_cls->client = client;
 
-  RPS_sampler_get_n_rand_peers (client_sampler, client_respond,
-                                client, num_peers, GNUNET_YES);
+  RPS_sampler_get_n_rand_peers (client_sampler,
+                                client_respond,
+                                reply_cls,
+                                num_peers);
 
   GNUNET_SERVER_receive_done (client,
                              GNUNET_OK);
@@ -1191,20 +1413,20 @@ handle_client_seed (void *cls,
        "Client seeded peers:\n");
   print_peer_list (peers, num_peers);
 
-  // TODO check for validity of ids
-
   for (i = 0 ; i < num_peers ; i++)
   {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
-         "Updating samplers with seed %" PRIX32 ": %s\n",
+         "Updating samplers with seed %" PRIu32 ": %s\n",
          i,
          GNUNET_i2s (&peers[i]));
 
-    RPS_sampler_update (prot_sampler,   &peers[i]);
-    RPS_sampler_update (client_sampler, &peers[i]);
+    new_peer_id (&peers[i]);
+
+    //RPS_sampler_update (prot_sampler,   &peers[i]);
+    //RPS_sampler_update (client_sampler, &peers[i]);
   }
 
-  //GNUNET_free (peers);
+  ////GNUNET_free (peers);
 
   GNUNET_SERVER_receive_done (client,
                                                GNUNET_OK);
@@ -1232,10 +1454,45 @@ handle_peer_push (void *cls,
 
   // (check the proof of work)
 
-  peer = (const struct GNUNET_PeerIdentity *) GNUNET_CADET_channel_get_info (channel, GNUNET_CADET_OPTION_PEER);
+  peer = (const struct GNUNET_PeerIdentity *)
+    GNUNET_CADET_channel_get_info (channel, GNUNET_CADET_OPTION_PEER);
   // FIXME wait for cadet to change this function
+
   LOG (GNUNET_ERROR_TYPE_DEBUG, "PUSH received (%s)\n", GNUNET_i2s (peer));
 
+  #ifdef ENABLE_MALICIOUS
+  struct AttackedPeer *tmp_att_peer;
+
+  tmp_att_peer = GNUNET_new (struct AttackedPeer);
+  memcpy (&tmp_att_peer->peer_id, peer, sizeof (struct GNUNET_PeerIdentity));
+  if (1 == mal_type
+      || 3 == mal_type)
+  { /* Try to maximise representation */
+    if (NULL == att_peer_set)
+      att_peer_set = GNUNET_CONTAINER_multipeermap_create (1, GNUNET_NO);
+    if (GNUNET_NO == GNUNET_CONTAINER_multipeermap_contains (att_peer_set,
+                                                             peer))
+    {
+      GNUNET_CONTAINER_DLL_insert (att_peers_head,
+                                   att_peers_tail,
+                                   tmp_att_peer);
+      add_peer_array_to_set (peer, 1, att_peer_set);
+    }
+    return GNUNET_OK;
+  }
+
+
+  else if (2 == mal_type)
+  { /* We attack one single well-known peer - simply ignore */
+    return GNUNET_OK;
+  }
+  else
+  {
+    GNUNET_free (tmp_att_peer);
+  }
+
+  #endif /* ENABLE_MALICIOUS */
+
   /* Add the sending peer to the push_list */
   if (GNUNET_NO == in_arr (push_list, push_list_size, peer))
     GNUNET_array_append (push_list, push_list_size, *peer);
@@ -1243,13 +1500,66 @@ handle_peer_push (void *cls,
   return GNUNET_OK;
 }
 
+
+/**
+ * Iterator over hash map entries.
+ * Called from #generate_view_array and writes every peer id into #view_array.
+ *
+ * @param cls closure - the pointer to the counter
+ * @param key current public key
+ * @param value value in the hash map
+ * @return #GNUNET_YES if we should continue to
+ * iterate,
+ * #GNUNET_NO if not.
+ */
+static int
+dump_id_to_view_array (void *cls,
+                       const struct GNUNET_PeerIdentity *key,
+                       void *value)
+{
+  unsigned int *i = (unsigned int *) cls;
+
+  view_array[(*i)++] = *key;
+  return GNUNET_YES;
+}
+
+
+/**
+ * Makes sure the view_array is filled with the peer ids currently in #view.
+ * Called from within #do_round before sending pushes and pulls and from
+ * #handle_peer_pull_request when a reply is sent.
+ */
+static void
+generate_view_array (unsigned int view_size)
+{
+  unsigned int *i;
+  int ret;
+
+  if (NULL == view_array)
+  {
+    view_array = GNUNET_new_array (view_size,
+                                   struct GNUNET_PeerIdentity);
+    i = GNUNET_new (unsigned int);
+    *i = 0;
+
+    ret = GNUNET_CONTAINER_multipeermap_iterate (view,
+                                                 dump_id_to_view_array,
+                                                 i);
+    GNUNET_assert (view_size == ret);
+    GNUNET_assert (view_size == *i);
+
+    GNUNET_free (i);
+  }
+}
+
+
 /**
  * Handle PULL REQUEST request message from another peer.
  *
- * Reply with the gossip list of PeerIDs.
+ * Reply with the view of PeerIDs.
  *
  * @param cls Closure
- * @param channel The channel the PUSH was received over
+ * @param channel The channel the PULL REQUEST was received over
  * @param channel_ctx The context associated with this channel
  * @param msg The message header
  */
@@ -1260,47 +1570,37 @@ handle_peer_pull_request (void *cls,
     const struct GNUNET_MessageHeader *msg)
 {
   struct GNUNET_PeerIdentity *peer;
-  uint32_t send_size;
-  struct GNUNET_MQ_Handle *mq;
-  struct GNUNET_MQ_Envelope *ev;
-  struct GNUNET_RPS_P2P_PullReplyMessage *out_msg;
-
+  unsigned int view_size;
 
   peer = (struct GNUNET_PeerIdentity *)
     GNUNET_CADET_channel_get_info (channel,
                                    GNUNET_CADET_OPTION_PEER);
   // FIXME wait for cadet to change this function
 
-  /* Compute actual size */
-  send_size = sizeof (struct GNUNET_RPS_P2P_PullReplyMessage) +
-              gossip_list_size * sizeof (struct GNUNET_PeerIdentity);
+  LOG (GNUNET_ERROR_TYPE_DEBUG, "PULL REQUEST received (%s)\n", GNUNET_i2s (peer));
 
-  if (GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE < send_size)
-    /* Compute number of peers to send
-     * If too long, simply truncate */
-  // TODO select random ones via permutation
-    send_size =
-      (GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE -
-       sizeof (struct GNUNET_RPS_P2P_PullReplyMessage)) /
-       sizeof (struct GNUNET_PeerIdentity);
-  else
-    send_size = gossip_list_size;
-
-  LOG (GNUNET_ERROR_TYPE_DEBUG,
-      "PULL REQUEST from peer %s received, going to send %u peers\n",
-      GNUNET_i2s (peer), send_size);
+  #ifdef ENABLE_MALICIOUS
+  if (1 == mal_type
+      || 3 == mal_type)
+  { /* Try to maximise representation */
+    send_pull_reply (peer, mal_peers, num_mal_peers);
+    return GNUNET_OK;
+  }
 
-  mq = get_mq (peer_map, peer);
+  else if (2 == mal_type)
+  { /* Try to partition network */
+    if (0 == GNUNET_CRYPTO_cmp_peer_identity (&attacked_peer, peer))
+    {
+      send_pull_reply (peer, mal_peers, num_mal_peers);
+    }
+    return GNUNET_OK;
+  }
+  #endif /* ENABLE_MALICIOUS */
 
-  ev = GNUNET_MQ_msg_extra (out_msg,
-                           send_size * sizeof (struct GNUNET_PeerIdentity),
-                           GNUNET_MESSAGE_TYPE_RPS_PP_PULL_REPLY);
-  //out_msg->num_peers = htonl (gossip_list_size);
-  out_msg->num_peers = htonl (send_size);
-  memcpy (&out_msg[1], gossip_list,
-         send_size * sizeof (struct GNUNET_PeerIdentity));
+  view_size = GNUNET_CONTAINER_multipeermap_size (view);
+  generate_view_array (view_size);
 
-  GNUNET_MQ_send (mq, ev);
+  send_pull_reply (peer, view_array, view_size);
 
   return GNUNET_OK;
 }
@@ -1323,8 +1623,6 @@ handle_peer_pull_reply (void *cls,
                         void **channel_ctx,
                         const struct GNUNET_MessageHeader *msg)
 {
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "PULL REPLY received\n");
-
   struct GNUNET_RPS_P2P_PullReplyMessage *in_msg;
   struct GNUNET_PeerIdentity *peers;
   struct PeerContext *peer_ctx;
@@ -1332,6 +1630,9 @@ handle_peer_pull_reply (void *cls,
   struct PeerContext *sender_ctx;
   struct PeerOutstandingOp out_op;
   uint32_t i;
+#ifdef ENABLE_MALICIOUS
+  struct AttackedPeer *tmp_att_peer;
+#endif /* ENABLE_MALICIOUS */
 
   /* Check for protocol violation */
   if (sizeof (struct GNUNET_RPS_P2P_PullReplyMessage) > ntohs (msg->size))
@@ -1359,39 +1660,81 @@ handle_peer_pull_reply (void *cls,
        // FIXME wait for cadet to change this function
   sender_ctx = get_peer_ctx (peer_map, sender);
 
-  if (GNUNET_YES == get_peer_flag (sender_ctx, PULL_REPLY_PENDING))
+  LOG (GNUNET_ERROR_TYPE_DEBUG, "PULL REPLY received (%s)\n", GNUNET_i2s (sender));
+
+  if (GNUNET_YES != get_peer_flag (sender_ctx, PULL_REPLY_PENDING))
   {
+    LOG (GNUNET_ERROR_TYPE_WARNING,
+        "Received a pull reply from a peer we didn't request one from!\n");
     GNUNET_break_op (0);
     return GNUNET_OK;
   }
 
 
+  #ifdef ENABLE_MALICIOUS
+  // We shouldn't even receive pull replies as we're not sending
+  if (2 == mal_type)
+    return GNUNET_OK;
+  #endif /* ENABLE_MALICIOUS */
+
   /* Do actual logic */
-  peers = (struct GNUNET_PeerIdentity *) &msg[1];
+  peers = (struct GNUNET_PeerIdentity *) &in_msg[1];
+
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "PULL REPLY received, got following %u peers:\n",
+       ntohl (in_msg->num_peers));
+
   for (i = 0 ; i < ntohl (in_msg->num_peers) ; i++)
   {
-    peer_ctx = get_peer_ctx (peer_map, &peers[i]);
-    if (GNUNET_YES == get_peer_flag (peer_ctx, VALID)
-        || NULL != peer_ctx->send_channel
-        || NULL != peer_ctx->recv_channel)
-    {
-      if (GNUNET_NO == in_arr (pull_list, pull_list_size, &peers[i])
-          && 0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity, &peers[i]))
-        GNUNET_array_append (pull_list, pull_list_size, peers[i]);
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "%u. %s\n",
+         i,
+         GNUNET_i2s (&peers[i]));
+
+    #ifdef ENABLE_MALICIOUS
+    if (1 == mal_type
+        || 3 == mal_type)
+    { /* Add attacked peer to local list */
+      // TODO check if we sent a request and this was the first reply
+      if (GNUNET_NO == GNUNET_CONTAINER_multipeermap_contains (att_peer_set,
+                                                               &peers[i])
+          && GNUNET_NO == GNUNET_CONTAINER_multipeermap_contains (mal_peer_set,
+                                                                  &peers[i])
+          && 0 != GNUNET_CRYPTO_cmp_peer_identity (&peers[i],
+                                                   &own_identity))
+      {
+        tmp_att_peer = GNUNET_new (struct AttackedPeer);
+        tmp_att_peer->peer_id = peers[i];
+        GNUNET_CONTAINER_DLL_insert (att_peers_head,
+                                     att_peers_tail,
+                                     tmp_att_peer);
+        add_peer_array_to_set (&peers[i], 1, att_peer_set);
+      }
+      continue;
     }
-    else if (GNUNET_NO == insert_in_pull_list_scheduled (peer_ctx))
+    #endif /* ENABLE_MALICIOUS */
+    if (0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity,
+                                              &peers[i]))
     {
-      out_op.op = insert_in_pull_list;
-      out_op.op_cls = NULL;
-      GNUNET_array_append (peer_ctx->outstanding_ops,
-                           peer_ctx->num_outstanding_ops,
-                           out_op);
-      check_peer_live (peer_ctx);
+      peer_ctx = get_peer_ctx (peer_map, &peers[i]);
+      if (GNUNET_YES == get_peer_flag (peer_ctx, VALID))
+      {
+        if (GNUNET_NO == in_arr (pull_list, pull_list_size, &peers[i]))
+          GNUNET_array_append (pull_list, pull_list_size, peers[i]);
+      }
+      else if (GNUNET_NO == insert_in_pull_list_scheduled (peer_ctx))
+      {
+        out_op.op = insert_in_pull_list;
+        out_op.op_cls = NULL;
+        GNUNET_array_append (peer_ctx->outstanding_ops,
+                             peer_ctx->num_outstanding_ops,
+                             out_op);
+        check_peer_live (peer_ctx);
+      }
     }
   }
 
   unset_peer_flag (sender_ctx, PULL_REPLY_PENDING);
-  rem_from_list (&pending_pull_reply_list, &pending_pull_reply_list_size, sender);
 
   return GNUNET_OK;
 }
@@ -1453,13 +1796,16 @@ send_pull_request (struct GNUNET_PeerIdentity *peer_id)
 {
   struct GNUNET_MQ_Envelope *ev;
   struct GNUNET_MQ_Handle *mq;
+  struct PeerContext *peer_ctx;
+
+  peer_ctx = get_peer_ctx (peer_map, peer_id);
+  GNUNET_assert (GNUNET_NO == get_peer_flag (peer_ctx, PULL_REPLY_PENDING));
+  set_peer_flag (peer_ctx, PULL_REPLY_PENDING);
 
   LOG (GNUNET_ERROR_TYPE_DEBUG,
-       "Sending PULL request to peer %s of gossiped list.\n",
+       "Sending PULL request to peer %s of view.\n",
        GNUNET_i2s (peer_id));
 
-  GNUNET_array_append (pending_pull_reply_list, pending_pull_reply_list_size, *peer_id);
-
   ev = GNUNET_MQ_msg_header (GNUNET_MESSAGE_TYPE_RPS_PP_PULL_REQUEST);
   mq = get_mq (peer_map, peer_id);
   GNUNET_MQ_send (mq, ev);
@@ -1478,7 +1824,7 @@ send_push (struct GNUNET_PeerIdentity *peer_id)
   struct GNUNET_MQ_Handle *mq;
 
   LOG (GNUNET_ERROR_TYPE_DEBUG,
-       "Sending PUSH to peer %s of gossiped list.\n",
+       "Sending PUSH to peer %s of view.\n",
        GNUNET_i2s (peer_id));
 
   ev = GNUNET_MQ_msg_header (GNUNET_MESSAGE_TYPE_RPS_PP_PUSH);
@@ -1499,8 +1845,7 @@ do_mal_round (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc);
  * Turn RPS service to act malicious.
  *
  * @param cls Closure
- * @param channel The channel the PUSH was received over
- * @param channel_ctx The context associated with this channel
+ * @param client The client that sent the message
  * @param msg The message header
  */
   static void
@@ -1543,6 +1888,7 @@ handle_client_act_malicious (void *cls,
   if (1 == mal_type)
   { /* Try to maximise representation */
     /* Add other malicious peers to those we already know */
+
     num_mal_peers_sent = ntohl (in_msg->num_peers);
     num_mal_peers_old = num_mal_peers;
     GNUNET_array_grow (mal_peers,
@@ -1552,11 +1898,18 @@ handle_client_act_malicious (void *cls,
             peers,
             num_mal_peers_sent * sizeof (struct GNUNET_PeerIdentity));
 
+    /* Add all mal peers to mal_peer_set */
+    add_peer_array_to_set (&mal_peers[num_mal_peers_old],
+                           num_mal_peers_sent,
+                           mal_peer_set);
+
     /* Substitute do_round () with do_mal_round () */
     GNUNET_SCHEDULER_cancel (do_round_task);
     do_round_task = GNUNET_SCHEDULER_add_now (&do_mal_round, NULL);
   }
-  else if (2 == mal_type)
+
+  else if (2 == mal_type
+           || 3 == mal_type)
   { /* Try to partition the network */
     /* Add other malicious peers to those we already know */
     num_mal_peers_sent = ntohl (in_msg->num_peers) - 1;
@@ -1568,9 +1921,14 @@ handle_client_act_malicious (void *cls,
             peers,
             num_mal_peers_sent * sizeof (struct GNUNET_PeerIdentity));
 
+    /* Add all mal peers to mal_peer_set */
+    add_peer_array_to_set (&mal_peers[num_mal_peers_old],
+                           num_mal_peers_sent,
+                           mal_peer_set);
+
     /* Store the one attacked peer */
     memcpy (&attacked_peer,
-            &peers[num_mal_peers_sent],
+            &in_msg->attacked_peer,
             sizeof (struct GNUNET_PeerIdentity));
 
     LOG (GNUNET_ERROR_TYPE_DEBUG,
@@ -1583,8 +1941,7 @@ handle_client_act_malicious (void *cls,
   }
   else if (0 == mal_type)
   { /* Stop acting malicious */
-    num_mal_peers = 0;
-    GNUNET_free (mal_peers);
+    GNUNET_array_grow (mal_peers, num_mal_peers, 0);
 
     /* Substitute do_mal_round () with do_round () */
     GNUNET_SCHEDULER_cancel (do_round_task);
@@ -1594,6 +1951,8 @@ handle_client_act_malicious (void *cls,
   {
     GNUNET_break (0);
   }
+
+  GNUNET_SERVER_receive_done (client,  GNUNET_OK);
 }
 
 
@@ -1608,30 +1967,107 @@ do_mal_round (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
   uint32_t num_pushes;
   uint32_t i;
   struct GNUNET_TIME_Relative time_next_round;
+  struct AttackedPeer *tmp_att_peer;
+  struct PeerContext *peer_ctx;
 
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "Going to execute next round maliciously.\n");
+  LOG (GNUNET_ERROR_TYPE_DEBUG, "Going to execute next round maliciously type %" PRIu32 ".\n",
+      mal_type);
 
   /* Do malicious actions */
   if (1 == mal_type)
   { /* Try to maximise representation */
-    num_pushes = min (min (push_limit, /* FIXME: attacked peer */ num_mal_peers), GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE);
+
+    /* The maximum of pushes we're going to send this round */
+    num_pushes = GNUNET_MIN (GNUNET_MIN (push_limit,
+                                         num_attacked_peers),
+                             GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE);
+
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Going to send %" PRIu32 " pushes\n",
+         num_pushes);
+
+    /* Send PUSHes to attacked peers */
     for (i = 0 ; i < num_pushes ; i++)
-    { /* Send PUSHes to attacked peers */
-      //GNUNET_CONTAINER_multihashmap_iterator_create
-      //send_push ();
+    {
+      if (att_peers_tail == att_peer_index)
+        att_peer_index = att_peers_head;
+      else
+        att_peer_index = att_peer_index->next;
 
-      // TODO send pulls
-      // send_pull_request
+      send_push (&att_peer_index->peer_id);
+    }
+
+    /* Send PULLs to some peers to learn about additional peers to attack */
+    tmp_att_peer = att_peer_index;
+    for (i = 0 ; i < num_pushes * alpha ; i++)
+    {
+      if (att_peers_tail == tmp_att_peer)
+        tmp_att_peer = att_peers_head;
+      else
+        att_peer_index = tmp_att_peer->next;
+
+      send_pull_request (&tmp_att_peer->peer_id);
     }
   }
+
+
   else if (2 == mal_type)
   { /**
      * Try to partition the network
-     * Send as many pushes to attacked peer as possible
+     * Send as many pushes to the attacked peer as possible
+     * That is one push per round as it will ignore more.
      */
       send_push (&attacked_peer);
   }
 
+
+  if (3 == mal_type)
+  { /* Combined attack */
+
+    /* Send PUSH to attacked peers */
+    peer_ctx = get_peer_ctx (peer_map, &attacked_peer);
+    if (GNUNET_YES == get_peer_flag (peer_ctx, VALID))
+    {
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+          "Goding to send push to attacked peer (%s)\n",
+          GNUNET_i2s (&attacked_peer));
+      send_push (&attacked_peer);
+    }
+    else
+      check_peer_live (peer_ctx);
+
+    /* The maximum of pushes we're going to send this round */
+    num_pushes = GNUNET_MIN (GNUNET_MIN (push_limit - 1,
+                                         num_attacked_peers),
+                             GNUNET_CONSTANTS_MAX_CADET_MESSAGE_SIZE);
+
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Going to send %" PRIu32 " pushes\n",
+         num_pushes);
+
+    for (i = 0 ; i < num_pushes ; i++)
+    {
+      if (att_peers_tail == att_peer_index)
+        att_peer_index = att_peers_head;
+      else
+        att_peer_index = att_peer_index->next;
+
+      send_push (&att_peer_index->peer_id);
+    }
+
+    /* Send PULLs to some peers to learn about additional peers to attack */
+    tmp_att_peer = att_peer_index;
+    for (i = 0 ; i < num_pushes * alpha ; i++)
+    {
+      if (att_peers_tail == tmp_att_peer)
+        tmp_att_peer = att_peers_head;
+      else
+        att_peer_index = tmp_att_peer->next;
+
+      send_pull_request (&tmp_att_peer->peer_id);
+    }
+  }
+
   /* Schedule next round */
   time_next_round = compute_rand_delay (round_interval, 2);
 
@@ -1643,7 +2079,7 @@ do_mal_round (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 
 
 /**
- * Send out PUSHes and PULLs, possibly update #gossip_list, samplers.
+ * Send out PUSHes and PULLs, possibly update #view, samplers.
  *
  * This is executed regylary.
  */
@@ -1653,136 +2089,189 @@ do_round (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
   LOG (GNUNET_ERROR_TYPE_DEBUG, "Going to execute next round.\n");
 
   uint32_t i;
+  unsigned int view_size;
   unsigned int *permut;
-  unsigned int n_peers; /* Number of peers we send pushes/pulls to */
+  unsigned int a_peers; /* Number of peers we send pushes to */
+  unsigned int b_peers; /* Number of peers we send pull requests to */
+  uint32_t first_border;
+  uint32_t second_border;
   struct GNUNET_PeerIdentity peer;
-  struct GNUNET_PeerIdentity *tmp_peer;
+  struct PeerContext *peer_ctx;
 
   LOG (GNUNET_ERROR_TYPE_DEBUG,
-       "Printing gossip list:\n");
-  for (i = 0 ; i < gossip_list_size ; i++)
+       "Printing view:\n");
+  to_file (file_name_view_log,
+           "___ new round ___");
+  view_size = GNUNET_CONTAINER_multipeermap_size (view);
+  generate_view_array (view_size);
+  for (i = 0 ; i < view_size ; i++)
+  {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
-         "\t%s\n", GNUNET_i2s (&gossip_list[i]));
-  // TODO log lists, ...
+         "\t%s\n", GNUNET_i2s (&view_array[i]));
+    to_file (file_name_view_log,
+             "=%s\t(do round)",
+             GNUNET_i2s_full (&view_array[i]));
+  }
 
-  /* Would it make sense to have one shuffeled gossip list and then
-   * to send PUSHes to first alpha peers, PULL requests to next beta peers and
-   * use the rest to update sampler?
-   * in essence get random peers with consumption */
 
-  /* Send PUSHes */
-  if (0 < gossip_list_size)
+  /* Send pushes and pull requests */
+  if (0 < view_size)
   {
     permut = GNUNET_CRYPTO_random_permute (GNUNET_CRYPTO_QUALITY_STRONG,
-                                           (unsigned int) gossip_list_size);
-    n_peers = ceil (alpha * gossip_list_size);
+                                           (unsigned int) view_size);
+
+    /* Send PUSHes */
+    a_peers = ceil (alpha * view_size);
+
     LOG (GNUNET_ERROR_TYPE_DEBUG,
-         "Going to send pushes to %u ceil (%f * %u) peers.\n",
-         n_peers, alpha, gossip_list_size);
-    for (i = 0 ; i < n_peers ; i++)
+         "Going to send pushes to %u (ceil (%f * %u)) peers.\n",
+         a_peers, alpha, view_size);
+    for (i = 0; i < a_peers; i++)
     {
-      peer = gossip_list[permut[i]];
+      peer = view_array[permut[i]];
       if (0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity, &peer)) // TODO
       { // FIXME if this fails schedule/loop this for later
         send_push (&peer);
       }
     }
-    GNUNET_free (permut);
-  }
 
-
-  /* Send PULL requests */
-  //permut = GNUNET_CRYPTO_random_permute (GNUNET_CRYPTO_QUALITY_STRONG, (unsigned int) sampler_list->size);
-  n_peers = ceil (beta * gossip_list_size);
-  LOG (GNUNET_ERROR_TYPE_DEBUG,
-       "Going to send pulls to %u ceil (%f * %u) peers.\n",
-       n_peers, beta, gossip_list_size);
-  for (i = 0 ; i < n_peers ; i++)
-  {
-    tmp_peer = get_rand_peer_ignore_list (gossip_list, gossip_list_size,
-        pending_pull_reply_list, pending_pull_reply_list_size);
-    if (NULL != tmp_peer)
+    /* Send PULL requests */
+    b_peers = ceil (beta * view_size);
+    first_border = a_peers;
+    second_border = a_peers + b_peers;
+    if (second_border > view_size)
     {
-      peer = *tmp_peer;
-      GNUNET_free (tmp_peer);
-
-      if (0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity, &peer))
-      {
+      first_border = view_size - b_peers;
+      second_border = view_size;
+    }
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+        "Going to send pulls to %u (ceil (%f * %u)) peers.\n",
+        b_peers, beta, view_size);
+    for (i = first_border; i < second_border; i++)
+    {
+      peer = view_array[permut[i]];
+      peer_ctx = get_peer_ctx (peer_map, &peer);
+      if (0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity, &peer) &&
+          GNUNET_NO == get_peer_flag (peer_ctx, PULL_REPLY_PENDING)) // TODO
+      { // FIXME if this fails schedule/loop this for later
         send_pull_request (&peer);
       }
     }
+
+    GNUNET_free (permut);
+    permut = NULL;
   }
 
 
-  /* Update gossip list */
+  /* Update view */
+  /* TODO see how many peers are in push-/pull- list! */
 
-  if ( push_list_size <= alpha * gossip_list_size &&
-       push_list_size != 0 &&
-       pull_list_size != 0 )
+  if (push_list_size <= alpha * view_size &&
+      0 < push_list_size &&
+      0 < pull_list_size)
   {
-    LOG (GNUNET_ERROR_TYPE_DEBUG, "Update of the gossip list.\n");
+    LOG (GNUNET_ERROR_TYPE_DEBUG, "Update of the view.\n");
 
-    uint32_t first_border;
-    uint32_t second_border;
-    uint32_t r_index;
+    uint32_t final_size;
     uint32_t peers_to_clean_size;
     struct GNUNET_PeerIdentity *peers_to_clean;
 
     peers_to_clean = NULL;
     peers_to_clean_size = 0;
-    GNUNET_array_grow (peers_to_clean, peers_to_clean_size, gossip_list_size);
+    GNUNET_array_grow (peers_to_clean, peers_to_clean_size, view_size);
     memcpy (peers_to_clean,
-            gossip_list,
-            gossip_list_size * sizeof (struct GNUNET_PeerIdentity));
-
-    first_border  =                ceil (alpha * sampler_size_est_need);
-    second_border = first_border + ceil (beta  * sampler_size_est_need);
-
-    GNUNET_array_grow (gossip_list, gossip_list_size, second_border);
+            view_array,
+            view_size * sizeof (struct GNUNET_PeerIdentity));
+
+    /* Seems like recreating is the easiest way of emptying the peermap */
+    GNUNET_CONTAINER_multipeermap_destroy (view);
+    view = GNUNET_CONTAINER_multipeermap_create (view_size, GNUNET_NO);
+    to_file (file_name_view_log,
+             "--- emptied ---");
+
+    first_border  = GNUNET_MIN (ceil (alpha * sampler_size_est_need),
+                                push_list_size);
+    second_border = first_border +
+                    GNUNET_MIN (floor (beta  * sampler_size_est_need),
+                                pull_list_size);
+    final_size    = second_border +
+      ceil ((1 - (alpha + beta)) * sampler_size_est_need);
+
+    GNUNET_array_grow (view_array, view_size, second_border);
+
+    /* Update view with peers received through PUSHes */
+    permut = GNUNET_CRYPTO_random_permute (GNUNET_CRYPTO_QUALITY_STRONG,
+                                           push_list_size);
+    for (i = 0; i < first_border; i++)
+    {
+      view_array[i] = push_list[permut[i]];
+      GNUNET_CONTAINER_multipeermap_put (view, &push_list[permut[i]], NULL,
+          GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_FAST);
 
-    for (i = 0 ; i < first_border ; i++)
-    { // TODO use RPS_sampler_get_n_rand_peers
-      /* Update gossip list with peers received through PUSHes */
-      r_index = GNUNET_CRYPTO_random_u64 (GNUNET_CRYPTO_QUALITY_STRONG,
-                                       push_list_size);
-      gossip_list[i] = push_list[r_index];
+      to_file (file_name_view_log,
+               "+%s\t(push list)",
+               GNUNET_i2s_full (&view_array[i]));
       // TODO change the peer_flags accordingly
     }
+    GNUNET_free (permut);
+    permut = NULL;
 
-    for (i = first_border ; i < second_border ; i++)
+    /* Update view with peers received through PULLs */
+    permut = GNUNET_CRYPTO_random_permute (GNUNET_CRYPTO_QUALITY_STRONG,
+                                           pull_list_size);
+    for (i = first_border; i < second_border; i++)
     {
-      /* Update gossip list with peers received through PULLs */
-      r_index = GNUNET_CRYPTO_random_u64 (GNUNET_CRYPTO_QUALITY_STRONG,
-                                       pull_list_size);
-      gossip_list[i] = pull_list[r_index];
-      // TODO change the peer_flags accordingly
-    }
+      view_array[i] = pull_list[permut[i]];
+      GNUNET_CONTAINER_multipeermap_put (view, &pull_list[permut[i]], NULL,
+          GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_FAST);
 
-    for (i = second_border ; i < sampler_size_est_need ; i++)
-    {
-      /* Update gossip list with peers from history */
-      RPS_sampler_get_n_rand_peers (prot_sampler, hist_update, NULL, 1, GNUNET_NO);
-      num_hist_update_tasks++;
+      to_file (file_name_view_log,
+               "+%s\t(pull list)",
+               GNUNET_i2s_full (&view_array[i]));
       // TODO change the peer_flags accordingly
     }
+    GNUNET_free (permut);
+    permut = NULL;
+
+    /* Update view with peers from history */
+    RPS_sampler_get_n_rand_peers (prot_sampler,
+                                  hist_update,
+                                  NULL,
+                                  final_size - second_border);
+    num_hist_update_tasks = final_size - second_border;
+    // TODO change the peer_flags accordingly
 
-    for (i = 0 ; i < gossip_list_size ; i++)
-      rem_from_list (&peers_to_clean, &peers_to_clean_size, &gossip_list[i]);
+    for (i = 0; i < view_size; i++)
+      rem_from_list (&peers_to_clean, &peers_to_clean_size, &view_array[i]);
 
-    for (i = 0 ; i < peers_to_clean_size ; i++)
+    /* Clean peers that were removed from the view */
+    for (i = 0; i < peers_to_clean_size; i++)
+    {
       peer_clean (&peers_to_clean[i]);
+      to_file (file_name_view_log,
+               "-%s",
+               GNUNET_i2s_full (&peers_to_clean[i]));
+    }
 
     GNUNET_free (peers_to_clean);
+    peers_to_clean = NULL;
   }
   else
   {
-    LOG (GNUNET_ERROR_TYPE_DEBUG, "No update of the gossip list.\n");
+    LOG (GNUNET_ERROR_TYPE_DEBUG, "No update of the view.\n");
   }
   // TODO independent of that also get some peers from CADET_get_peers()?
 
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Received %u pushes and %u pulls last round (alpha (%.2f) * view_size (%u) = %.2f)\n",
+       push_list_size,
+       pull_list_size,
+       alpha,
+       view_size,
+       alpha * view_size);
 
   /* Update samplers */
-  for ( i = 0 ; i < push_list_size ; i++ )
+  for (i = 0; i < push_list_size; i++)
   {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
          "Updating with peer %s from push list\n",
@@ -1792,13 +2281,13 @@ do_round (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
     // TODO set in_flag?
   }
 
-  for ( i = 0 ; i < pull_list_size ; i++ )
+  for (i = 0; i < pull_list_size; i++)
   {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
          "Updating with peer %s from pull list\n",
          GNUNET_i2s (&pull_list[i]));
-    RPS_sampler_update (prot_sampler,   &push_list[i]);
-    RPS_sampler_update (client_sampler, &push_list[i]);
+    RPS_sampler_update (prot_sampler,   &pull_list[i]);
+    RPS_sampler_update (client_sampler, &pull_list[i]);
     // TODO set in_flag?
   }
 
@@ -1812,7 +2301,6 @@ do_round (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
   time_next_round = compute_rand_delay (round_interval, 2);
 
   /* Schedule next round */
-  //do_round_task = GNUNET_SCHEDULER_add_delayed (round_interval, &do_round, NULL);
   do_round_task = GNUNET_SCHEDULER_add_delayed (time_next_round, &do_round, NULL);
   LOG (GNUNET_ERROR_TYPE_DEBUG, "Finished round\n");
 }
@@ -1836,66 +2324,36 @@ init_peer_cb (void *cls,
               unsigned int best_path) // "How long is the best path?
                                       // (0 = unknown, 1 = ourselves, 2 = neighbor)"
 {
-  struct PeerOutstandingOp out_op;
-  struct PeerContext *peer_ctx;
-
-  if (NULL != peer
-      && 0 != GNUNET_CRYPTO_cmp_peer_identity (&own_identity, peer))
+  if (NULL != peer)
   {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
-        "Got peer %s (at %p) from CADET (gossip_list_size: %u)\n",
-        GNUNET_i2s (peer), peer, gossip_list_size);
-
-    // maybe create a function for that
-    peer_ctx = get_peer_ctx (peer_map, peer);
-    if (GNUNET_YES != get_peer_flag (peer_ctx, VALID))
-    {
-      if (GNUNET_NO == insert_in_sampler_scheduled (peer_ctx))
-      {
-        out_op.op = insert_in_sampler;
-        out_op.op_cls = NULL;
-        GNUNET_array_append (peer_ctx->outstanding_ops,
-                             peer_ctx->num_outstanding_ops,
-                             out_op);
-      }
-
-      if (GNUNET_NO == insert_in_gossip_list_scheduled (peer_ctx))
-      {
-        out_op.op = insert_in_gossip_list;
-        out_op.op_cls = NULL;
-        GNUNET_array_append (peer_ctx->outstanding_ops,
-                             peer_ctx->num_outstanding_ops,
-                             out_op);
-      }
-
-      /* Trigger livelyness test on peer */
-      check_peer_live (peer_ctx);
-    }
-
-    // send push/pull to each of those peers?
+         "Got peer_id %s from cadet\n",
+         GNUNET_i2s (peer));
+    new_peer_id (peer);
   }
 }
 
 
 /**
- * Clean the send channel of a peer
+ * Iterator over peers from peerinfo.
+ *
+ * @param cls closure
+ * @param peer id of the peer, NULL for last call
+ * @param hello hello message for the peer (can be NULL)
+ * @param error message
  */
 void
-peer_clean (const struct GNUNET_PeerIdentity *peer)
+process_peerinfo_peers (void *cls,
+                        const struct GNUNET_PeerIdentity *peer,
+                        const struct GNUNET_HELLO_Message *hello,
+                        const char *err_msg)
 {
-  struct PeerContext *peer_ctx;
-  struct GNUNET_CADET_Channel *channel;
-
-  if (GNUNET_YES != in_arr (gossip_list, gossip_list_size, peer)
-      && GNUNET_YES == GNUNET_CONTAINER_multipeermap_contains (peer_map, peer))
+  if (NULL != peer)
   {
-    peer_ctx = get_peer_ctx (peer_map, peer);
-    if (NULL != peer_ctx->send_channel)
-    {
-      channel = peer_ctx->send_channel;
-      peer_ctx->send_channel = NULL;
-      GNUNET_CADET_channel_destroy (channel);
-    }
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "Got peer_id %s from peerinfo\n",
+         GNUNET_i2s (peer));
+    new_peer_id (peer);
   }
 }
 
@@ -1909,60 +2367,104 @@ peer_remove_cb (void *cls, const struct GNUNET_PeerIdentity *key, void *value)
   struct PeerContext *peer_ctx;
   const struct GNUNET_CADET_Channel *channel =
     (const struct GNUNET_CADET_Channel *) cls;
-  struct GNUNET_CADET_Channel *recv;
-  struct GNUNET_CADET_Channel *send;
-
-  if (GNUNET_YES == GNUNET_CONTAINER_multipeermap_contains (peer_map, value))
-  {
-    peer_ctx = (struct PeerContext *) value;
 
-    if (0 != peer_ctx->num_outstanding_ops)
-      GNUNET_array_grow (peer_ctx->outstanding_ops,
-                         peer_ctx->num_outstanding_ops,
-                         0);
+  peer_ctx = (struct PeerContext *) value;
+  set_peer_flag (peer_ctx, TO_DESTROY);
 
-    if (NULL != peer_ctx->mq)
-    {
-      GNUNET_MQ_destroy (peer_ctx->mq);
-      peer_ctx->mq = NULL;
-    }
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+       "Going to clean peer %s\n",
+       GNUNET_i2s (&peer_ctx->peer_id));
 
+  /* If operations are still scheduled for this peer cancel those */
+  if (0 != peer_ctx->num_outstanding_ops)
+  {
+    GNUNET_array_grow (peer_ctx->outstanding_ops,
+                       peer_ctx->num_outstanding_ops,
+                       0);
+  }
 
-    if (NULL != peer_ctx->is_live_task)
-    {
+  /* If we are still waiting for notification whether this peer is live
+   * cancel the according task */
+  if (NULL != peer_ctx->is_live_task)
+  {
     LOG (GNUNET_ERROR_TYPE_DEBUG,
          "Trying to cancle is_live_task for peer %s\n",
          GNUNET_i2s (key));
-      GNUNET_CADET_notify_transmit_ready_cancel (peer_ctx->is_live_task);
-      peer_ctx->is_live_task = NULL;
-    }
+    GNUNET_CADET_notify_transmit_ready_cancel (peer_ctx->is_live_task);
+    peer_ctx->is_live_task = NULL;
+  }
+
+  unset_peer_flag (peer_ctx, PULL_REPLY_PENDING);
+
+  to_file (file_name_view_log,
+           "-%s\t(cleanup channel, other peer)",
+           GNUNET_i2s_full (key));
+  GNUNET_CONTAINER_multipeermap_remove_all (view, key);
+
+  /* If there is still a mq destroy it */
+  if (NULL != peer_ctx->mq)
+  {
+    GNUNET_MQ_destroy (peer_ctx->mq);
+    peer_ctx->mq = NULL;
+  }
 
-    send = peer_ctx->send_channel;
+
+  /* Remove the send_channel
+   * This function should be called again from #cleanup_channel (callback
+   * called on the destruction of channels) and clean up the rest. */
+  if (NULL != peer_ctx->send_channel &&
+      channel != peer_ctx->send_channel)
+  {
+    GNUNET_CADET_channel_destroy (peer_ctx->send_channel);
     peer_ctx->send_channel = NULL;
-    if (NULL != send
-        && channel != send)
-    {
-      GNUNET_CADET_channel_destroy (send);
-    }
+  }
 
-    recv = peer_ctx->send_channel;
+  /* Remove the recv_channel
+   * This function should be called again from #cleanup_channel (callback
+   * called on the destruction of channels) and clean up the rest. */
+  if (NULL != peer_ctx->recv_channel &&
+      channel != peer_ctx->recv_channel)
+  {
+    GNUNET_CADET_channel_destroy (peer_ctx->recv_channel);
     peer_ctx->recv_channel = NULL;
-    if (NULL != recv
-        && channel != recv)
-    {
-      GNUNET_CADET_channel_destroy (recv);
-    }
-
-    if (GNUNET_YES != GNUNET_CONTAINER_multipeermap_remove_all (peer_map, key))
-      LOG (GNUNET_ERROR_TYPE_WARNING, "removing peer from peer_map failed\n");
-    else
-      GNUNET_free (peer_ctx);
   }
 
+  /* If there is no channel we have to remove the context now */
+  if (GNUNET_YES != GNUNET_CONTAINER_multipeermap_remove_all (peer_map, key))
+    LOG (GNUNET_ERROR_TYPE_WARNING, "removing peer from peer_map failed\n");
+
+  GNUNET_free (peer_ctx);
+
   return GNUNET_YES;
 }
 
 
+/**
+ * Clean the send channel of a peer
+ * If there is also no channel to receive messages from that peer, remove it
+ * from the peermap.
+ */
+void
+peer_clean (const struct GNUNET_PeerIdentity *peer)
+{
+  struct PeerContext *peer_ctx;
+  /* struct GNUNET_CADET_Channel *channel; */
+
+  if (GNUNET_YES != GNUNET_CONTAINER_multipeermap_contains (view, peer) &&
+      GNUNET_YES == GNUNET_CONTAINER_multipeermap_contains (peer_map, peer))
+  {
+    peer_ctx = get_peer_ctx (peer_map, peer);
+    GNUNET_CADET_channel_destroy (peer_ctx->send_channel);
+    peer_ctx->send_channel = NULL;
+
+    if (NULL == peer_ctx->recv_channel)
+    {
+      peer_remove_cb (NULL, peer, peer_ctx);
+    }
+  }
+}
+
+
 /**
  * Task run during shutdown.
  *
@@ -1973,15 +2475,18 @@ static void
 shutdown_task (void *cls,
                     const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
+
   LOG (GNUNET_ERROR_TYPE_DEBUG, "RPS is going down\n");
 
+  GNUNET_PEERINFO_notify_cancel (peerinfo_notify_handle);
+  GNUNET_PEERINFO_disconnect (peerinfo_handle);
+
   if (NULL != do_round_task)
   {
     GNUNET_SCHEDULER_cancel (do_round_task);
     do_round_task = NULL;
   }
 
-
   {
   if (GNUNET_SYSERR ==
         GNUNET_CONTAINER_multipeermap_iterate (peer_map, peer_remove_cb, NULL))
@@ -1990,20 +2495,30 @@ shutdown_task (void *cls,
   }
 
   GNUNET_NSE_disconnect (nse);
-  GNUNET_CADET_disconnect (cadet_handle);
   RPS_sampler_destroy (prot_sampler);
   RPS_sampler_destroy (client_sampler);
   LOG (GNUNET_ERROR_TYPE_DEBUG,
        "Size of the peermap: %u\n",
        GNUNET_CONTAINER_multipeermap_size (peer_map));
   GNUNET_break (0 == GNUNET_CONTAINER_multipeermap_size (peer_map));
+  GNUNET_CADET_disconnect (cadet_handle);
   GNUNET_CONTAINER_multipeermap_destroy (peer_map);
-  GNUNET_array_grow (gossip_list, gossip_list_size, 0);
+  GNUNET_CONTAINER_multipeermap_destroy (view);
+  view = NULL;
   GNUNET_array_grow (push_list, push_list_size, 0);
   GNUNET_array_grow (pull_list, pull_list_size, 0);
   #ifdef ENABLE_MALICIOUS
+  struct AttackedPeer *tmp_att_peer;
   GNUNET_array_grow (mal_peers, num_mal_peers, 0);
-  // TODO empty attacked_peers DLL
+  if (NULL != mal_peer_set)
+    GNUNET_CONTAINER_multipeermap_destroy (mal_peer_set);
+  if (NULL != att_peer_set)
+    GNUNET_CONTAINER_multipeermap_destroy (att_peer_set);
+  while (NULL != att_peers_head)
+  {
+    tmp_att_peer = att_peers_head;
+    GNUNET_CONTAINER_DLL_remove (att_peers_head, att_peers_tail, tmp_att_peer);
+  }
   #endif /* ENABLE_MALICIOUS */
 }
 
@@ -2061,6 +2576,12 @@ handle_inbound_channel (void *cls,
   (void) GNUNET_CONTAINER_multipeermap_put (peer_map, &peer, peer_ctx,
       GNUNET_CONTAINER_MULTIHASHMAPOPTION_REPLACE);
 
+  /* This would make the push-message unnecessary */
+  LOG (GNUNET_ERROR_TYPE_DEBUG,
+      "Got peer_id %s from peerinfo\n",
+      GNUNET_i2s (&peer));
+  new_peer_id (&peer);
+
   peer_is_live (peer_ctx);
 
   return NULL; // TODO
@@ -2068,7 +2589,7 @@ handle_inbound_channel (void *cls,
 
 
 /**
- * This is called when a remote peer destroys a channel.
+ * This is called when a channel is destroyed.
  *
  * @param cls The closure
  * @param channel The channel being closed
@@ -2088,33 +2609,57 @@ cleanup_channel (void *cls,
        // FIXME wait for cadet to change this function
 
   if (GNUNET_YES == GNUNET_CONTAINER_multipeermap_contains (peer_map, peer))
-  {
+  {/* We don't want to implicitly create a context that we're about to kill */
     peer_ctx = GNUNET_CONTAINER_multipeermap_get (peer_map, peer);
-
     if (NULL == peer_ctx) /* It could have been removed by shutdown_task */
       return;
 
-    if (channel == peer_ctx->send_channel)
-    { /* Peer probably went down */
-      LOG (GNUNET_ERROR_TYPE_DEBUG,
-           "Peer %s destroyed send channel - probably went down, cleaning up\n",
-           GNUNET_i2s (peer));
-      rem_from_list (&gossip_list, &gossip_list_size, peer);
-      rem_from_list (&pending_pull_reply_list, &pending_pull_reply_list_size, peer);
-
-      peer_ctx->send_channel = NULL;
-      /* Somwewhat {ab,re}use the iterator function */
-      /* Cast to void is ok, because it's used as void in peer_remove_cb */
-      (void) peer_remove_cb ((void *) channel, peer, peer_ctx);
+    if (get_peer_flag (peer_ctx, TO_DESTROY))
+    {/* We initiatad the destruction of this particular peer */
+      if (channel == peer_ctx->send_channel)
+        peer_ctx->send_channel = NULL;
+      else if (channel == peer_ctx->recv_channel)
+        peer_ctx->recv_channel = NULL;
+
+      to_file (file_name_view_log,
+               "-%s\t(cleanup channel, ourself)",
+               GNUNET_i2s_full (peer));
     }
-    else if (channel == peer_ctx->recv_channel)
-    { /* Other peer doesn't want to send us messages anymore */
-      LOG (GNUNET_ERROR_TYPE_DEBUG,
-           "Peer %s destroyed recv channel - cleaning up channel\n",
-           GNUNET_i2s (peer));
-      peer_ctx->recv_channel = NULL;
+
+    else
+    { /* We did not initiate the destruction of this peer */
+      if (channel == peer_ctx->send_channel)
+      { /* Something (but us) killd the channel - clean up peer */
+        LOG (GNUNET_ERROR_TYPE_DEBUG,
+            "send channel (%s) was destroyed - cleaning up\n",
+            GNUNET_i2s (peer));
+        peer_ctx->send_channel = NULL;
+        /* Somwewhat {ab,re}use the iterator function */
+        /* Cast to void is ok, because it's used as void in peer_remove_cb */
+        (void) peer_remove_cb ((void *) channel, peer, peer_ctx);
+      }
+      else if (channel == peer_ctx->recv_channel)
+      { /* Other peer doesn't want to send us messages anymore */
+        LOG (GNUNET_ERROR_TYPE_DEBUG,
+             "Peer %s destroyed recv channel - cleaning up channel\n",
+             GNUNET_i2s (peer));
+        peer_ctx->recv_channel = NULL;
+      }
+      else
+      {
+        LOG (GNUNET_ERROR_TYPE_WARNING,
+             "unknown channel (%s) was destroyed\n",
+             GNUNET_i2s (peer));
+      }
     }
   }
+
+  else
+  { /* We don't know a context to that peer */
+    LOG (GNUNET_ERROR_TYPE_DEBUG,
+         "channel (%s) without associated context was destroyed\n",
+         GNUNET_i2s (peer));
+  }
 }
 
 
@@ -2136,9 +2681,9 @@ rps_start (struct GNUNET_SERVER_Handle *server)
 
   GNUNET_SERVER_add_handlers (server, handlers);
   GNUNET_SERVER_disconnect_notify (server,
-                                  &handle_client_disconnect,
-                                  NULL);
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "Ready to receive requests from clients\n");
+                                   &handle_client_disconnect,
+                                   NULL);
+  LOG (GNUNET_ERROR_TYPE_INFO, "Ready to receive requests from clients\n");
 
 
   do_round_task = GNUNET_SCHEDULER_add_now (&do_round, NULL);
@@ -2162,6 +2707,9 @@ run (void *cls,
      struct GNUNET_SERVER_Handle *server,
      const struct GNUNET_CONFIGURATION_Handle *c)
 {
+  int size;
+  int out_size;
+
   // TODO check what this does -- copied from gnunet-boss
   // - seems to work as expected
   GNUNET_log_setup ("rps", GNUNET_error_type_to_string (GNUNET_ERROR_TYPE_DEBUG), NULL);
@@ -2174,7 +2722,7 @@ run (void *cls,
               "STARTING SERVICE (rps) for peer [%s]\n",
               GNUNET_i2s (&own_identity));
   #ifdef ENABLE_MALICIOUS
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+  GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
               "Malicious execution compiled in.\n");
   #endif /* ENABLE_MALICIOUS */
 
@@ -2185,30 +2733,52 @@ run (void *cls,
                                                         "ROUNDINTERVAL",
                                                         &round_interval))
   {
-    LOG (GNUNET_ERROR_TYPE_DEBUG, "Failed to read ROUNDINTERVAL from config\n");
+    GNUNET_log_config_missing (GNUNET_ERROR_TYPE_ERROR,
+                               "RPS", "ROUNDINTERVAL");
     GNUNET_SCHEDULER_shutdown ();
     return;
   }
 
-  /* Get initial size of sampler/gossip list from the configuration */
-  if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_number (cfg, "RPS",
-                                                         "INITSIZE",
-                                                         (long long unsigned int *) &sampler_size_est_need))
+  /* Get initial size of sampler/view from the configuration */
+  if (GNUNET_OK !=
+      GNUNET_CONFIGURATION_get_value_number (cfg, "RPS", "INITSIZE",
+        (long long unsigned int *) &sampler_size_est_need))
   {
-    LOG (GNUNET_ERROR_TYPE_DEBUG, "Failed to read INITSIZE from config\n");
+    GNUNET_log_config_missing (GNUNET_ERROR_TYPE_ERROR,
+                               "RPS", "INITSIZE");
     GNUNET_SCHEDULER_shutdown ();
     return;
   }
   LOG (GNUNET_ERROR_TYPE_DEBUG, "INITSIZE is %" PRIu64 "\n", sampler_size_est_need);
 
 
-  gossip_list = NULL;
+  view = GNUNET_CONTAINER_multipeermap_create (4, GNUNET_NO);
+
+  /* file_name_view_log */
+  if (GNUNET_OK != GNUNET_DISK_directory_create ("/tmp/rps/"))
+  {
+    LOG (GNUNET_ERROR_TYPE_WARNING,
+         "Failed to create directory /tmp/rps/\n");
+  }
+
+  size = (14 + strlen (GNUNET_i2s_full (&own_identity)) + 1) * sizeof (char);
+  file_name_view_log = GNUNET_malloc (size);
+  out_size = GNUNET_snprintf (file_name_view_log,
+                              size,
+                              "/tmp/rps/view-%s",
+                              GNUNET_i2s_full (&own_identity));
+  if (size < out_size ||
+      0 > out_size)
+  {
+    LOG (GNUNET_ERROR_TYPE_WARNING,
+         "Failed to write string to buffer (size: %i, out_size: %i)\n",
+         size,
+         out_size);
+  }
 
 
   /* connect to NSE */
   nse = GNUNET_NSE_connect (cfg, nse_callback, NULL);
-  // TODO check whether that was successful
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "Connected to NSE\n");
 
 
   alpha = 0.45;
@@ -2234,8 +2804,8 @@ run (void *cls,
                                        &cleanup_channel,
                                        cadet_handlers,
                                        ports);
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "Connected to CADET\n");
 
+  peerinfo_handle = GNUNET_PEERINFO_connect (cfg);
 
   /* Initialise sampler */
   struct GNUNET_TIME_Relative half_round_interval;
@@ -2244,16 +2814,14 @@ run (void *cls,
   half_round_interval = GNUNET_TIME_relative_multiply (round_interval, .5);
   max_round_interval = GNUNET_TIME_relative_add (round_interval, half_round_interval);
 
-  prot_sampler =   RPS_sampler_init (sampler_size_est_need, max_round_interval);
-  client_sampler = RPS_sampler_init (sampler_size_est_need, max_round_interval);
+  prot_sampler =   RPS_sampler_init     (sampler_size_est_need, max_round_interval);
+  client_sampler = RPS_sampler_mod_init (sampler_size_est_need, max_round_interval);
 
   /* Initialise push and pull maps */
   push_list = NULL;
   push_list_size = 0;
   pull_list = NULL;
   pull_list_size = 0;
-  pending_pull_reply_list = NULL;
-  pending_pull_reply_list_size = 0;
 
 
   num_hist_update_tasks = 0;
@@ -2263,6 +2831,10 @@ run (void *cls,
   GNUNET_CADET_get_peers (cadet_handle, &init_peer_cb, NULL);
   // TODO send push/pull to each of those peers?
 
+  peerinfo_notify_handle = GNUNET_PEERINFO_notify (cfg,
+                                                   GNUNET_NO,
+                                                   process_peerinfo_peers,
+                                                   NULL);
 
   rps_start (server);
 }