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

Peer: don't null out the future object to avoid a race where the pong can be received before the ping call has returned.

Browse Source
This commit is contained in:
Mike Hearn 2015-06-21 14:39:55 +02:00
parent d6ed5069d4
commit ce50e0b755
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
core/src/main/java/org/bitcoinj/core/Peer.java
View File

@ -1379,12 +1379,12 @@ public class Peer extends PeerSocketHandler {
}
public void complete() {
checkNotNull(future, "Already completed");
Long elapsed = Utils.currentTimeMillis() - startTimeMsec;
Peer.this.addPingTimeData(elapsed);
log.debug("{}: ping time is {} msec", Peer.this.toString(), elapsed);
future.set(elapsed);
future = null;
if (!future.isDone()) {
Long elapsed = Utils.currentTimeMillis() - startTimeMsec;
Peer.this.addPingTimeData(elapsed);
log.debug("{}: ping time is {} msec", Peer.this.toString(), elapsed);
future.set(elapsed);
}
}
}