Prevent empty transactions from being added to vtxPrev #3356

1. 
   laanwj commented at 2:32 pm on December 4, 2013: member

   CWalletTx::AddSupportingTransactions() was adding empty transaction to vtxPrev in some cases. Skip over these.

   Part one of the solution to #3190. This prevents invalid vtxPrev from entering the wallet, but not current ones being transmitted.

   This will need to go into 0.8.6 as well.

2. Prevent empty transactions from being added to vtxPrev

   CWalletTx::AddSupportingTransactions() was adding empty transaction
   to vtxPrev in some cases. Skip over these.
   
   Part one of the solution to #3190. This prevents invalid vtxPrev from
   entering the wallet, but not current ones being transmitted.

   d3ef9b00ec
3. 
   BitcoinPullTester commented at 3:13 pm on December 4, 2013: none

   Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/d3ef9b00ecdc9cc449bea073b1ee54a8dcc6aaf1 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.
4. 
   gavinandresen commented at 4:19 am on December 5, 2013: contributor

   ACK
5. laanwj referenced this in commit b1961523f1 on Dec 5, 2013
6. laanwj merged this on Dec 5, 2013
7. laanwj closed this on Dec 5, 2013

   ---

8. laanwj deleted the branch on Apr 9, 2014
9. DrahtBot locked this on Sep 8, 2021

Contributors
laanwj BitcoinPullTester gavinandresen