-fix build issues
[oweals/gnunet.git] / src / ats-tool / gnunet-ats.c
index 2fd30c1a55d9ff1c54e58dcc078977ae7e5e7666..d4569b31bb495209284b44c5c9aaf12dce9e8723 100644 (file)
@@ -1,22 +1,22 @@
 /*
    This file is part of GNUnet.
    (C) 2009--2013 Christian Grothoff (and other contributing authors)
-
    GNUnet is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published
    by the Free Software Foundation; either version 3, or (at your
    option) any later version.
-
    GNUnet is distributed in the hope that it will be useful, but
    WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
    General Public License for more details.
-
    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.
-*/
+ This file is part of GNUnet.
+ (C) 2009--2013 Christian Grothoff (and other contributing authors)
+
+ GNUnet is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published
+ by the Free Software Foundation; either version 3, or (at your
+ option) any later version.
+
+ GNUnet is distributed in the hope that it will be useful, but
+ WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ General Public License for more details.
+
+ 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.
+ */
 
 /**
  * @file ats-tool/gnunet-ats.c
@@ -79,7 +79,6 @@ static int op_print_quotas;
  */
 static int op_monitor;
 
-
 static struct GNUNET_ATS_PerformanceHandle *ph;
 
 static struct GNUNET_ATS_AddressListHandle *alh;
@@ -88,6 +87,8 @@ static struct GNUNET_CONFIGURATION_Handle *cfg;
 
 static GNUNET_SCHEDULER_TaskIdentifier end_task;
 
+static struct GNUNET_CONTAINER_MultiPeerMap *addresses;
+
 
 struct PendingResolutions
 {
@@ -104,15 +105,34 @@ struct PendingResolutions
   struct GNUNET_TRANSPORT_AddressToStringContext * tats_ctx;
 };
 
+struct ATSAddress
+{
+  struct GNUNET_HELLO_Address *address;
+  struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out;
+  struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in;
+};
+
+
 
 static struct PendingResolutions *head;
 
 static struct PendingResolutions *tail;
 
+static int
+free_addr_it (void *cls,
+    const struct GNUNET_PeerIdentity *key,
+    void *value)
+{
+  struct ATSAddress *a = value;
+  GNUNET_break (GNUNET_OK != GNUNET_CONTAINER_multipeermap_remove (addresses,
+      key, a));
+  GNUNET_HELLO_address_free (a->address);
+  GNUNET_free (a);
+  return GNUNET_OK;
+}
 
