From 5cc1256fd679cbb8cb80502494b8c02befc757c8 Mon Sep 17 00:00:00 2001
From: Felix Lange <fjl@twurst.com>
Date: Fri, 13 Feb 2015 14:50:14 +0100
Subject: [PATCH] p2p: ensure we don't dial ourself

addPeer doesn't allow self connects, but we can avoid opening
connections in the first place.
---
 p2p/server.go | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/p2p/server.go b/p2p/server.go
index b933401501..e510be521b 100644
--- a/p2p/server.go
+++ b/p2p/server.go
@@ -289,10 +289,13 @@ func (srv *Server) dialLoop() {
 			go srv.findPeers()
 
 		case dest := <-srv.peerConnect:
+			// avoid dialing nodes that are already connected.
+			// there is another check for this in addPeer,
+			// which runs after the handshake.
 			srv.lock.Lock()
 			_, isconnected := srv.peers[dest.ID]
 			srv.lock.Unlock()
-			if isconnected || dialing[dest.ID] {
+			if isconnected || dialing[dest.ID] || dest.ID == srv.ntab.Self() {
 				continue
 			}