do not update inflight txs set if already received (#280)

Conflux-Chain · Jul 1, 2019 · 891af59 · 891af59
1 parent d8faf34
commit 891af59
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/core/src/sync/request_manager/mod.rs b/core/src/sync/request_manager/mod.rs
@@ -373,13 +373,13 @@ impl RequestManager {
             let mut indices = Vec::new();
 
             for (idx, tx_id) in received_tx_ids.iter().enumerate() {
-                if !inflight_transactions.insert(*tx_id) {
-                    // Already being requested
+                if received_transactions.contains_txid(tx_id) {
+                    // Already received
                     continue;
                 }
 
-                if received_transactions.contains_txid(tx_id) {
-                    // Already received
+                if !inflight_transactions.insert(*tx_id) {
+                    // Already being requested
                     continue;
                 }