-static void 
-end (void *cls,
-     const struct GNUNET_SCHEDULER_TaskContext *tc)
+static void
+end(void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct PendingResolutions * pr;
   struct PendingResolutions * next;
@@ -120,8 +140,8 @@ end (void *cls,
 
   if (NULL != alh)
   {
-      GNUNET_ATS_performance_list_addresses_cancel (alh);
-      alh = NULL;
+    GNUNET_ATS_performance_list_addresses_cancel (alh);
+    alh = NULL;
   }
 
   if (NULL != ph)
@@ -134,23 +154,26 @@ end (void *cls,
   next = head;
   while (NULL != (pr = next))
   {
-      next = pr->next;
-      GNUNET_CONTAINER_DLL_remove (head, tail, pr);
-      GNUNET_TRANSPORT_address_to_string_cancel (pr->tats_ctx);
-      GNUNET_free (pr->address);
-      GNUNET_free (pr);
-      pending ++;
+    next = pr->next;
+    GNUNET_CONTAINER_DLL_remove(head, tail, pr);
+    GNUNET_TRANSPORT_address_to_string_cancel (pr->tats_ctx);
+    GNUNET_free(pr->address);
+    GNUNET_free(pr);
+    pending++;
   }
+
+  GNUNET_CONTAINER_multipeermap_iterate(addresses, &free_addr_it, NULL);
+  GNUNET_CONTAINER_multipeermap_destroy(addresses);
+
   if (0 < pending)
     fprintf (stderr, _("%u address resolutions had a timeout\n"), pending);
   if (op_list_used || op_list_all)
-       fprintf (stderr, _("ATS returned results for %u addresses\n"), results);
+    fprintf (stderr, _("ATS returned results for %u addresses\n"), results);
   ret = 0;
 }
 
-
-static void 
-transport_addr_to_str_cb (void *cls, const char *address)
+static void
+transport_addr_to_str_cb(void *cls, const char *address)
 {
   struct PendingResolutions * pr = cls;
   char *ats_str;
@@ -168,90 +191,148 @@ transport_addr_to_str_cb (void *cls, const char *address)
     network = GNUNET_ATS_NET_UNSPECIFIED;
     for (c = 0; c < pr->ats_count; c++)
     {
-        ats_tmp = ats_str;
-
-        ats_type = ntohl(pr->ats[c].type);
-        ats_value = ntohl(pr->ats[c].value);
-
-        if (ats_type > GNUNET_ATS_PropertyCount)
+      ats_tmp = ats_str;
+
+      ats_type = ntohl (pr->ats[c].type);
+      ats_value = ntohl (pr->ats[c].value);
+
+      if (ats_type > GNUNET_ATS_PropertyCount)
+      {
+        fprintf (stderr, "Invalid ATS property type %u %u for address %s\n", ats_type, pr->ats[c].type,
+            address);
+        continue;
+      }
+
+      switch (ats_type)
+      {
+      case GNUNET_ATS_NETWORK_TYPE:
+        if (ats_value > GNUNET_ATS_NetworkTypeCount)
         {
-            GNUNET_break (0);
-            continue;
+          GNUNET_break(0);
+          continue;
         }
+        network = ats_value;
+        GNUNET_asprintf (&ats_prop_value, "%s",
+            GNUNET_ATS_print_network_type (ats_value));
+        break;
+      default:
+        GNUNET_asprintf (&ats_prop_value, "%u", ats_value);
+        break;
+      }
+      if ((verbose) && (ats_type < GNUNET_ATS_PropertyCount))
+      {
+        GNUNET_asprintf (&ats_str, "%s%s=%s, ", ats_tmp, ats_prop_arr[ats_type],
+            ats_prop_value);
+        GNUNET_free(ats_tmp);
+      }
+      GNUNET_free(ats_prop_value);
+    }
 
-        switch (ats_type) {
-          case GNUNET_ATS_NETWORK_TYPE:
-            if (ats_value > GNUNET_ATS_NetworkTypeCount)
-            {
-                GNUNET_break (0);
-                continue;
-            }
-            network = ats_value;
-            GNUNET_asprintf (&ats_prop_value, "%s", GNUNET_ATS_print_network_type(ats_value));
-            break;
-          default:
-            GNUNET_asprintf (&ats_prop_value, "%u", ats_value);
-            break;
-        }
-        if ((verbose) && (ats_type < GNUNET_ATS_PropertyCount))
-        {
-          GNUNET_asprintf (&ats_str, "%s%s=%s, ", ats_tmp, ats_prop_arr[ats_type] , ats_prop_value);
-          GNUNET_free (ats_tmp);
-        }
-        GNUNET_free (ats_prop_value);
-               }
-
-    fprintf (stderr, _("Peer `%s' plugin `%s', address `%s', `%s' bw out: %u Bytes/s, bw in %u Bytes/s, %s\n"),
-      GNUNET_i2s (&pr->address->peer), pr->address->transport_name, address,
-      GNUNET_ATS_print_network_type(network),
-      ntohl (pr->bandwidth_out.value__), ntohl (pr->bandwidth_in.value__),ats_str);
-    GNUNET_free (ats_str);
+    fprintf (stderr,
+        _("Peer `%s' plugin `%s', address `%s', `%s' bw out: %u Bytes/s, bw in %u Bytes/s, %s\n"),
+        GNUNET_i2s (&pr->address->peer), pr->address->transport_name, address,
+        GNUNET_ATS_print_network_type (network),
+        ntohl (pr->bandwidth_out.value__), ntohl (pr->bandwidth_in.value__),
+        ats_str);
+    GNUNET_free(ats_str);
   }
   else
   {
     /* We're done */
-    GNUNET_CONTAINER_DLL_remove (head, tail, pr);
-    GNUNET_free (pr->address);
-    GNUNET_free (pr);
+    GNUNET_CONTAINER_DLL_remove(head, tail, pr);
+    GNUNET_free(pr->address);
+    GNUNET_free(pr);
     pending--;
 
     if ((GNUNET_YES == receive_done) && (0 == pending))
     {
-        /* All messages received and no resolutions pending*/
-        if (end_task != GNUNET_SCHEDULER_NO_TASK)
-          GNUNET_SCHEDULER_cancel (end_task);
-        end_task = GNUNET_SCHEDULER_add_now (end, NULL);
+      /* All messages received and no resolutions pending*/
+      if (end_task != GNUNET_SCHEDULER_NO_TASK )
+        GNUNET_SCHEDULER_cancel (end_task);
+      end_task = GNUNET_SCHEDULER_add_now (end, NULL );
     }
   }
 }
 
 
