From 3a45ec10588a8d44c965a86bce623ccd0f64e501 Mon Sep 17 00:00:00 2001
From: David Hill <dhill@conformal.com>
Date: Wed, 9 Jul 2014 21:24:06 -0400
Subject: [PATCH] Change AddAddressByIP to return an error.

---
 addrmgr/addrmanager.go | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/addrmgr/addrmanager.go b/addrmgr/addrmanager.go
index ca4c5915..1c19b476 100644
--- a/addrmgr/addrmanager.go
+++ b/addrmgr/addrmanager.go
@@ -662,29 +662,26 @@ func (a *AddrManager) AddAddress(addr *btcwire.NetAddress,
 
 // AddAddressByIP adds an address where we are given an ip:port and not a
 // btcwire.NetAddress.
-func (a *AddrManager) AddAddressByIP(addrIP string) {
+func (a *AddrManager) AddAddressByIP(addrIP string) error {
 	// Split IP and port
 	addr, portStr, err := net.SplitHostPort(addrIP)
 	if err != nil {
-		log.Warnf("AddADddressByIP given bullshit adddress (%s): %v",
-			err)
-		return
+		return err
 	}
 	// Put it in btcwire.Netaddress
 	var na btcwire.NetAddress
 	na.Timestamp = time.Now()
 	na.IP = net.ParseIP(addr)
 	if na.IP == nil {
-		log.Error("Invalid ip address:", addr)
-		return
+		return fmt.Errorf("invalid ip address %s", addr)
 	}
 	port, err := strconv.ParseUint(portStr, 10, 0)
 	if err != nil {
-		log.Error("Invalid port: ", portStr, err)
-		return
+		return fmt.Errorf("invalid port %s: %v", portStr, err)
 	}
 	na.Port = uint16(port)
 	a.AddAddress(&na, &na) // XXX use correct src address
+	return nil
 }
 
 // NeedMoreAddresses returns whether or not the address manager needs more