From 75c4741d9a07b7648f45c524c03d9fd96c3b23ef Mon Sep 17 00:00:00 2001
From: Dan Paulat <dpaulat@gmail.com>
Date: Thu, 13 Feb 2025 22:42:02 -0600
Subject: [PATCH] Take a lock on the refresh timer mutex before checking
 whether refresh is enabled

---
 scwx-qt/source/scwx/qt/manager/radar_product_manager.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scwx-qt/source/scwx/qt/manager/radar_product_manager.cpp b/scwx-qt/source/scwx/qt/manager/radar_product_manager.cpp
index b51d5c70..ae9d0313 100644
--- a/scwx-qt/source/scwx/qt/manager/radar_product_manager.cpp
+++ b/scwx-qt/source/scwx/qt/manager/radar_product_manager.cpp
@@ -794,10 +794,10 @@ void RadarProductManagerImpl::RefreshDataSync(
       interval = kSlowRetryInterval_;
    }
 
+   std::unique_lock lock(providerManager->refreshTimerMutex_);
+
    if (providerManager->refreshEnabled_)
    {
-      std::unique_lock lock(providerManager->refreshTimerMutex_);
-
       logger_->debug(
          "[{}] Scheduled refresh in {:%M:%S}",
          providerManager->name(),