-static void 
-ats_perf_cb (void *cls,
-            const struct GNUNET_HELLO_Address *address,
-            int active,
-            struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out,
-            struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in,
-            const struct GNUNET_ATS_Information *ats,
-            uint32_t ats_count)
+struct AddressFindCtx
+{
+  const struct GNUNET_HELLO_Address *src;
+  struct ATSAddress *res;
+};
+
+static int
+find_address_it (void *cls,
+    const struct GNUNET_PeerIdentity *key,
+    void *value)
+{
+  struct AddressFindCtx *actx = cls;
+  struct ATSAddress *exist = value;
+
+  if (0 == GNUNET_HELLO_address_cmp (actx->src, exist->address))
+  {
+    actx->res = exist;
+    return GNUNET_NO;
+  }
+
+  return GNUNET_YES;
+}
+
+static void
+ats_perf_cb(void *cls, const struct GNUNET_HELLO_Address *address, int active,
+    struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out,
+    struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in,
+    const struct GNUNET_ATS_Information *ats, uint32_t ats_count)
 {
   struct PendingResolutions * pr;
 
+  if ((GNUNET_YES == op_monitor) && (GNUNET_NO == verbose))
+  {
+    struct AddressFindCtx actx;
+
+    actx.src = address;
+    actx.res = NULL;
+
+    GNUNET_CONTAINER_multipeermap_iterate (addresses, find_address_it, &actx);
+    if ((actx.res != NULL))
+    {
+      if ((bandwidth_in.value__ == actx.res->bandwidth_in.value__) &&
+          (bandwidth_out.value__ == actx.res->bandwidth_out.value__) )
+      {
+        return; /* Nothing to do here */
+      }
+      else
+      {
+        actx.res->bandwidth_in = bandwidth_in;
+        actx.res->bandwidth_out = bandwidth_out;
+      }
+    }
+
+    struct ATSAddress *a = GNUNET_new (struct ATSAddress);
+    a->address = GNUNET_HELLO_address_copy(address);
+    a->bandwidth_in = bandwidth_in;
+    a->bandwidth_out = bandwidth_out;
+    GNUNET_CONTAINER_multipeermap_put (addresses, &address->peer, a,
+        GNUNET_CONTAINER_MULTIHASHMAPOPTION_MULTIPLE);
+  }
+
   if (NULL != address)
   {
     pr = GNUNET_malloc (sizeof (struct PendingResolutions) +
-                        ats_count * sizeof (struct GNUNET_ATS_Information));
+        ats_count * sizeof (struct GNUNET_ATS_Information));
 
     pr->ats_count = ats_count;
     pr->ats = (struct GNUNET_ATS_Information *) &pr[1];
     if (ats_count > 0)
-      memcpy (pr->ats, ats, ats_count * sizeof (struct GNUNET_ATS_Information));
+      memcpy (pr->ats, ats, ats_count * sizeof(struct GNUNET_ATS_Information));
     pr->address = GNUNET_HELLO_address_copy (address);
     pr->bandwidth_in = bandwidth_in;
     pr->bandwidth_out = bandwidth_out;
-    pr->tats_ctx = GNUNET_TRANSPORT_address_to_string(cfg, address,
-                      resolve_addresses_numeric, GNUNET_TIME_UNIT_FOREVER_REL, transport_addr_to_str_cb, pr);
-    GNUNET_CONTAINER_DLL_insert (head, tail, pr);
+    pr->tats_ctx = GNUNET_TRANSPORT_address_to_string (cfg, address,
+        resolve_addresses_numeric,
+        GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 10),
+        transport_addr_to_str_cb, pr);
+    GNUNET_CONTAINER_DLL_insert(head, tail, pr);
     results++;
     pending++;
   }
