Qt Splash Screen is deleted (and accesses wallet) after Shutdown() (and wallets are deleted). #12372

1. 
   TheBlueMatt commented at 10:55 pm on February 6, 2018: member

   In trying to debug #12337 I added a sleep at the end of AppInitMain() to simulate being able to close the application late in the AppInitMain() process. If I close the splash screen at this point, I get a reliable crash when the splashs screen is deleted as it tries to disconnect itself from the wallet ShowProgress signal after Shutdown() has been called and the wallet has been deleted.
2. MarcoFalke added this to the milestone 0.16.0 on Feb 6, 2018
3. MarcoFalke added the label Wallet on Feb 6, 2018
4. MarcoFalke removed the label Wallet on Feb 7, 2018
5. 
   MarcoFalke commented at 3:10 am on February 7, 2018: member

   @TheBlueMatt Are you sure this is something with the wallet?

   I think the issue is that the shutdown process has started in requestShutdown, but initialize will happily continue with initializeResult and start threads late in the shutdown progess. Deleting this running thread will crash the application according to the qt docs: https://github.com/qt/qtbase/blob/e5033a5c9b769815112e922d0b224af860afd219/src/corelib/thread/qthread.cpp#L412-L415

6. 
   MarcoFalke commented at 3:11 am on February 7, 2018: member

   Potentially I am talking about a different issue.
7. MarcoFalke added the label GUI on Feb 7, 2018
8. laanwj referenced this in commit c9ffab0719 on Feb 7, 2018
9. 
   laanwj commented at 9:35 am on February 7, 2018: member

   Should be fixed by #12374.
10. laanwj referenced this in commit f5a4c3ddf4 on Feb 7, 2018
11. laanwj referenced this in commit 36a927c525 on Feb 8, 2018
12. laanwj closed this on Feb 8, 2018

    ---

13. laanwj referenced this in commit 11f3eac793 on Feb 8, 2018
14. laanwj referenced this in commit daaae36e15 on Feb 8, 2018
15. Willtech referenced this in commit 8b4345ef5c on Feb 11, 2018
16. hkjn referenced this in commit b3f378a116 on Feb 12, 2018
17. HashUnlimited referenced this in commit 8cbba5006d on Mar 16, 2018
18. ccebrecos referenced this in commit 81a1379193 on Sep 14, 2018
19. lionello referenced this in commit 447517dbbb on Nov 7, 2018
20. PastaPastaPasta referenced this in commit 6b32a15735 on Mar 14, 2020
21. PastaPastaPasta referenced this in commit 5ed53ff93c on Mar 14, 2020
22. PastaPastaPasta referenced this in commit bf1d2cecbd on Mar 14, 2020
23. PastaPastaPasta referenced this in commit 61bd9bdc6b on Mar 14, 2020
24. PastaPastaPasta referenced this in commit 7e5c5d40d1 on Mar 15, 2020
25. PastaPastaPasta referenced this in commit aa34483efd on Mar 15, 2020
26. ckti referenced this in commit 1a7a939e50 on Mar 28, 2021
27. ckti referenced this in commit 1d72660574 on Mar 28, 2021
28. MarcoFalke locked this on Sep 8, 2021
29. gades referenced this in commit caaa5ca020 on Mar 10, 2022

Contributors
TheBlueMatt MarcoFalke laanwj

Labels
GUI

Milestone
0.16.0