Fix direct UDP communciation with pre-relaying 1.1 nodes.
authorEtienne Dechamps <etienne@edechamps.fr>
Mon, 18 May 2015 20:06:16 +0000 (21:06 +0100)
committerEtienne Dechamps <etienne@edechamps.fr>
Mon, 18 May 2015 20:08:43 +0000 (21:08 +0100)
try_tx_sptps() gives up on UDP communication if the recipient doesn't
support relaying. This is too restrictive - we only need the other node
to support relaying if we actually want to relay through them. If the
packet is sent directly, it's fine to send it to an old pre-node-IDs
tinc-1.1 node.

src/net_packet.c

index 4c06045..fc24c9a 100644 (file)
@@ -1190,15 +1190,13 @@ static void try_tx_sptps(node_t *n, bool mtu) {
 
        node_t *via = (n->via == myself) ? n->nexthop : n->via;
 
-       /* If the static relay doesn't support SPTPS, everything goes via TCP anyway. */
+       /* If we do have a static relay, try everything with that one instead, if it supports relaying. */
 
-       if((via->options >> 24) < 4)
-               return;
-
-       /* If we do have a static relay, try everything with that one instead. */
-
-       if(via != n)
+       if(via != n) {
+               if((via->options >> 24) < 4)
+                       return;
                return try_tx_sptps(via, mtu);
+       }
 
        /* Otherwise, try to establish UDP connectivity. */