@@ -263,15 +344,15 @@ ats_perf_cb (void *cls,
     if (0 == pending)
     {
       /* All messages received and no resolutions pending*/
-      if (end_task != GNUNET_SCHEDULER_NO_TASK)
+      if (end_task != GNUNET_SCHEDULER_NO_TASK )
         GNUNET_SCHEDULER_cancel (end_task);
-      end_task = GNUNET_SCHEDULER_add_now (end, NULL);
+      end_task = GNUNET_SCHEDULER_add_now (end, NULL );
     }
   }
 }
 
 static unsigned int
-print_quotas (const struct GNUNET_CONFIGURATION_Handle *cfg)
+print_quotas(const struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   char *network_str[GNUNET_ATS_NetworkTypeCount] = GNUNET_ATS_NetworkTypeString;
   char * entry_in = NULL;
@@ -289,57 +370,64 @@ print_quotas (const struct GNUNET_CONFIGURATION_Handle *cfg)
     GNUNET_asprintf (&entry_in, "%s_QUOTA_IN", network_str[c]);
 
     /* quota out */
-    if (GNUNET_OK == GNUNET_CONFIGURATION_get_value_string(cfg, "ats", entry_out, &quota_out_str))
+    if (GNUNET_OK
+        == GNUNET_CONFIGURATION_get_value_string (cfg, "ats", entry_out,
+            &quota_out_str))
     {
-      if (0 == strcmp(quota_out_str, BIG_M_STRING) ||
-          (GNUNET_SYSERR == GNUNET_STRINGS_fancy_size_to_bytes (quota_out_str, &quota_out)))
+      if (0 == strcmp (quota_out_str, BIG_M_STRING)
+          || (GNUNET_SYSERR
+              == GNUNET_STRINGS_fancy_size_to_bytes (quota_out_str, &quota_out)))
         quota_out = UINT32_MAX;
 
-      GNUNET_free (quota_out_str);
+      GNUNET_free(quota_out_str);
       GNUNET_asprintf (&quota_out_str, "%llu", quota_out);
     }
     else
     {
-        fprintf (stderr, "Outbound quota for network `%11s' not configured!\n",
-            network_str[c]);
-        GNUNET_asprintf (&quota_out_str, "-");
+      fprintf (stderr, "Outbound quota for network `%11s' not configured!\n",
+          network_str[c]);
+      GNUNET_asprintf (&quota_out_str, "-");
     }
-    GNUNET_free (entry_out);
+    GNUNET_free(entry_out);
 
     /* quota in */
-    if (GNUNET_OK == GNUNET_CONFIGURATION_get_value_string(cfg, "ats", entry_in, &quota_in_str))
+    if (GNUNET_OK
+        == GNUNET_CONFIGURATION_get_value_string (cfg, "ats", entry_in,
+            &quota_in_str))
     {
-      if (0 == strcmp(quota_in_str, BIG_M_STRING) ||
-          (GNUNET_SYSERR == GNUNET_STRINGS_fancy_size_to_bytes (quota_in_str, &quota_in)))
-          quota_in = UINT32_MAX;
-      GNUNET_free (quota_in_str);
+      if (0 == strcmp (quota_in_str, BIG_M_STRING)
+          || (GNUNET_SYSERR
+              == GNUNET_STRINGS_fancy_size_to_bytes (quota_in_str, &quota_in)))
+        quota_in = UINT32_MAX;
+      GNUNET_free(quota_in_str);
       GNUNET_asprintf (&quota_in_str, "%llu", quota_in);
     }
     else
     {
-        fprintf (stderr, "Inbound quota for network `%11s' not configured!\n",
-            network_str[c]);
-        GNUNET_asprintf (&quota_in_str, "-");
+      fprintf (stderr, "Inbound quota for network `%11s' not configured!\n",
+          network_str[c]);
+      GNUNET_asprintf (&quota_in_str, "-");
     }
-    GNUNET_free (entry_in);
+    GNUNET_free(entry_in);
 
-    fprintf (stderr, _("Quota for network `%11s' (in/out): %10s / %10s\n"), network_str[c], quota_in_str, quota_out_str );
-    GNUNET_free (quota_out_str);
-    GNUNET_free (quota_in_str);
+    fprintf (stderr, _("Quota for network `%11s' (in/out): %10s / %10s\n"),
+        network_str[c], quota_in_str, quota_out_str);
+    GNUNET_free(quota_out_str);
+    GNUNET_free(quota_in_str);
   }
   return GNUNET_ATS_NetworkTypeCount;
 }
 
