ban function requires a ban duration as an argument

p2p/pex/addrbook.go

@@ -339,7 +339,7 @@ func (a *addrBook) ReinstateBadPeers() {
 	a.mtx.Lock()
 	defer a.mtx.Unlock()
 	for _, ka := range a.badPeers {
-		if !ka.isBanned(defaultBanTime) {
+		if !ka.isBanned() {
 			bucket := a.calcNewBucket(ka.Addr, ka.Src)
 			a.addToNewBucket(ka, bucket)
 			delete(a.badPeers, ka.ID())
@@ -761,7 +761,7 @@ func (a *addrBook) addBadPeer(addr *p2p.NetAddress) bool {
 	if ka != nil {
 		if _, alreadyBadPeer := a.badPeers[addr.ID]; !alreadyBadPeer {
 			// add to bad peer list
-			ka.ban()
+			ka.ban(defaultBanTime)
 			a.badPeers[addr.ID] = ka
 		}
 		return true

p2p/pex/known_address.go

@@ -55,12 +55,12 @@ func (ka *knownAddress) markGood() {
 	ka.LastSuccess = now
 }
 
-func (ka *knownAddress) ban() {
-	ka.LastBanTime = time.Now()
+func (ka *knownAddress) ban(banTime time.Duration) {
+	ka.LastBanTime = time.Now().Add(banTime)
 }
 
-func (ka *knownAddress) isBanned(banTime time.Duration) bool {
-	return ka.LastBanTime.Add(banTime).Before(time.Now())
+func (ka *knownAddress) isBanned() bool {
+	return ka.LastBanTime.Before(time.Now())
 }
 
 func (ka *knownAddress) addBucketRef(bucketIdx int) int {