From: Etienne Dechamps <etienne@edechamps.fr>
Date: Sat, 14 Mar 2015 14:04:50 +0000 (+0000)
Subject: Don't send UDP probes past static relays.
X-Git-Tag: release-1.1pre12~189^2
X-Git-Url: https://tinc-vpn.org/git/browse?p=tinc;a=commitdiff_plain;h=fa432426df7e2e364e310ab4bac28e60f732a3c9

Don't send UDP probes past static relays.

Ironically, commit 0f8e2cc78cafe47a087d3fc9b480551b841aeb30 introduced
a regression on its own, since it accidently removed a return statement
that prevented try_tx_sptps() from sending UDP/MTU probes to nodes that
are past static relays.
---

diff --git a/src/net_packet.c b/src/net_packet.c
index 8dba3258..45989518 100644
--- a/src/net_packet.c
+++ b/src/net_packet.c
@@ -1137,7 +1137,7 @@ static void try_tx_sptps(node_t *n, bool mtu) {
 	/* If we do have a static relay, try everything with that one instead. */
 
 	if(via != n)
-		try_tx_sptps(via, mtu);
+		return try_tx_sptps(via, mtu);
 
 	/* Otherwise, try to establish UDP connectivity. */