-
 static void
-testservice_ats (void *cls,
-                int result)
+testservice_ats(void *cls, int result)
 {
   struct GNUNET_CONFIGURATION_Handle *cfg = cls;
   struct GNUNET_PeerIdentity pid;
   unsigned int c;
   unsigned int type;
 
+  addresses = GNUNET_CONTAINER_multipeermap_create (10, GNUNET_YES);
+
   if (GNUNET_YES != result)
   {
     FPRINTF (stderr, _("Service `%s' is not running\n"), "ats");
@@ -350,7 +438,9 @@ testservice_ats (void *cls,
 
   if (NULL != pid_str)
   {
-    if (GNUNET_OK != GNUNET_CRYPTO_hash_from_string (pid_str, &pid.hashPubKey))
+    if (GNUNET_OK
+        != GNUNET_CRYPTO_eddsa_public_key_from_string (pid_str,
+            strlen (pid_str), &pid.public_key))
     {
       FPRINTF (stderr, _("Failed to parse peer identity `%s'\n"), pid_str);
       return;
@@ -360,102 +450,104 @@ testservice_ats (void *cls,
   c = op_list_all + op_list_used + op_monitor + op_set_pref;
   if ((1 < c))
   {
-      FPRINTF (stderr, _("Please select one operation : %s or %s or %s or %s or %s\n"),
-               "--used", "--all", "--monitor", "--preference", "--quotas");
-      return;
+    FPRINTF (stderr,
+        _("Please select one operation : %s or %s or %s or %s or %s\n"),
+        "--used", "--all", "--monitor", "--preference", "--quotas");
+    return;
   }
   if ((0 == c))
     op_list_used = GNUNET_YES; /* set default */
-    if (op_print_quotas)
+  if (op_print_quotas)
+  {
+    ret = print_quotas (cfg);
+    return;
+  }
+  if (op_list_all)
+  {
+    ph = GNUNET_ATS_performance_init (cfg, NULL, NULL );
+    if (NULL == ph)
     {
-        ret = print_quotas (cfg);
-        return;
+      fprintf (stderr, _("Cannot connect to ATS service, exiting...\n") );
+      return;
     }
-    if (op_list_all)
-    {
-        ph = GNUNET_ATS_performance_init (cfg, NULL, NULL);
-        if (NULL == ph)
-        {
-          fprintf (stderr, _("Cannot connect to ATS service, exiting...\n"));
-          return;
-        }
 
-        alh = GNUNET_ATS_performance_list_addresses (ph,
-                                               (NULL == pid_str) ? NULL : &pid,
-                                               GNUNET_YES, ats_perf_cb, NULL);
-        if (NULL == alh)
-        {
-          fprintf (stderr, _("Cannot issue request to ATS service, exiting...\n"));
-          end_task = GNUNET_SCHEDULER_add_now (&end, NULL);
-          return;
-        }
-        end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_FOREVER_REL, &end, NULL);
-    }
-    else if (op_list_used)
+    alh = GNUNET_ATS_performance_list_addresses (ph,
+        (NULL == pid_str) ? NULL : &pid, GNUNET_YES, ats_perf_cb, NULL );
+    if (NULL == alh)
     {
-        ph = GNUNET_ATS_performance_init (cfg, NULL, NULL);
-        if (NULL == ph)
-          fprintf (stderr, _("Cannot connect to ATS service, exiting...\n"));
-
-        alh = GNUNET_ATS_performance_list_addresses (ph,
-                                               (NULL == pid_str) ? NULL : &pid,
-                                               GNUNET_NO, ats_perf_cb, NULL);
-        if (NULL == alh)
-        {
-          fprintf (stderr, _("Cannot issue request to ATS service, exiting...\n"));
-          end_task = GNUNET_SCHEDULER_add_now (&end, NULL);
-          return;
-        }
-        end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_FOREVER_REL, &end, NULL);
+      fprintf (stderr, _("Cannot issue request to ATS service, exiting...\n") );
+      end_task = GNUNET_SCHEDULER_add_now (&end, NULL );
+      return;
     }
-    else if (op_monitor)
+    end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_FOREVER_REL, &end,
+        NULL );
+  }
+  else if (op_list_used)
+  {
+    ph = GNUNET_ATS_performance_init (cfg, NULL, NULL );
+    if (NULL == ph)
+      fprintf (stderr, _("Cannot connect to ATS service, exiting...\n") );
+
+    alh = GNUNET_ATS_performance_list_addresses (ph,
+        (NULL == pid_str) ? NULL : &pid, GNUNET_NO, ats_perf_cb, NULL );
+    if (NULL == alh)
     {
-        ph = GNUNET_ATS_performance_init (cfg, ats_perf_cb, NULL);
-        if (NULL == ph)
-          fprintf (stderr, _("Cannot connect to ATS service, exiting...\n"));
-        end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_FOREVER_REL, &end, NULL);
+      fprintf (stderr, _("Cannot issue request to ATS service, exiting...\n") );
+      end_task = GNUNET_SCHEDULER_add_now (&end, NULL );
+      return;
+    }
+    end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_FOREVER_REL, &end,
+        NULL );
+  }
+  else if (op_monitor)
+  {
+    ph = GNUNET_ATS_performance_init (cfg, &ats_perf_cb, NULL );
+    if (NULL == ph)
+      fprintf (stderr, _("Cannot connect to ATS service, exiting...\n") );
+    end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_FOREVER_REL, &end,
+        NULL );
 
+  }
+  else if (op_set_pref)
+  {
+    if (NULL == type_str)
+    {
+      fprintf (stderr, _("No preference type given!\n") );
+      return;
     }
-    else if (op_set_pref)
+    if (NULL == pid_str)
     {
-               if (NULL == type_str)
-               {
-                       fprintf (stderr, _("No preference type given!\n"));
-                       return;
-               }
-               if (NULL == pid_str)
-               {
-                       fprintf (stderr, _("No peer given!\n"));
-                       return;
-               }
-
-
+      fprintf (stderr, _("No peer given!\n") );
+      return;
+    }
 
-        for (c = 0; c<strlen(type_str); c++)
-        {
-          if (isupper (type_str[c]))
-            type_str[c] = tolower (type_str[c]);
-        }
+    for (c = 0; c < strlen (type_str); c++)
+    {
+      if (isupper (type_str[c]))
+        type_str[c] = tolower (type_str[c]);
+    }
 
-        if (0 == strcasecmp("latency", type_str))
-          type = GNUNET_ATS_PREFERENCE_LATENCY;
-        else if (0 == strcasecmp("bandwidth", type_str))
-          type = GNUNET_ATS_PREFERENCE_BANDWIDTH;
-        else
-        {
-          FPRINTF (stderr, "%s", _("Valid type required\n"));
-          return;
-        }
+    if (0 == strcasecmp ("latency", type_str))
+      type = GNUNET_ATS_PREFERENCE_LATENCY;
+    else if (0 == strcasecmp ("bandwidth", type_str))
+      type = GNUNET_ATS_PREFERENCE_BANDWIDTH;
+    else
+    {
+      FPRINTF (stderr, "%s", _("Valid type required\n") );
+      return;
+    }
 
-            /* set */
-            ph = GNUNET_ATS_performance_init (cfg, NULL, NULL);
-            if (NULL == ph)
-              fprintf (stderr, _("Cannot connect to ATS service, exiting...\n"));
+    /* set */
+    ph = GNUNET_ATS_performance_init (cfg, NULL, NULL );
+    if (NULL == ph)
+      fprintf (stderr, _("Cannot connect to ATS service, exiting...\n") );
 
-            GNUNET_ATS_change_preference (ph, &pid, type, (double) value, GNUNET_ATS_PREFERENCE_END);
+    GNUNET_ATS_performance_change_preference (ph, &pid, type, (double) value,
+        GNUNET_ATS_PREFERENCE_END);
 
-            end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS, &end, NULL);
-    }
+    end_task = GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS, &end,
+        NULL );
+  }
   ret = 1;
 }
 
