Remove broken option to skip input checking for wallet txn. #2769

1. 
   TheBlueMatt commented at 10:24 am on June 14, 2013: member

   This breaks the mempool, causes an assert(false) in -debug mode and can cause SEGFAULT in non-debug mode
2. Remove broken option to skip input checking for wallet txn. b1f15b218b
3. 
   sipa commented at 10:27 am on June 14, 2013: member

   ACK
4. 
   BitcoinPullTester commented at 12:07 pm on June 14, 2013: none

   Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/b1f15b218be8dfd1d2dbd375d469794bb7326748 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.
5. 
   jgarzik commented at 1:50 pm on June 14, 2013: contributor

   ACK
6. 
   petertodd commented at 5:37 pm on June 14, 2013: contributor

   ACK

   In my mempool rewrite work I also implemented this change and haven’t found any downside to doing so. The wallet code is perfectly ok if there are tx’s in the wallet that don’t make it to the mempool for whatever reason.

7. sipa referenced this in commit 9e3a5deaa3 on Jun 15, 2013
8. sipa merged this on Jun 15, 2013
9. sipa closed this on Jun 15, 2013

   ---

10. 
    petertodd commented at 10:19 am on July 4, 2013: contributor

    Actually you broke transaction resurrection in the event of a multi-block re-org with this patch because transactions get added back to the mempool in the wrong order, the correct order within a block, but newest block first. See SetBestChain() for details.
11. DrahtBot locked this on Sep 8, 2021

Contributors
TheBlueMatt sipa BitcoinPullTester jgarzik petertodd