From: Etienne Dechamps <etienne@edechamps.fr>
Date: Sat, 23 May 2015 16:24:05 +0000 (+0100)
Subject: Don't set up an ongoing connection to myself.
X-Git-Tag: release-1.1pre12~130
X-Git-Url: https://www.tinc-vpn.org/git/browse?a=commitdiff_plain;h=3e61c7233b087b8400c29ca7a8d079aad8b706d8;hp=3e61c7233b087b8400c29ca7a8d079aad8b706d8;p=tinc

Don't set up an ongoing connection to myself.

It is entirely possible that the configuration file could contain a
ConnectTo statement refering to its own name; that's a reasonable
scenario when one deploys semi-automatically generated tinc.conf files.

Amusingly, tinc does not like that at all, and actually sets up an
outgoing_t structure to myself (which obviously makes no sense). This is
mostly benign, though it does result in non-sensical "Already connected
to myself" messages every retry interval.

However, that also makes things blow up in close_network_connections(),
because there we delete the entire outgoing list and *then* the myself
node, which still has a reference to the freshly deleted outgoing
structure. Boom.
---