[wallet] [moveonly] Check non-null pindex before potentially referencing #10265

1. 
   kallewoof commented at 11:50 pm on April 23, 2017: member

   The once-a-minute LogPrintf in ScanForWalletTransactions will do a null-ref if it happens to hit the 60s mark right at the last block (where pindex becomes null). This was moved up to the top of the block instead, where the while (pindex && !fAbortRescan) has checked pindex for us.
2. fanquake added the label Wallet on Apr 23, 2017
3. 
   jonasschnelli commented at 6:14 am on April 24, 2017: contributor

   Nice catch! utACK 6b816dc56b73025ca8fcb8b2a1e1dab8d18b4fd0
4. 
   NicolasDorier commented at 7:10 am on April 24, 2017: contributor

   utACK 6b816dc56b73025ca8fcb8b2a1e1dab8d18b4fd0. I am surprised there were no more bug report about that.
5. 
   mchrostowski commented at 6:57 am on April 25, 2017: contributor

   @kallewoof I would go with temptation and move the GetTime() >= nNow check to line 1554 as it is vaguely related to ShowProgress, this could help with readability too.
6. 
   kallewoof commented at 7:10 am on April 25, 2017: member

   @mchrostowski Did you mean 1545 by any chance?
7. 
   paveljanik commented at 7:14 am on April 25, 2017: contributor

   Concept ACK @kallewoof Agree with the move to the top of the block.
8. [wallet] Make sure pindex is non-null before possibly referencing in LogPrintf call. c36ea693ee
9. kallewoof force-pushed on Apr 25, 2017
10. 
    kallewoof commented at 7:20 am on April 25, 2017: member

    This PR is now a MOVEONLY. 4 lines moved up to L1547-1550 from L1560-1563.
11. kallewoof renamed this:
    [wallet] Check non-null pindex before potentially referencing
    [wallet] [moveonly] Check non-null pindex before potentially referencing
    on Apr 25, 2017
12. 
    laanwj commented at 2:04 pm on April 25, 2017: member

    utACK c36ea69
13. laanwj merged this on Apr 25, 2017
14. laanwj closed this on Apr 25, 2017

    ---

15. laanwj referenced this in commit c29a0d4812 on Apr 25, 2017
16. kallewoof deleted the branch on Apr 25, 2017
17. luke-jr referenced this in commit 211783512d on Jun 3, 2017
18. luke-jr referenced this in commit 19c0de4670 on Jun 3, 2017
19. luke-jr referenced this in commit 9697b48160 on Jun 3, 2017
20. luke-jr referenced this in commit df465ba435 on Jun 5, 2017
21. luke-jr referenced this in commit ac02502288 on Jun 5, 2017
22. luke-jr referenced this in commit ff13f592c5 on Jun 5, 2017
23. nomnombtc referenced this in commit 506bdbdfa9 on Jul 17, 2017
24. codablock referenced this in commit e2225f37c2 on Jan 26, 2018
25. andvgal referenced this in commit 0ed8f87464 on Jan 6, 2019
26. CryptoCentric referenced this in commit 0c2579e067 on Feb 27, 2019
27. DrahtBot locked this on Dec 16, 2021

Contributors
kallewoof jonasschnelli NicolasDorier mchrostowski paveljanik laanwj

Labels
Wallet