diff options
author | Andrew Chow <github@achow101.com> | 2022-11-28 18:27:27 -0500 |
---|---|---|
committer | Andrew Chow <github@achow101.com> | 2022-11-29 19:31:27 -0500 |
commit | 86ef7b3c7be84e4183098f448c77ecc9ea7367ab (patch) | |
tree | 5800ff35e4e6d19a65b94fc9208f986e92dd327d /src/wallet/wallet.cpp | |
parent | 9f650062fc4350d8c50ac8890cddd4d99f20f895 (diff) | |
download | bitcoin-86ef7b3c7be84e4183098f448c77ecc9ea7367ab.tar.xz |
wallet: Avoid null pointer deref when cleaning up migratewallet
If migratewallet fails, we do a cleanup which removes the watchonly and
solvables wallets if they were created. However, if they were not, their
pointers are nullptr and we don't check for that, which causes a
segfault during the cleanup. So check that they aren't nullptr before
cleaning them up.
Diffstat (limited to 'src/wallet/wallet.cpp')
-rw-r--r-- | src/wallet/wallet.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp index e0f1655ab7..81c79037c6 100644 --- a/src/wallet/wallet.cpp +++ b/src/wallet/wallet.cpp @@ -4093,8 +4093,8 @@ util::Result<MigrationResult> MigrateLegacyToDescriptor(std::shared_ptr<CWallet> // Make list of wallets to cleanup std::vector<std::shared_ptr<CWallet>> created_wallets; - created_wallets.push_back(std::move(res.watchonly_wallet)); - created_wallets.push_back(std::move(res.solvables_wallet)); + if (res.watchonly_wallet) created_wallets.push_back(std::move(res.watchonly_wallet)); + if (res.solvables_wallet) created_wallets.push_back(std::move(res.solvables_wallet)); // Get the directories to remove after unloading for (std::shared_ptr<CWallet>& w : created_wallets) { |