[GNUnet-SVN] r34212 - gnunet/src/dht

[gnunet]

Author: supriti
Date: 2014-08-24 02:44:51 +0200 (Sun, 24 Aug 2014)
New Revision: 34212

Modified:
   gnunet/src/dht/gnunet-service-xdht_neighbours.c
Log:
Fix



Modified: gnunet/src/dht/gnunet-service-xdht_neighbours.c
===================================================================
--- gnunet/src/dht/gnunet-service-xdht_neighbours.c     2014-08-24 00:22:25 UTC 
(rev 34211)
+++ gnunet/src/dht/gnunet-service-xdht_neighbours.c     2014-08-24 00:44:51 UTC 
(rev 34212)
@@ -5332,11 +5332,21 @@
   else
     next_hop = trail[my_index + 1];
   /* Add an entry in routing table for trail from source to its new successor. 
*/
-  if (GNUNET_SYSERR == GDS_ROUTING_add (trail_id, *peer, next_hop))
-  {
-    GNUNET_break(0);
-    return GNUNET_OK;
-  }
+  /* TODO : Verify the logic below
+   * Removed th following error check because GNUNET_SYSERR is returned when 
+   * route with trail_id was already in the routing table. This can happen, 
when
+   * notify_successor was being retried. This should not be an error, and 
should
+   *  be ignored.
+   */
+  GDS_ROUTING_add(trail_id, *peer, next_hop);
+//  if (GNUNET_SYSERR == GDS_ROUTING_add (trail_id, *peer, next_hop))
+//  {
+//    
+//    GNUNET_break(0);
+//    return GNUNET_OK;
+//    
+//  }
+  
   target_friend =
                  GNUNET_CONTAINER_multipeermap_get (friend_peermap, &next_hop);
   if (NULL == target_friend)