[0.21] test: Backports #21614

1. 
   MarcoFalke commented at 9:29 am on April 6, 2021: member

   Backport tests
2. fanquake added the label Backport on Apr 6, 2021
3. fanquake added this to the milestone 0.21.1 on Apr 6, 2021
4. 
   fanquake commented at 9:32 am on April 6, 2021: member

   Should/can this include a backport for #21489?
5. 
   MarcoFalke commented at 9:40 am on April 6, 2021: member

   Good idea. Done
6. in src/test/fuzz/versionbits.cpp:352 in 7ed4ddb5ad outdated

   348@@ -343,3 +349,4 @@ void test_one_input(const std::vector<uint8_t>& buffer)
   349         }
   350     }
   351 }
   352+} // namespace
   

   ---

   ---

   

   ajtowns commented at 4:54 am on April 7, 2021:

   This } // namespace line needs to be removed for the backport.

   (EDIT: the patch that’s being backported moved it lower from earlier in the file, but the previous pr that was backported has moved that same line earlier in the file. git just sees the original line being deleted in both patches, and considers it redundant and fine to omit, rather than marking it as a conflict)

   ---

   

   MarcoFalke commented at 5:27 am on April 7, 2021:

   Correct. Fixed.
7. MarcoFalke force-pushed on Apr 7, 2021
8. fanquake requested review from vasild on Apr 16, 2021
9. 
   vasild commented at 7:37 am on April 16, 2021: member

   ACK 4546ac5af1c4105a0cf78a407fefcb189822bcf5

   I verified that the corresponding commits are same in master and in the backport (modulo mechanical diffs due to diverged code bases).

10. vasild approved
11. fanquake added the label Needs rebase on Apr 16, 2021
12. MarcoFalke removed this from the milestone 0.21.1 on Apr 16, 2021
13. MarcoFalke added this to the milestone 0.21.2 on Apr 16, 2021
14. 
    MarcoFalke commented at 9:15 am on April 16, 2021: member

    Moved milestone due to lack of interest
15. 
    jnewbery commented at 9:19 am on April 16, 2021: member

    ACK 4546ac5af1c4105a0cf78a407fefcb189822bcf5

    Verified cherrypick/rebase.

16. test: use pointers in denialofservice_tests/peer_discouragement

    This is a non-functional change that replaces the `CNode` on-stack
    variables with `CNode` pointers.
    
    The reason for this is that it would allow us to add those `CNode`s
    to `CConnman::vNodes[]` which in turn would allow us to check that they
    are disconnected properly - a `CNode` object must be in
    `CConnman::vNodes[]` in order for its `fDisconnect` flag to be set.
    
    If we store pointers to the on-stack variables in `CConnman` then it
    would crash at the end, trying to `delete` them.
    
    Github-Pull: #21571
    Rebased-From: 4d6e246fa46f2309e2998b542e4c104d73d29071

    dfeb6c10bb
17. test: also check disconnect in denialofservice_tests/peer_discouragement

    Use `CConnmanTest` instead of `CConnman` and add the nodes to it
    so that their `fDisconnect` flag is set during disconnection.
    
    Github-Pull: #21571
    Rebased-From: 637bb6da368b87711005b909f451f94909400092

    b765f41164
18. test: make sure non-IP peers get discouraged and disconnected

    Github-Pull: #21571
    Rebased-From: 81747b21719b3fa6b0fdfc3b084c0104d64903f9

    79cdb4a198
19. fuzz: cleanups for versionbits fuzzer

    Github-Pull: #21489
    Rebased-From: aa7f418fe32b3ec53285693a7731decd99be4528

    b8af67eeef
20. MarcoFalke force-pushed on Apr 16, 2021
21. 
    MarcoFalke commented at 9:22 am on April 16, 2021: member

    Rebased
22. fanquake removed the label Needs rebase on Apr 16, 2021
23. 
    jnewbery commented at 9:28 am on April 16, 2021: member

    ACK b8af67eeefc9fc9622f839ec8919b7391d91bf6f

    Verified rebase.

24. 
    ajtowns commented at 9:50 am on April 16, 2021: member

    ACK b8af67eeefc9fc9622f839ec8919b7391d91bf6f ; visually compared individual commits to originals, checked original commits are in master
25. vasild approved
26. 
    vasild commented at 9:50 am on April 16, 2021: member

    ACK b8af67eeefc9fc9622f839ec8919b7391d91bf6f

    Part of the last commit is already applied via b529222ad18f7facbaff394455875b4aa65d653e.

27. fanquake removed this from the milestone 0.21.2 on Apr 16, 2021
28. fanquake added this to the milestone 0.21.1 on Apr 16, 2021
29. fanquake merged this on Apr 16, 2021
30. fanquake closed this on Apr 16, 2021

    ---

31. MarcoFalke deleted the branch on Apr 17, 2021
32. DrahtBot locked this on Aug 16, 2022

Contributors
MarcoFalke fanquake ajtowns vasild jnewbery

Labels
Backport

Milestone
0.21.1