Qortal/altcoinj
3
0
Fork 0
mirror of https://github.com/Qortal/altcoinj.git synced 2025-01-30 23:02:15 +00:00
Code Issues Projects Releases Wiki Activity

Fix PeerGroup.peers iterator synchronization

Browse Source
This commit is contained in:
Miron Cuperman (devrandom) 2011-07-25 20:06:51 +00:00
parent 652a394fc1
commit 8469c8fd86
1 changed files with 23 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/com/google/bitcoin/core/PeerGroup.java
View File

@ -160,12 +160,14 @@ public class PeerGroup {
*/
public boolean broadcastTransaction(Transaction tx) {
boolean success = false;
for (Peer peer : peers) {
try {
peer.broadcastTransaction(tx);
success = true;
} catch (IOException e) {
log.error("failed to broadcast to " + peer, e);
synchronized (peers) {
for (Peer peer : peers) {
try {
peer.broadcastTransaction(tx);
success = true;
} catch (IOException e) {
log.error("failed to broadcast to " + peer, e);
}
}
}
return success;
@ -196,8 +198,10 @@ public class PeerGroup {
peerPool.shutdownNow();
for (Peer peer : peers) {
peer.disconnect();
synchronized (peers) {
for (Peer peer : peers) {
peer.disconnect();
}
}
}
@ -268,8 +272,12 @@ public class PeerGroup {
// TODO be more nuanced about which peer to download from. We can also try
// downloading from multiple peers and handle the case when a new peer comes along
// with a longer chain after we thought we were done.
if (!peers.isEmpty())
startBlockChainDownloadFromPeer(peers.iterator().next());
synchronized (peers) {
if (!peers.isEmpty())
{
startBlockChainDownloadFromPeer(peers.iterator().next());
}
}
}
/**
@ -296,8 +304,11 @@ public class PeerGroup {
protected synchronized void handlePeerDeath(Peer peer) {
if (peer == downloadPeer) {
downloadPeer = null;
if (downloadListener != null && !peers.isEmpty())
startBlockChainDownloadFromPeer(peers.iterator().next());
synchronized (peers) {
if (downloadListener != null && !peers.isEmpty()) {
startBlockChainDownloadFromPeer(peers.iterator().next());
}
}
}
}