I’m using -txindex in my normal environment, when using payment requests without the client running, the client is started without any parameters and detects the missing -txindex and asks me Do you want to rebuild the block database now? answering no here crashes the client.
https://github.com/bitcoin/bitcoin/blob/master/src/init.cpp#L834 @sipa Can you check that part of the code, as most of it was done by you :).
Edit: Error is on Win7 x64 and it’s a BEX (Buffer Overflow Exception).
I tried to debug this, but failed badly… I get a segfault related to path.hpp, which is from Boost filesystem. This happens after 2013-11-14 20:09:45 Shutdown : done. I just see that the object contains the correct path to my testnet3 datadir and is unrelated to our datadir caching (as I disabled it for testing purposes).
Perhaps there is some global that get’s destructed after everything is already shutdown, dunno yet… @laanwj Any idea how I can nail this down further?
Edit: I’m going to try Boost 1.55.0 tomorrow and see if that helps ;).
Yes that’s probably an destruction order problem, do you get a traceback?
Otherwise I don’t know a good way to debug it except for selectively commenting out code.
Tested a minute ago, yes still happens with 0.9.1.
I found out that this code in path.hpp causes the segfault:
This line causes it:
DbEnv(0).remove(path.string().c_str(), 0);
https://github.com/bitcoin/bitcoin/blob/master/src/db.cpp#L49
I was able to prevent the crash by not using a boost::path there, but add a std::string to the class, init that with path.string() at the beginning and use that in the above call.
@laanwj Any idea for this? A use-after-free or something like that?
This is yet another shutdown order issue.
CDbEnv ideally shouldn’t be a global object that has the lifetime of the program. I remember this also came up for testing by @SergioDemianLerner #4688.
I tested this on Windows 8.1 x64 and Windows 10 TP with the 0.9, 0.10 releases, as well as the master branch (build via Gitian route).
To reproduce:
bitcoin-qt.exe, pointing to a new datadir (-datadir=xxx, -choosedatadir)bitcoin-qt.exe -txindex=1A similar event can be triggered and observed, when executing bitcoind.exe without rpcuser and rpcpassword defined, which also results in an application crash.
FWIW, I uploaded the crash report, including error dump files: http://bitwatch.co/uploads/AppCrash_bitcoin-qt_w10x64.zip (1.414 KB)
Running Process Monitor hints a buffer overflow during the shutdown:
This error does not occure, when using -disablewallet or a wallet-less build.
As follow up: it’s a memory access violation and I sort-of debugged it on an assembler level, where I saw that the crash occured when jumping to 0xFEEEFEEE + x, which indicates previously freed memory on Windows.
The hint about const std::string string() const { return string(codecvt()); } was golden, as this indeed pointed to root cause. Boost path uses a static initialized pointer internally, which appears to be deinitialized during shutdown, but if this is not done by the main thread, the pointer appears to be already deinitialized, even though there is still an attempted access later. An appearingly working solution was to force the initialization by the main thread.
