[GNUnet-SVN] r19435 - gnunet/src/ats

[gnunet]

Author: wachs
Date: 2012-01-26 14:23:49 +0100 (Thu, 26 Jan 2012)
New Revision: 19435

Modified:
   gnunet/src/ats/ats_api.c
   gnunet/src/ats/gnunet-service-ats_addresses_mlp.c
Log:
cppcheck

Modified: gnunet/src/ats/ats_api.c
===================================================================
--- gnunet/src/ats/ats_api.c    2012-01-26 13:20:06 UTC (rev 19434)
+++ gnunet/src/ats/ats_api.c    2012-01-26 13:23:49 UTC (rev 19435)
@@ -252,14 +252,6 @@
   return GNUNET_NO;
 }
 
-
-int
-map_it (void *cls, const GNUNET_HashCode * key, void *value)
-{
-  LOG (GNUNET_ERROR_TYPE_DEBUG, "Found entry for %s\n", GNUNET_h2s (key));
-  return GNUNET_YES;
-}
-
 /**
  * We would like to establish a new connection with a peer.
  * ATS should suggest a good address to begin with.

Modified: gnunet/src/ats/gnunet-service-ats_addresses_mlp.c
===================================================================
--- gnunet/src/ats/gnunet-service-ats_addresses_mlp.c   2012-01-26 13:20:06 UTC 
(rev 19434)
+++ gnunet/src/ats/gnunet-service-ats_addresses_mlp.c   2012-01-26 13:23:49 UTC 
(rev 19435)
@@ -1418,6 +1418,8 @@
   struct ATS_Peer * peer;
   struct ATS_Peer * tmp;
 
+  GNUNET_assert (mlp != NULL);
+
   if (mlp->mlp_task != GNUNET_SCHEDULER_NO_TASK)
   {
     GNUNET_SCHEDULER_cancel(mlp->mlp_task);
@@ -1425,16 +1427,13 @@
   }
 
   /* clean up peer list */
-  if (mlp != NULL)
+  peer = mlp->peer_head;
+  while (peer != NULL)
   {
-    peer = mlp->peer_head;
-    while (peer != NULL)
-    {
-      GNUNET_CONTAINER_DLL_remove(mlp->peer_head, mlp->peer_tail, peer);
-      tmp = peer->next;
-      GNUNET_free (peer);
-      peer = tmp;
-    }
+    GNUNET_CONTAINER_DLL_remove(mlp->peer_head, mlp->peer_tail, peer);
+    tmp = peer->next;
+    GNUNET_free (peer);
+    peer = tmp;
   }
   mlp_delete_problem (mlp);