test_bitcoin fails valgrind #9278

1. 
   TheBlueMatt commented at 5:11 am on December 5, 2016: member

   On recent master, test_bitcoin generates new valgrind warnings, specifically, rpc_tests.cpp:91 calls getnetworkinfo(), which, on rpc/net.cpp:437 looks up the connman’s nLocalServices, which were never initialized (normally this would happen with CConnman::Start). @theuni might want to take a look here.
2. laanwj added the label Tests on Dec 5, 2016
3. jonasschnelli assigned theuni on Dec 5, 2016
4. 
   theuni commented at 3:05 am on December 6, 2016: member

   Thanks. This really needs an Init() that the ctor and Start both call.

   Will fix.

5. MarcoFalke added this to the milestone 0.14.0 on Dec 6, 2016
6. MarcoFalke added this to the milestone 0.15.0 on Jan 27, 2017
7. MarcoFalke removed this from the milestone 0.14.0 on Jan 27, 2017
8. 
   achow101 commented at 7:44 pm on July 14, 2017: member

   This is still a problem. Any fixes yet? @theuni
9. MarcoFalke added this to the milestone Future on Aug 1, 2017
10. MarcoFalke removed this from the milestone 0.15.0 on Aug 1, 2017
11. 
    fanquake commented at 1:49 am on August 9, 2017: member

    Was this fixed by #10977 ?
12. 
    achow101 commented at 2:15 am on August 9, 2017: member

    Looks fixed to me.
13. fanquake closed this on Aug 9, 2017

    ---

14. MarcoFalke removed this from the milestone Future on Aug 12, 2018
15. DrahtBot locked this on Sep 8, 2021

Contributors
TheBlueMatt theuni achow101 fanquake

Labels
Tests