From b9f1e211fe167855bbd971fe1b4b7a805e55ba19 Mon Sep 17 00:00:00 2001
From: gg <chown_tee@proton.me>
Date: Sun, 18 Jan 2026 22:21:45 -0500
Subject: [PATCH] fix race condition when starting two instances at once

---
 src/libwalletqt/Wallet.cpp | 8 +++++++-
 src/libwalletqt/Wallet.h   | 1 +
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/libwalletqt/Wallet.cpp b/src/libwalletqt/Wallet.cpp
index 79d9ee01..3cba2857 100644
--- a/src/libwalletqt/Wallet.cpp
+++ b/src/libwalletqt/Wallet.cpp
@@ -68,7 +68,7 @@ Wallet::Wallet(Monero::Wallet *wallet, QObject *parent)
     m_coinsModel = new CoinsModel(this, m_coins);
 
     if (this->status() == Status_Ok) {
-        startRefreshThread();
+        // startRefreshThread(); // Moved to startRefresh()
 
         // Store the wallet every 2 minutes
         m_storeTimer->start(2 * 60 * 1000);
@@ -481,6 +481,7 @@ void Wallet::initAsync(const QString &daemonAddress, bool trustedDaemon, quint64
 // #################### Synchronization (Refresh) ####################
 
 void Wallet::startRefresh() {
+    startRefreshThread();
     m_refreshEnabled = true;
     m_refreshNow = true;
 }
@@ -508,6 +509,11 @@ void Wallet::updateNetworkStatus() {
 
 void Wallet::startRefreshThread()
 {
+    bool expected = false;
+    if (!m_refreshThreadStarted.compare_exchange_strong(expected, true)) {
+        return;
+    }
+
     const auto future = m_scheduler.run([this] {
         // Beware! This code does not run in the GUI thread.
 
diff --git a/src/libwalletqt/Wallet.h b/src/libwalletqt/Wallet.h
index 69096654..7498df47 100644
--- a/src/libwalletqt/Wallet.h
+++ b/src/libwalletqt/Wallet.h
@@ -546,6 +546,7 @@ private:
     std::atomic<bool> m_rangeSyncActive{false};
     std::atomic<bool> m_syncPaused{false};
     std::atomic<int64_t> m_lastRefreshTime{0};
+    std::atomic<bool> m_refreshThreadStarted{false};
 };
 
 #endif // FEATHER_WALLET_H
-- 
2.52.0