@@ -468,17 +560,14 @@ testservice_ats (void *cls,
  * @param my_cfg configuration
  */
 static void
-run (void *cls, char *const *args, const char *cfgfile,
-     const struct GNUNET_CONFIGURATION_Handle *my_cfg)
+run(void *cls, char * const *args, const char *cfgfile,
+    const struct GNUNET_CONFIGURATION_Handle *my_cfg)
 {
   cfg = (struct GNUNET_CONFIGURATION_Handle *) my_cfg;
-  GNUNET_CLIENT_service_test ("ats", cfg,
-                              TIMEOUT,
-                              &testservice_ats,
-                              (void *) cfg);
+  GNUNET_CLIENT_service_test ("ats", cfg, TIMEOUT, &testservice_ats,
+      (void *) cfg);
 }
 
-
 /**
  * The main function.
  *
@@ -487,7 +576,7 @@ run (void *cls, char *const *args, const char *cfgfile,
  * @return 0 ok, 1 on error
  */
 int
-main (int argc, char *const *argv)
+main(int argc, char * const *argv)
 {
   int res;
   resolve_addresses_numeric = GNUNET_NO;
@@ -499,49 +588,41 @@ main (int argc, char *const *argv)
   receive_done = GNUNET_NO;
   type_str = NULL;
 
-  static const struct GNUNET_GETOPT_CommandLineOption options[] = {
-      {'u', "used", NULL,
-       gettext_noop ("get list of active addresses currently used"),
-       0, &GNUNET_GETOPT_set_one, &op_list_used},
-       {'a', "all", NULL,
-        gettext_noop ("get list of all active addresses"),
-        0, &GNUNET_GETOPT_set_one, &op_list_all},
-      {'n', "numeric", NULL,
-       gettext_noop ("do not resolve IP addresses to hostnames"),
-       0, &GNUNET_GETOPT_set_one, &resolve_addresses_numeric},
-       {'m', "monitor", NULL,
-        gettext_noop ("monitor mode"),
-        0, &GNUNET_GETOPT_set_one, &op_monitor},
-       {'p', "preference", NULL,
-         gettext_noop ("set preference for the given peer"),
-         0, &GNUNET_GETOPT_set_one, &op_set_pref},
-       {'q', "quotas", NULL,
-         gettext_noop ("print all configured quotas"),
-         0, &GNUNET_GETOPT_set_one, &op_print_quotas},
-       {'i', "id", "TYPE",
-         gettext_noop ("peer id"),
-         1, &GNUNET_GETOPT_set_string, &pid_str},
-       {'t', "type", "TYPE",
-         gettext_noop ("preference type to set: latency | bandwidth"),
-         1, &GNUNET_GETOPT_set_string, &type_str},
-       {'k', "value", "VALUE",
-         gettext_noop ("preference value"),
-         1, &GNUNET_GETOPT_set_uint, &value},
-       {'V', "verbose", NULL,
-        gettext_noop ("verbose output (include ATS address properties)"),
-        0, &GNUNET_GETOPT_set_one, &verbose},
-    GNUNET_GETOPT_OPTION_END
-  };
+  static const struct GNUNET_GETOPT_CommandLineOption options[] =
+  {
+  { 'u', "used", NULL,
+      gettext_noop ("get list of active addresses currently used"), 0,
+      &GNUNET_GETOPT_set_one, &op_list_used },
+  { 'a', "all", NULL, gettext_noop ("get list of all active addresses"), 0,
+      &GNUNET_GETOPT_set_one, &op_list_all },
+  { 'n', "numeric", NULL,
+      gettext_noop ("do not resolve IP addresses to hostnames"), 0,
+      &GNUNET_GETOPT_set_one, &resolve_addresses_numeric },
+  { 'm', "monitor", NULL, gettext_noop ("monitor mode"), 0,
+      &GNUNET_GETOPT_set_one, &op_monitor },
+  { 'p', "preference", NULL, gettext_noop ("set preference for the given peer"),
+      0, &GNUNET_GETOPT_set_one, &op_set_pref },
+  { 'q', "quotas", NULL, gettext_noop ("print all configured quotas"), 0,
+      &GNUNET_GETOPT_set_one, &op_print_quotas },
+  { 'i', "id", "TYPE", gettext_noop ("peer id"), 1, &GNUNET_GETOPT_set_string,
+      &pid_str },
+  { 't', "type", "TYPE",
+      gettext_noop ("preference type to set: latency | bandwidth"), 1,
+      &GNUNET_GETOPT_set_string, &type_str },
+  { 'k', "value", "VALUE", gettext_noop ("preference value"), 1,
+      &GNUNET_GETOPT_set_uint, &value },
+  { 'V', "verbose", NULL,
+      gettext_noop ("verbose output (include ATS address properties)"), 0,
+      &GNUNET_GETOPT_set_one, &verbose }, GNUNET_GETOPT_OPTION_END };
 
   if (GNUNET_OK != GNUNET_STRINGS_get_utf8_args (argc, argv, &argc, &argv))
     return 2;
 
   res = GNUNET_PROGRAM_run (argc, argv, "gnunet-ats",
-                              gettext_noop ("Print information about ATS state"), options, &run,
-                              NULL);
-  GNUNET_free_non_null (pid_str);
-  GNUNET_free_non_null (type_str);
-  GNUNET_free ((void *) argv);
+      gettext_noop ("Print information about ATS state"), options, &run, NULL );
+  GNUNET_free_non_null(pid_str);
+  GNUNET_free_non_null(type_str);
+  GNUNET_free((void * ) argv);
 
   if (GNUNET_OK == res)
     return ret;