wallet: Check specified wallet exists before migration

The previous error message for non-existent wallets of "Already a descriptor wallet" is misleading. Return a more specific error when a non-existent wallet is specified.
AreaLayer · Nov 13, 2024 · f42ec0f · f42ec0f
1 parent c9e67e2
commit f42ec0f
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp
@@ -4407,6 +4407,9 @@ util::Result<MigrationResult> MigrateLegacyToDescriptor(const std::string& walle
         if (!wallet_path) {
             return util::Error{util::ErrorString(wallet_path)};
         }
+        if (!fs::exists(*wallet_path)) {
+            return util::Error{_("Error: Wallet does not exist")};
+        }
         if (!IsBDBFile(BDBDataFile(*wallet_path))) {
             return util::Error{_("Error: This wallet is already a descriptor wallet")};
         }