changes
[oweals/gnunet.git] / src / ats / gnunet-service-ats_addresses.c
index 47ba3312b3de8ac7aea8f6e282ac05b16406560f..49d6d0b593f71cb95783b7c87481179dc3e3e99a 100644 (file)
@@ -62,19 +62,84 @@ enum ATS_Mode
   MODE_MLP
 };
 
-static struct GNUNET_CONTAINER_MultiHashMap *addresses;
+/**
+ * Handle for ATS address component
+ */
+struct GAS_Addresses_Handle
+{
+  /**
+   * A multihashmap to store all addresses
+   */
+  struct GNUNET_CONTAINER_MultiHashMap *addresses;
+
+  /**
+   * Configure WAN quota in
+   */
+  unsigned long long wan_quota_in;
+
+  /**
+   * Configure WAN quota out
+   */
+  unsigned long long wan_quota_out;
+
+  /**
+   * Number of active addresses
+   */
+  unsigned int active_addr_count;
+
+  /**
+   * Is ATS addresses running
+   */
+  int running;
+
+  /**
+   * Configured ATS solver
+   */
+  int ats_mode;
+
+  /**
+   *  Solver handle
+   */
+  void *solver;
+
+  /* Solver functions */
 
-static void *solver;
+  /**
+   * Initialize solver
+   */
+  GAS_solver_init s_init;
+
+  /**
+   * Update address in solver
+   */
+  GAS_solver_address_update s_update;
+
+  /**
+   * Get address from solver
+   */
+  GAS_solver_get_preferred_address s_get;
 
-static unsigned long long wan_quota_in;
+  /**
+   * Delete address in solver
+   */
+  GAS_solver_address_delete s_del;
 
-static unsigned long long wan_quota_out;
+  /**
+   * Change preference for quality in solver
+   */
+  GAS_solver_address_change_preference s_pref;
 
-static unsigned int active_addr_count;
+  /**
+   * Shutdown solver
+   */
+  GAS_solver_done s_done;
+};
 
-static int ats_mode;
 
-static int running;
+/**
+ * Temporary handle
+ */
+struct GAS_Addresses_Handle *handle;
 
 
 static unsigned int
@@ -143,12 +208,12 @@ update_bw_simple_it (void *cls, const struct GNUNET_HashCode * key, void *value)
 
   if (GNUNET_YES != aa->active)
     return GNUNET_OK;
-  GNUNET_assert (active_addr_count > 0);
+  GNUNET_assert (handle->active_addr_count > 0);
 
 
   /* Simple method */
-  aa->assigned_bw_in.value__ = htonl (wan_quota_in / active_addr_count);
-  aa->assigned_bw_out.value__ = htonl (wan_quota_out / active_addr_count);
+  aa->assigned_bw_in.value__ = htonl (handle->wan_quota_in / handle->active_addr_count);
+  aa->assigned_bw_out.value__ = htonl (handle->wan_quota_out / handle->active_addr_count);
 
   send_bw_notification (aa);
 
@@ -167,10 +232,10 @@ recalculate_assigned_bw ()
               "Recalculating bandwidth for all active connections\n");
   GNUNET_STATISTICS_update (GSA_stats, "# bandwidth recalculations performed",
                             1, GNUNET_NO);
