ci: failure in Windows cross-test #32291

1. 
   fanquake commented at 10:48 am on April 17, 2025: member

   https://github.com/bitcoin/bitcoin/actions/runs/14511726586/job/40715228086#step:6:10

   0Running with -sanity-check option, output is being suppressed as benchmark results will be useless.
   1terminate called after throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
   2  what():  filesystem error: cannot remove all: The process cannot access the file because it is being used by another process [C:\Users\RUNNER~1\AppData\Local\Temp\test_common bitcoin\WalletMigration\9ab4a7c9fd08639214fd] [C:\Users\RUNNER~1\AppData\Local\Temp\test_common bitcoin\WalletMigration\9ab4a7c9fd08639214fd\regtest\wallet.dat]
   3Error: Process completed with exit code 3.
   

2. fanquake added the label Windows on Apr 17, 2025
3. fanquake added the label Tests on Apr 17, 2025
4. 
   fanquake commented at 1:55 pm on April 17, 2025: member

   https://github.com/bitcoin/bitcoin/actions/runs/14515873780/job/40726323869
5. 
   hebasto commented at 4:30 pm on April 17, 2025: member

   Reminds #27352.
6. 
   maflcko commented at 6:10 am on April 18, 2025: member

   The error message is a bit confusing, because the benchmarks should be a single process? Thus, it seems odd that another process was accessing the wallet file.

   I don’t have Windows to debug this, so I guess it is best to temporarily revert. A follow-up can fix the underlying issue and enable the CI check again.

7. maflcko added this to the milestone 30.0 on Apr 18, 2025
8. 
   hebasto commented at 10:58 am on April 19, 2025: member

   The failure in the WalletMigration migration benchmark is not Windows-specific.

   On Ubuntu 24.04:

   0$ ./build/bin/bench_bitcoin -filter=WalletMigration -min-time=100000
   1...
   2Segmentation fault (core dumped)
   

   Here is the backtrace:

    0(lldb) bt
    1error: bench_bitcoin 0x00067927: DW_TAG_member '_M_local_buf' refers to type 0x00000000000b3ba7 which extends beyond the bounds of 0x0006791d
    2* thread [#1](/bitcoin-bitcoin/1/), name = 'b-test', stop reason = signal SIGSEGV: address not mapped to object (fault address: 0xc8)
    3  * frame [#0](/bitcoin-bitcoin/0/): 0x00005555556a3f33 bench_bitcoin`std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>::basic_string(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> const&) [inlined] std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>::length(this=<unavailable>) const at basic_string.h:1079:16
    4    frame [#1](/bitcoin-bitcoin/1/): 0x00005555556a3f33 bench_bitcoin`std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>::basic_string(this="", __str=0x00000000000000c0) at basic_string.h:551:62
    5    frame [#2](/bitcoin-bitcoin/2/): 0x0000555555b7d544 bench_bitcoin`wallet::MigrateLegacyToDescriptor(std::shared_ptr<wallet::CWallet>, std::__cxx11::basic_string<char, std::char_traits<char>, secure_allocator<char>> const&, wallet::WalletContext&, bool) at wallet.cpp:4483:59
    6    frame [#3](/bitcoin-bitcoin/3/): 0x0000555555759f65 bench_bitcoin`ankerl::nanobench::Bench& ankerl::nanobench::Bench::run<wallet::WalletMigration(ankerl::nanobench::Bench&)::'lambda'()>(wallet::WalletMigration(ankerl::nanobench::Bench&)::'lambda'()&&) (.isra.0) at wallet_migration.cpp:67:138
    7    frame [#4](/bitcoin-bitcoin/4/): 0x000055555575b325 bench_bitcoin`wallet::WalletMigration(ankerl::nanobench::Bench&) at wallet_migration.cpp:66:38
    8    frame [#5](/bitcoin-bitcoin/5/): 0x00005555556a22b3 bench_bitcoin`benchmark::BenchRunner::RunAll(benchmark::Args const&) at std_function.h:591:9
    9    frame [#6](/bitcoin-bitcoin/6/): 0x000055555566efa0 bench_bitcoin`main at bench_bitcoin.cpp:150:39
   10    frame [#7](/bitcoin-bitcoin/7/): 0x00007ffff762a1ca libc.so.6`__libc_start_call_main(main=(bench_bitcoin`main), argc=3, argv=0x00007fffffffd808) at libc_start_call_main.h:58:16
   11    frame [#8](/bitcoin-bitcoin/8/): 0x00007ffff762a28b libc.so.6`__libc_start_main_impl(main=(bench_bitcoin`main), argc=3, argv=0x00007fffffffd808, init=<unavailable>, fini=<unavailable>, rtld_fini=<unavailable>, stack_end=0x00007fffffffd7f8) at libc-start.c:360:3
   12    frame [#9](/bitcoin-bitcoin/9/): 0x000055555569b115 bench_bitcoin`_start + 37
   

   cc @achow101 @furszy

9. hebasto added the label Wallet on Apr 19, 2025
10. maflcko removed the label Windows on Apr 19, 2025
11. 
    furszy commented at 9:59 pm on April 23, 2025: member

    Can be closed.
12. 
    hebasto commented at 10:24 pm on April 23, 2025: member

    Fixed in #32309.
13. hebasto closed this on Apr 23, 2025

    ---

Contributors
fanquake hebasto maflcko furszy

Labels
Wallet Tests

Milestone
30.0