Backport to v0.11: In (strCommand == "tx"), return if AlreadyHave() #6785

1. 
   dgenr8 commented at 3:37 AM on October 9, 2015: contributor

   Fills a gap in recent reject tracking (backport #6750).

   This is #6588 in master.

2. In (strCommand == "tx"), return if AlreadyHave()

   The main effect is to exit processing for recently-rejected hashes,
   in case they are pushed to us without prior advertisement.  This
   behavior was seen in the wild.
   
   An additional effect is to do early checks for mempool or mapOrphan
   existence.  No logging or nDoS tracking is needed for failures of
   these checks.

   36f14bf2e7
3. laanwj added the label P2P on Oct 9, 2015
4. 
   laanwj commented at 11:47 AM on October 9, 2015: member

   utACK - confirmed that the changes are the same as in 9524c4d35cb6ec4bd7becf1d938d21c536669d1f

5. laanwj added this to the milestone 0.11.0 on Oct 9, 2015
6. laanwj merged this on Oct 9, 2015
7. laanwj closed this on Oct 9, 2015

   ---

8. laanwj referenced this in commit b4dc33e9fb on Oct 9, 2015
9. 
   btcdrak commented at 1:00 PM on October 9, 2015: contributor

   utACK

10. dgenr8 deleted the branch on Sep 19, 2018
11. DrahtBot locked this on Sep 8, 2021

Milestone
0.11.0