Qortal/altcoinj
3
0
Fork 0
mirror of https://github.com/Qortal/altcoinj.git synced 2025-02-14 19:25:51 +00:00
Code Issues Projects Releases Wiki Activity

Don't use peers that claim to have a block chain and then report a height of zero. These are probably buggy reimplementations of the protocol.

Browse Source
This commit is contained in:
Mike Hearn 2012-05-14 20:52:06 -07:00
parent c41b6d74fa
commit 3d5a7ea7ec
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
core/src/main/java/com/google/bitcoin/core/TCPNetworkConnection.java
View File

@ -25,7 +25,6 @@ import java.io.OutputStream;
import java.net.InetAddress; import java.net.InetAddress;
import java.net.InetSocketAddress; import java.net.InetSocketAddress;
import java.net.Socket; import java.net.Socket;
import java.net.SocketException;
import java.util.Date; import java.util.Date;
import java.util.LinkedHashMap; import java.util.LinkedHashMap;
@ -120,8 +119,10 @@ public class TCPNetworkConnection implements NetworkConnection {
versionMessage.bestHeight versionMessage.bestHeight
}); });
// BitCoinJ is a client mode implementation. That means there's not much point in us talking to other client // BitCoinJ is a client mode implementation. That means there's not much point in us talking to other client
// mode nodes because we can't download the data from them we need to find/verify transactions. // mode nodes because we can't download the data from them we need to find/verify transactions. Some bogus
if (!versionMessage.hasBlockChain()) { // implementations claim to have a block chain in their services field but then report a height of zero, filter
// them out here.
if (!versionMessage.hasBlockChain() || versionMessage.bestHeight <= 0) {
// Shut down the socket // Shut down the socket
try { try {
shutdown(); shutdown();