fixing resource leaks

[oweals/gnunet.git] / src / ats / gnunet-service-ats_performance.c

diff --git a/src/ats/gnunet-service-ats_performance.c b/src/ats/gnunet-service-ats_performance.c
index 6509dbe9de304ee11e2a2f7f3b04656eda870a0b..5a1d8cc66f081c5430b5569efa2f5d74128eef5a 100644 (file)
--- a/src/ats/gnunet-service-ats_performance.c
+++ b/src/ats/gnunet-service-ats_performance.c
@@ -261,11 +261,11 @@ peerinfo_it (void *cls,
     return;
 
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Callback for peer `%s' plugin `%s' BW out %lu, BW in %lu \n",
+              "Callback for peer `%s' plugin `%s' BW out %u, BW in %u \n",
               GNUNET_i2s (id),
               plugin_name,
-              ntohl (bandwidth_out.value__),
-              ntohl (bandwidth_in.value__));
+              (unsigned int) ntohl (bandwidth_out.value__),
+              (unsigned int) ntohl (bandwidth_in.value__));
   GAS_performance_notify_client(pc,
                                 id,
                                 plugin_name,
@@ -405,31 +405,31 @@ req_addr_peerinfo_it (void *cls,
       return;
   }
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Callback for  %s peer `%s' plugin `%s' BW out %lu, BW in %lu \n",
+              "Callback for  %s peer `%s' plugin `%s' BW out %u, BW in %u\n",
               (active == GNUNET_YES) ? "ACTIVE" : "INACTIVE",
               GNUNET_i2s (id),
               plugin_name,
-              ntohl (bandwidth_out.value__),
-              ntohl (bandwidth_in.value__));
+              (unsigned int) ntohl (bandwidth_out.value__),
+              (unsigned int) ntohl (bandwidth_in.value__));
 
   /* Transmit result */
   if ((GNUNET_YES == ai->all) || (GNUNET_YES == active))
   {
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  "Sending result for  %s peer `%s' plugin `%s' BW out %lu, BW in %lu \n",
+                  "Sending result for  %s peer `%s' plugin `%s' BW out %u, BW in %u\n",
                   (active == GNUNET_YES) ? "ACTIVE" : "INACTIVE",
                   GNUNET_i2s (id),
                   plugin_name,
-                  ntohl (bandwidth_out.value__),
-                  ntohl (bandwidth_in.value__));
+                  (unsigned int) ntohl (bandwidth_out.value__),
+                  (unsigned int) ntohl (bandwidth_in.value__));
     transmit_req_addr (cls,
-        id,
-        plugin_name,
-        plugin_addr, plugin_addr_len,
-        active,
-        atsi,
-        atsi_count,
-        bandwidth_out, bandwidth_in);
+                       id,
+                       plugin_name,
+                       plugin_addr, plugin_addr_len,
+                       active,
+                       atsi,
+                       atsi_count,
+                       bandwidth_out, bandwidth_in);
   }
 }
 
@@ -507,52 +507,42 @@ GAS_handle_request_address_list (void *cls, struct GNUNET_SERVER_Client *client,
 
 void
 GAS_handle_performance_update (struct GNUNET_PeerIdentity *peer,
-                                                                                                                        const char *plugin_name,
-                                                                                                                        const void *plugin_addr,
-                                                                                                                        size_t plugin_addr_len,
-                                                                                                                        const int active,
-                                                                                                                        struct GNUNET_ATS_Information *ats,
-                                                                                                                        uint32_t ats_count,
-                                                                                                                        struct GNUNET_BANDWIDTH_Value32NBO
-                                                                                                                        bandwidth_out,
-                                                                                                                        struct GNUNET_BANDWIDTH_Value32NBO
-                                                                                                                        bandwidth_in)
+    const char *plugin_name, const void *plugin_addr, size_t plugin_addr_len,
+    const int active, struct GNUNET_ATS_Information *ats, uint32_t ats_count,
+    struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out,
+    struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in)
 {
-/* Notify here */
-       GAS_performance_notify_all_clients (peer,
-                                                                                                                                                       plugin_name,
-                                                                                                                                                       plugin_addr, plugin_addr_len,
-                                                                                                                                                       active,
-                                                                                                                                                       ats, ats_count,
-                                                                                                                                                       bandwidth_out, bandwidth_in);
+  /* Notify here */
+  GAS_performance_notify_all_clients (peer, plugin_name, plugin_addr,
+      plugin_addr_len, active, ats, ats_count, bandwidth_out, bandwidth_in);
 
 #if 0
-       struct PerformanceClient *cur;
-       struct PerformanceMonitorClient *curm;
-       struct MonitorResponseMessage *mrm;
-       size_t msglen;
-
-       msglen = sizeof (struct MonitorResponseMessage) +
-                                        ats_count * sizeof (struct GNUNET_ATS_Information);
-       mrm = GNUNET_malloc (msglen);
-
-       mrm->header.type = htons (GNUNET_MESSAGE_TYPE_ATS_MONITOR_RESPONSE);
-       mrm->header.size = htons (msglen);
-       mrm->ats_count = htonl (ats_count);
-       mrm->peer = *peer;
-       memcpy (&mrm[1], ats, ats_count * sizeof (struct GNUNET_ATS_Information));
-
-       for (cur = pc_head; NULL != cur; cur = cur->next)
-               for (curm = cur->pm_head; NULL != curm; curm = curm->next)
-               {
-                               /* Notify client about update */
-                               mrm->id = htonl (curm->id);
-                         GNUNET_SERVER_notification_context_unicast (nc,
-                                       cur->client,
-                                       (struct GNUNET_MessageHeader *) mrm,
-                                       GNUNET_YES);
-               }
-       GNUNET_free (mrm);
+  struct PerformanceClient *cur;
+  struct PerformanceMonitorClient *curm;
+  struct MonitorResponseMessage *mrm;
+  size_t msglen;
+
+  msglen = sizeof (struct MonitorResponseMessage) +
+  ats_count * sizeof (struct GNUNET_ATS_Information);
+  mrm = GNUNET_malloc (msglen);
+
+  mrm->header.type = htons (GNUNET_MESSAGE_TYPE_ATS_MONITOR_RESPONSE);
+  mrm->header.size = htons (msglen);
+  mrm->ats_count = htonl (ats_count);
+  mrm->peer = *peer;
+  memcpy (&mrm[1], ats, ats_count * sizeof (struct GNUNET_ATS_Information));
+
+  for (cur = pc_head; NULL != cur; cur = cur->next)
+  for (curm = cur->pm_head; NULL != curm; curm = curm->next)
+  {
+    /* Notify client about update */
+    mrm->id = htonl (curm->id);
+    GNUNET_SERVER_notification_context_unicast (nc,
+        cur->client,
+        (struct GNUNET_MessageHeader *) mrm,
+        GNUNET_YES);
+  }
+  GNUNET_free (mrm);
 #endif
 }