Fix panic

Fix a panic. If the deleted item is the last one, the ConnectingAddrs is out of range will panic. So cann't assign the changed list to itself. Signed-off-by: Jin Qing <[email protected]>
DNAProject · Jul 11, 2017 · 0c0e700 · 0c0e700
1 parent 6f15a40
commit 0c0e700
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/net/node/node.go b/net/node/node.go
@@ -112,11 +112,13 @@ func (node *node) SetAddrInConnectingList(addr string) {
 func (node *node) RemoveAddrInConnectingList(addr string) {
 	node.ConnectingNodes.Lock()
 	defer node.ConnectingNodes.Unlock()
+	addrs := []string{}
 	for i, a := range node.ConnectingAddrs {
 		if strings.Compare(a, addr) == 0 {
-			node.ConnectingAddrs = append(node.ConnectingAddrs[:i], node.ConnectingAddrs[i+1:]...)
+			addrs = append(node.ConnectingAddrs[:i], node.ConnectingAddrs[i+1:]...)
 		}
 	}
+	node.ConnectingAddrs = addrs
 }
 
 func (node *node) UpdateInfo(t time.Time, version uint32, services uint64,