Test CHECKMULTISIG with m == 0 and/or n == 0 #3860

1. 
   petertodd commented at 1:42 am on March 13, 2014: contributor

   Turns out that’s valid, and we don’t test it.
2. Fix script test handling of empty scripts

   Previously an empty script would evaluate to OP_0

   b41e594773
3. Test CHECKMULTISIG with m == 0 and n == 0 d3a33fc869
4. 
   laanwj commented at 10:16 am on March 13, 2014: member

   ACK
5. 
   BitcoinPullTester commented at 9:14 pm on March 18, 2014: none

   Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/d3a33fc869d23fb13c010a6e9995bdf6574a621e for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
6. 
   petertodd commented at 5:13 am on May 7, 2014: contributor

   Can we get this merged? Writing more tests and it’s annoying not having this in master.
7. laanwj added this to the milestone 0.9.2 on May 7, 2014
8. 
   laanwj commented at 5:57 am on May 7, 2014: member

   Well, it looks fine to me, just a bit surprised that no one else has reviewed it in all that time. @thebluematt @mikehearn as this test data is used by BitcoinJ as well, can you have a look here?
9. 
   petertodd commented at 7:10 am on May 7, 2014: contributor

   Yeah, tests aren’t sexy. Used in python-bitcoinlib too now.
10. 
    sipa commented at 10:56 am on May 7, 2014: member

    Untested ACK
11. 
    mikehearn commented at 12:17 pm on May 9, 2014: contributor

    Sorry, I didn’t review it because I didn’t know about it. At-mentioning me to get a review is definitely the right thing to do!

    The code looks fine and I took the json files and ran them through bitcoinj, and they passed. So it all LGTM. Please merge.

12. gavinandresen referenced this in commit aaab675a78 on May 9, 2014
13. gavinandresen merged this on May 9, 2014
14. gavinandresen closed this on May 9, 2014

    ---

15. markblundeberg referenced this in commit 78bd3d936e on Jun 7, 2019
16. jtoomim referenced this in commit 5a27b8cccd on Jun 29, 2019
17. jonspock referenced this in commit b1ea011d92 on Jul 4, 2019
18. jonspock referenced this in commit 2c1056d027 on Jul 4, 2019
19. proteanx referenced this in commit 4d52f0671d on Jul 5, 2019
20. jonspock referenced this in commit 0b4a16fa7a on Jul 9, 2019
21. dagurval referenced this in commit 6d63b455a3 on Aug 31, 2019
22. sickpig referenced this in commit 47ef86232a on Sep 4, 2019
23. sickpig referenced this in commit c76b8caca6 on Sep 5, 2019
24. dagurval referenced this in commit 4656b09933 on Sep 6, 2019
25. DrahtBot locked this on Sep 8, 2021

Contributors
petertodd laanwj BitcoinPullTester sipa mikehearn