server: prevent adding peers if already disconnected

This addresses an issue where the server ends up tracking a peer that has been disconnected due to it processing a peer's `done` message before its `add` message.
btcsuite · Oct 14, 2019 · baeb789 · baeb789
1 parent 769c4e1
commit baeb789
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/server.go b/server.go
@@ -1563,7 +1563,7 @@ func (s *server) handleUpdatePeerHeights(state *peerState, umsg updatePeerHeight
 // handleAddPeerMsg deals with adding new peers.  It is invoked from the
 // peerHandler goroutine.
 func (s *server) handleAddPeerMsg(state *peerState, sp *serverPeer) bool {
-	if sp == nil {
+	if sp == nil || !sp.Connected() {
 		return false
 	}