From: Aaron Krebs <aaron.krebs@everbase.net>
Date: Tue, 29 Jul 2014 09:29:41 +0000 (+0200)
Subject: Revert fix to partition healing problem.
X-Git-Url: https://git.meshlink.io/?a=commitdiff_plain;h=d35161640af5862b80df6ae15806c0d079e2cf7e;p=meshlink

Revert fix to partition healing problem.

More complex solution needed.
This reverts commit e16463732db2045c884fa47aafa36a07c55b0c5c.
This reverts commit 3610b382c6971808aa1988e8af945a7816e7246b.
---

diff --git a/src/net.c b/src/net.c
index 25cc1504..5f50531b 100644
--- a/src/net.c
+++ b/src/net.c
@@ -162,18 +162,8 @@ static void periodic_handler(event_loop_t *loop, void *data) {
 			int i = 0;
 
 			for splay_each(node_t, n, mesh->nodes) {
-				bool trying_unreachable = false;
-
-				if(i++ != r) {
-					if(n->status.reachable) {
-						continue;
-					} else {
-						/* If we see an unreachable node
-						   before node i, try it anyway.
-						*/
-						trying_unreachable = true;
-					}
-				}
+				if(i++ != r)
+					continue;
 
 				if(n->connection)
 					break;
@@ -195,15 +185,6 @@ static void periodic_handler(event_loop_t *loop, void *data) {
 					outgoing->name = xstrdup(n->name);
 					list_insert_tail(mesh->outgoings, outgoing);
 					setup_outgoing_connection(mesh, outgoing);
-				} else if(trying_unreachable) {
-					/* We're trying an unreachable node instead
-					   of node i. We already have an outgoing
-					   to it. Try the next node rather than
-					   breaking here, to avoid churning on a
-					   connection attempt to the first
-					   unreachable node.
-					*/
-					continue;
 				}
 				break;
 			}