Jeffro

I personally would just omit the `r` variable and do `if (tools::wallet2::...) { ... } else { ... }` for clarity and so someone doesn't recreate the same issue of...

Will that `MDB_FIXEDSIZE` change affect the behavior in Monero as-is?

> Some testing with asan+tsan was also performed on the unit tests and monerod Curious, what results did that produce?

@vtnerd Thanks for taking the time to write that all out engage with this! I'll try to answer all of it > Does this support out-of-order fields? What about unknown...

Looks like some `std::deque` ran out of space. The error is thrown [in this file](https://gcc.gnu.org/onlinedocs/gcc-4.6.2/libstdc++/api/a00858_source.html) at line 832. Edit 1: The only `std::deque` declared in `wallet2.h` is the `m_blockchain` field...

@plowsof This didn't happen to be a multisig wallet, did it?

> has 0 txs after the one the user requests to scan, then scan_tx will continue to completion. Doesn't this still give away to the untrusted daemon which transaction you're...

Sounds possibly related to the issue @vtnerd is working on ironing out: https://github.com/monero-project/monero/pull/8592

Put it in a `.tar.gz` and upload it somewhere and share the link

Nope! That's not in my plans. Would you put it in #8540?