-  GNUNET_STATISTICS_set (GSA_stats, "# active addresses", active_addr_count,
+  GNUNET_STATISTICS_set (GSA_stats, "# active addresses", handle->active_addr_count,
                          GNUNET_NO);
 
-  GNUNET_CONTAINER_multihashmap_iterate (addresses, &update_bw_simple_it, NULL);
+  GNUNET_CONTAINER_multihashmap_iterate (handle->addresses, &update_bw_simple_it, NULL);
 }
 
 /**
@@ -226,18 +291,18 @@ destroy_address (struct ATS_Address *addr)
 
   ret = GNUNET_NO;
   GNUNET_assert (GNUNET_YES ==
-                 GNUNET_CONTAINER_multihashmap_remove (addresses,
+                 GNUNET_CONTAINER_multihashmap_remove (handle->addresses,
                                                        &addr->peer.hashPubKey,
                                                        addr));
 
 #if HAVE_LIBGLPK
-  if (ats_mode == MODE_MLP)
-    GAS_mlp_address_delete (solver, addresses, addr);
+  if (handle->ats_mode == MODE_MLP)
+    GAS_mlp_address_delete (handle->solver, handle->addresses, addr);
 #endif
 
   if (GNUNET_YES == addr->active)
   {
-    active_addr_count--;
+    handle->active_addr_count--;
     addr->active = GNUNET_NO;
     ret = GNUNET_YES;
   }
@@ -346,7 +411,7 @@ find_address (const struct GNUNET_PeerIdentity *peer,
   cac.exact_address = NULL;
   cac.base_address = NULL;
   cac.search = addr;
-  GNUNET_CONTAINER_multihashmap_get_multiple (addresses, &peer->hashPubKey,
+  GNUNET_CONTAINER_multihashmap_get_multiple (handle->addresses, &peer->hashPubKey,
                                               &compare_address_it, &cac);
 
 #if 0
@@ -396,7 +461,7 @@ lookup_address (const struct GNUNET_PeerIdentity *peer,
 }
 
 
-#if DEADCODE
+#if 0
 static int
 compare_address_session_it (void *cls, const struct GNUNET_HashCode * key, void *value)
 {
@@ -407,7 +472,7 @@ compare_address_session_it (void *cls, const struct GNUNET_HashCode * key, void
   {
       if ((0 == memcmp (aa->addr, cac->search->addr, aa->addr_len)) && (aa->session_id == cac->search->session_id))
       {
-        cac->exact_address = aa;
+       cac->exact_address = aa;
         return GNUNET_NO;
       }
   }
@@ -432,13 +497,12 @@ find_exact_address (const struct GNUNET_PeerIdentity *peer,
 
   cac.exact_address = NULL;
   cac.search = addr;
-  GNUNET_CONTAINER_multihashmap_get_multiple (addresses, &peer->hashPubKey,
+  GNUNET_CONTAINER_multihashmap_get_multiple (handle->addresses, &peer->hashPubKey,
                                               &compare_address_session_it, &cac);
   return cac.exact_address;
 }
 #endif
 
-
 void
 GAS_addresses_add (const struct GNUNET_PeerIdentity *peer,
                       const char *plugin_name, const void *plugin_addr,
@@ -449,10 +513,10 @@ GAS_addresses_add (const struct GNUNET_PeerIdentity *peer,
   struct ATS_Address *aa;
   struct ATS_Address *old;
 
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return;
 
-  GNUNET_assert (NULL != addresses);
+  GNUNET_assert (NULL != handle->addresses);
 
   aa = create_address (peer,
                        plugin_name,
@@ -470,7 +534,7 @@ GAS_addresses_add (const struct GNUNET_PeerIdentity *peer,
   {
     /* We have a new address */
     GNUNET_assert (GNUNET_OK ==
-                   GNUNET_CONTAINER_multihashmap_put (addresses,
+                   GNUNET_CONTAINER_multihashmap_put (handle->addresses,
                                                       &peer->hashPubKey, aa,
                                                       GNUNET_CONTAINER_MULTIHASHMAPOPTION_MULTIPLE));
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Added new address for peer `%s' session id %u, %p\n",
@@ -513,10 +577,10 @@ GAS_addresses_update (const struct GNUNET_PeerIdentity *peer,
   struct ATS_Address *old;
   uint32_t i;
 
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return;
 
-  GNUNET_assert (NULL != addresses);
+  GNUNET_assert (NULL != handle->addresses);
 
   /* Get existing address */
   old = lookup_address (peer, plugin_name, plugin_addr, plugin_addr_len,
@@ -564,18 +628,8 @@ GAS_addresses_update (const struct GNUNET_PeerIdentity *peer,
       break;
     }
 
-  switch (ats_mode)
-  {
-    case MODE_MLP:
-      GAS_mlp_address_update (solver, addresses, old);
-      break;
-    case MODE_SIMPLISTIC:
-      GAS_simplistic_address_update (solver, addresses, old);
-      break;
-    default:
-      GNUNET_break (0);
-      break;
-  }
+  /* Tell solver about update */
+  handle->s_update (handle->solver, handle->addresses, old);
 }
 
 
@@ -628,7 +682,7 @@ destroy_by_session_id (void *cls, const struct GNUNET_HashCode * key, void *valu
   if (GNUNET_YES == aa->active)
   {
     aa->active = GNUNET_NO;
-    active_addr_count--;
+    handle->active_addr_count--;
     recalculate_assigned_bw ();
   }
 
@@ -644,8 +698,8 @@ destroy_by_session_id (void *cls, const struct GNUNET_HashCode * key, void *valu
   {
     /* session was set to 0, update address */
 #if HAVE_LIBGLPK
-  if (ats_mode == MODE_MLP)
-    GAS_mlp_address_update (solver, addresses, aa);
+  if (handle->ats_mode == MODE_MLP)
+    GAS_mlp_address_update (handle->solver, handle->addresses, aa);
 #endif
   }
 
@@ -661,7 +715,7 @@ GAS_addresses_destroy (const struct GNUNET_PeerIdentity *peer,
   struct ATS_Address *aa;
   struct ATS_Address *old;
 
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return;
 
   /* Get existing address */
@@ -678,7 +732,7 @@ GAS_addresses_destroy (const struct GNUNET_PeerIdentity *peer,
   GNUNET_break (0 < strlen (plugin_name));
   aa = create_address (peer, plugin_name, plugin_addr, plugin_addr_len, session_id);
 
-  GNUNET_CONTAINER_multihashmap_get_multiple (addresses, &peer->hashPubKey,
+  GNUNET_CONTAINER_multihashmap_get_multiple (handle->addresses, &peer->hashPubKey,
                                               &destroy_by_session_id, aa);
 
   free_address (aa);
@@ -787,7 +841,7 @@ GAS_addresses_in_use (const struct GNUNET_PeerIdentity *peer,
 
   struct ATS_Address *old;
 
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return GNUNET_SYSERR;
 
   old = lookup_address (peer, plugin_name, plugin_addr, plugin_addr_len, session_id, NULL, 0);
@@ -812,10 +866,10 @@ GAS_addresses_in_use (const struct GNUNET_PeerIdentity *peer,
     return GNUNET_SYSERR;
   }
   old->used = in_use;
-#if HAVE_LIBGLPK
-  if (ats_mode == MODE_MLP)
-     GAS_mlp_address_update (solver, addresses, old);
-#endif
+
+  /* Tell solver about update */
+  handle->s_update (handle->solver, handle->addresses, old);
+
   return GNUNET_OK;
 }
 
@@ -847,8 +901,7 @@ request_address_mlp (const struct GNUNET_PeerIdentity *peer)
   if (aa->active == GNUNET_NO)
   {
     aa->active = GNUNET_YES;
-    active_addr_count++;
-
+    handle->active_addr_count++;
     send_bw_notification (aa);
   }
   else
@@ -871,7 +924,7 @@ request_address_simple (const struct GNUNET_PeerIdentity *peer)
   aa = NULL;
 
   /* Get address with: stick to current address, lower distance, lower latency */
-  GNUNET_CONTAINER_multihashmap_get_multiple (addresses, &peer->hashPubKey,
+  GNUNET_CONTAINER_multihashmap_get_multiple (handle->addresses, &peer->hashPubKey,
                                               &find_address_it, &aa);
   if (aa == NULL)
   {
@@ -886,8 +939,8 @@ request_address_simple (const struct GNUNET_PeerIdentity *peer)
   if (aa->active == GNUNET_NO)
   {
     aa->active = GNUNET_YES;
-    active_addr_count++;
-    if (ats_mode == MODE_SIMPLISTIC)
+    handle->active_addr_count++;
+    if (handle->ats_mode == MODE_SIMPLISTIC)
     {
       recalculate_assigned_bw ();
     }
@@ -907,14 +960,14 @@ request_address_simple (const struct GNUNET_PeerIdentity *peer)
 void
 GAS_addresses_request_address (const struct GNUNET_PeerIdentity *peer)
 {
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return;
 
-  if (ats_mode == MODE_SIMPLISTIC)
+  if (handle->ats_mode == MODE_SIMPLISTIC)
   {
     request_address_simple (peer);
   }
-  if (ats_mode == MODE_MLP)
+  if (handle->ats_mode == MODE_MLP)
   {
     request_address_mlp(peer);
   }
@@ -938,27 +991,23 @@ reset_address_it (void *cls, const struct GNUNET_HashCode * key, void *value)
 void
 GAS_addresses_handle_backoff_reset (const struct GNUNET_PeerIdentity *peer)
 {
-  GNUNET_break (GNUNET_SYSERR != GNUNET_CONTAINER_multihashmap_get_multiple (addresses,
+  GNUNET_break (GNUNET_SYSERR != GNUNET_CONTAINER_multihashmap_get_multiple (handle->addresses,
                                               &peer->hashPubKey,
                                               &reset_address_it,
                                               NULL));
 }
 
 
-
-// FIXME: this function should likely end up in the LP-subsystem and
-// not with 'addresses' in the future...
 void
 GAS_addresses_change_preference (const struct GNUNET_PeerIdentity *peer,
                                  enum GNUNET_ATS_PreferenceKind kind,
                                  float score)
 {
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return;
-#if HAVE_LIBGLPK
-  if (ats_mode == MODE_MLP)
-    GAS_mlp_address_change_preference (solver, peer, kind, score);
-#endif
+
+  /* Tell solver about update */
+  handle->s_pref (handle->solver, peer, kind, score);
 }
 
 
@@ -969,54 +1018,54 @@ GAS_addresses_change_preference (const struct GNUNET_PeerIdentity *peer,
  * @param cfg configuration to use
  * @param stats the statistics handle to use
  */
-void
+struct GAS_Addresses_Handle *
 GAS_addresses_init (const struct GNUNET_CONFIGURATION_Handle *cfg,
                     const struct GNUNET_STATISTICS_Handle *stats)
 {
-  int c;
+  struct GAS_Addresses_Handle *ah;
   char *quota_wan_in_str;
   char *quota_wan_out_str;
   char *mode_str;
-  running = GNUNET_NO;
+  int c;
+
+  ah = GNUNET_malloc (sizeof (struct GAS_Addresses_Handle));
+  handle = ah;
+  handle->running = GNUNET_NO;
 
   /* Initialize the system with configuration values */
   if (GNUNET_OK == GNUNET_CONFIGURATION_get_value_string(cfg, "ats", "WAN_QUOTA_IN", &quota_wan_in_str))
   {
     if (0 == strcmp(quota_wan_in_str, "unlimited") ||
-        (GNUNET_SYSERR == GNUNET_STRINGS_fancy_size_to_bytes (quota_wan_in_str, &wan_quota_in)))
-      wan_quota_in = (UINT32_MAX) /10;
+        (GNUNET_SYSERR == GNUNET_STRINGS_fancy_size_to_bytes (quota_wan_in_str, &ah->wan_quota_in)))
+      ah->wan_quota_in = (UINT32_MAX) /10;
 
     GNUNET_free (quota_wan_in_str);
     quota_wan_in_str = NULL;
   }
   else
-  {
-    wan_quota_in = (UINT32_MAX) /10;
-  }
+      ah->wan_quota_in = (UINT32_MAX) /10;
 
   if (GNUNET_OK == GNUNET_CONFIGURATION_get_value_string(cfg, "ats", "WAN_QUOTA_OUT", &quota_wan_out_str))
   {
     if (0 == strcmp(quota_wan_out_str, "unlimited") ||
-        (GNUNET_SYSERR == GNUNET_STRINGS_fancy_size_to_bytes (quota_wan_out_str, &wan_quota_out)))
-      wan_quota_out = (UINT32_MAX) /10;
+        (GNUNET_SYSERR == GNUNET_STRINGS_fancy_size_to_bytes (quota_wan_out_str, &ah->wan_quota_out)))
+      ah->wan_quota_out = (UINT32_MAX) /10;
 
     GNUNET_free (quota_wan_out_str);
     quota_wan_out_str = NULL;
   }
   else
-  {
-    wan_quota_out = (UINT32_MAX) /10;
-  }
+    ah->wan_quota_out = (UINT32_MAX) /10;
 
   /* Initialize the addresses database */
-  addresses = GNUNET_CONTAINER_multihashmap_create (128, GNUNET_NO);
-  GNUNET_assert (NULL != addresses);
+  ah->addresses = GNUNET_CONTAINER_multihashmap_create (128, GNUNET_NO);
+  GNUNET_assert (NULL != ah->addresses);
 
   /* Figure out configured solution method */
   if (GNUNET_SYSERR == GNUNET_CONFIGURATION_get_value_string (cfg, "ats", "MODE", &mode_str))
   {
       GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "No ressource assignment method configured, using simplistic approch\n");
-      ats_mode = MODE_SIMPLISTIC;
+      ah->ats_mode = MODE_SIMPLISTIC;
   }
   else
   {
@@ -1024,58 +1073,75 @@ GAS_addresses_init (const struct GNUNET_CONFIGURATION_Handle *cfg,
         mode_str[c] = toupper (mode_str[c]);
       if (0 == strcmp (mode_str, "SIMPLISTIC"))
       {
-          ats_mode = MODE_SIMPLISTIC;
+          ah->ats_mode = MODE_SIMPLISTIC;
       }
       else if (0 == strcmp (mode_str, "MLP"))
       {
-          ats_mode = MODE_MLP;
+          ah->ats_mode = MODE_MLP;
 #if !HAVE_LIBGLPK
           GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Assignment method `%s' configured, but GLPK is not availabe, please install \n", mode_str);
-          ats_mode = MODE_SIMPLISTIC;
+          ah->ats_mode = MODE_SIMPLISTIC;
 #endif
       }
       else
       {
           GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Invalid ressource assignment method `%s' configured, using simplistic approch\n", mode_str);
-          ats_mode = MODE_SIMPLISTIC;
+          ah->ats_mode = MODE_SIMPLISTIC;
       }
+      GNUNET_free (mode_str);
   }
-
   /* Start configured solution method */
-  switch (ats_mode)
+  switch (ah->ats_mode)
   {
     case MODE_MLP:
       /* Init the MLP solver with default values */
-      solver = GAS_mlp_init (cfg, stats, MLP_MAX_EXEC_DURATION, MLP_MAX_ITERATIONS);
-      if (NULL != solver)
-      {
-          ats_mode = MODE_MLP;
-          GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "ATS started in %s mode\n", "MLP");
-          break;
-      }
-      GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Failed to initialize MLP solver!\n");
+#if HAVE_LIBGLPK
+      ah->ats_mode = MODE_MLP;
+      ah->s_init = &GAS_mlp_init;
+      ah->s_update = &GAS_mlp_address_update;
+      ah->s_get = &GAS_mlp_get_preferred_address;
+      ah->s_pref = &GAS_mlp_address_change_preference;
+      ah->s_del =  &GAS_mlp_address_delete;
+      ah->s_done = &GAS_mlp_done;
+#else
+      GNUNET_free (ah);
+      return NULL;
+#endif
+      break;
     case MODE_SIMPLISTIC:
       /* Init the simplistic solver with default values */
+      ah->ats_mode = MODE_SIMPLISTIC;
+      ah->s_init = &GAS_simplistic_init;
+      ah->s_update = &GAS_simplistic_address_update;
+      ah->s_get = &GAS_simplistic_get_preferred_address;
+      ah->s_pref = &GAS_simplistic_address_change_preference;
+      ah->s_del  = &GAS_simplistic_address_delete;
+      ah->s_done = &GAS_simplistic_done;
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "ATS started in %s mode\n", "SIMPLISTIC");
-      solver = GAS_simplistic_init (cfg, stats);
-      if (NULL != solver)
-      {
-          ats_mode = MODE_SIMPLISTIC;
-          GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "ATS started in %s mode\n", "SIMPLISTIC");
-          break;
-      }
-      else
-      {
-        GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Failed to initialize simplistic solver!\n");
-        return;
-      }
       break;
     default:
-      GNUNET_break (0);
+      return NULL;
       break;
   }
+
+  GNUNET_assert (NULL != ah->s_init);
+  GNUNET_assert (NULL != ah->s_update);
+  GNUNET_assert (NULL != ah->s_get);
+  GNUNET_assert (NULL != ah->s_pref);
+  GNUNET_assert (NULL != ah->s_del);
+  GNUNET_assert (NULL != ah->s_done);
+
+  ah->solver = ah->s_init (cfg, stats);
+  if (NULL == ah->solver)
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Failed to initialize MLP solver!\n");
+    GNUNET_free (ah);
+    return NULL;
+  }
+
   /* up and running */
-  running = GNUNET_YES;
+  ah->running = GNUNET_YES;
+  return ah;
 }
 
 
@@ -1100,12 +1166,12 @@ free_address_it (void *cls, const struct GNUNET_HashCode * key, void *value)
 void
 GAS_addresses_destroy_all ()
 {
-  if (GNUNET_NO == running)
+  if (GNUNET_NO == handle->running)
     return;
 
-  if (addresses != NULL)
-    GNUNET_CONTAINER_multihashmap_iterate (addresses, &free_address_it, NULL);
-  GNUNET_assert (active_addr_count == 0);
+  if (handle->addresses != NULL)
+    GNUNET_CONTAINER_multihashmap_iterate (handle->addresses, &free_address_it, NULL);
+  GNUNET_assert (handle->active_addr_count == 0);
 }
 
 
@@ -1113,27 +1179,17 @@ GAS_addresses_destroy_all ()
  * Shutdown address subsystem.
  */
 void
-GAS_addresses_done ()
+GAS_addresses_done (struct GAS_Addresses_Handle *handle)
 {
-  GAS_addresses_destroy_all ();
-  running = GNUNET_NO;
-  GNUNET_CONTAINER_multihashmap_destroy (addresses);
-  addresses = NULL;
+  GNUNET_assert (NULL != handle);
 
+  GAS_addresses_destroy_all ();
+  handle->running = GNUNET_NO;
+  GNUNET_CONTAINER_multihashmap_destroy (handle->addresses);
+  handle->addresses = NULL;
+  GNUNET_free (handle);
   /* Stop configured solution method */
-  switch (ats_mode)
-  {
-    case MODE_MLP:
-      /* Init the MLP solver with default values */
-      GAS_mlp_done (solver);
-      break;
-    case MODE_SIMPLISTIC:
-      /* Init the simplistic solver with default values */
-      GAS_simplistic_done (solver);
-      break;
-    default:
-      break;
-  }
+
 }
 
 struct PeerIteratorContext
@@ -1176,15 +1232,15 @@ GAS_addresses_iterate_peers (GNUNET_ATS_Peer_Iterator p_it, void *p_it_cls)
 
   if (NULL == p_it)
       return;
-  GNUNET_assert (NULL != addresses);
+  GNUNET_assert (NULL != handle->addresses);
 
-  size = GNUNET_CONTAINER_multihashmap_size(addresses);
+  size = GNUNET_CONTAINER_multihashmap_size(handle->addresses);
   if (0 != size)
   {
     ip_ctx.it = p_it;
     ip_ctx.it_cls = p_it_cls;
     ip_ctx.peers_returned = GNUNET_CONTAINER_multihashmap_create (size, GNUNET_NO);
-    GNUNET_CONTAINER_multihashmap_iterate (addresses, &peer_it, &ip_ctx);
+    GNUNET_CONTAINER_multihashmap_iterate (handle->addresses, &peer_it, &ip_ctx);
     GNUNET_CONTAINER_multihashmap_destroy (ip_ctx.peers_returned);
   }
   p_it (p_it_cls, NULL);
@@ -1238,7 +1294,7 @@ GAS_addresses_get_peer_info (const struct GNUNET_PeerIdentity *peer, GNUNET_ATS_
   struct PeerInfoIteratorContext pi_ctx;
   struct GNUNET_BANDWIDTH_Value32NBO zero_bw;
   GNUNET_assert (NULL != peer);
-  GNUNET_assert (NULL != addresses);
+  GNUNET_assert (NULL != handle->addresses);
   if (NULL == pi_it)
     return; /* does not make sense without callback */
 
@@ -1246,7 +1302,7 @@ GAS_addresses_get_peer_info (const struct GNUNET_PeerIdentity *peer, GNUNET_ATS_
   pi_ctx.it = pi_it;
   pi_ctx.it_cls = pi_it_cls;
 
-  GNUNET_CONTAINER_multihashmap_get_multiple (addresses, &peer->hashPubKey, &peerinfo_it, &pi_ctx);
+  GNUNET_CONTAINER_multihashmap_get_multiple (handle->addresses, &peer->hashPubKey, &peerinfo_it, &pi_ctx);
 
   if (NULL != pi_it)
     pi_it (pi_it_cls, NULL, NULL, NULL, 0, GNUNET_NO, NULL, 0, zero_bw, zero_bw);