fix -maxconnections #42

pull dooglus wants to merge 1 commits into bitcoin:master from dooglus:maxconnections-fix changing 1 files +14 −1

dooglus commented at 11:53 PM on January 23, 2011: contributor

The -maxconnections=N command line argument is meant to limit the total number of connections to N. It reserves 8 (or less, if N<8) slots for outgoing connections, and is meant to allow N-8 incoming connections. But the calculation is slightly wrong, and the existing outgoing connections are counted twice.

This fixes it.
Fix -maxconnections. It used to account for the 8 outbound connections twice when calculating the number of slots left for incoming connections. 4698dd9a02
msgilligan referenced this in commit ac06646ace on Jul 17, 2014
rdponticelli referenced this in commit 83fd36c6f8 on Nov 26, 2014
dexX7 referenced this in commit 5054ebab9f on May 15, 2015
tempgit9 referenced this in commit 66c5ffb6eb on Jan 16, 2017
yes-maxheart referenced this in commit d283cc3078 on Dec 16, 2018
Warchant referenced this in commit 040921987f on Dec 31, 2019
velesnetwork referenced this in commit 8a5cfba386 on Jan 12, 2020
MarcoFalke referenced this in commit 54532f46c4 on Nov 18, 2020
MarcoFalke referenced this in commit 90c0f267bd on Dec 8, 2020
satindergrewal referenced this in commit 31d59f9d8f on Jun 22, 2021
rajarshimaitra referenced this in commit 0d546e693f on Aug 5, 2021
DrahtBot locked this on Sep 8, 2021

Contributors