From 51781f04b686d677c4ebd7e6b35b229b746a3d96 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolan.mcnamara@collabora.com>
Date: Sat, 23 Aug 2025 15:30:46 +0100
Subject: [PATCH] cid#1660144 silence bogus Data race condition
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Change-Id: I023214c154ab3a44fca3b3b504c20b4670da2f99
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190444
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
---
 extensions/source/update/check/updatecheck.cxx | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/extensions/source/update/check/updatecheck.cxx b/extensions/source/update/check/updatecheck.cxx
index 36ca9bfbb757..a43f48c37522 100644
--- a/extensions/source/update/check/updatecheck.cxx
+++ b/extensions/source/update/check/updatecheck.cxx
@@ -934,6 +934,8 @@ UpdateCheck::shutdownThread(bool join)
 void
 UpdateCheck::enableAutoCheck(bool enable)
 {
+    // The (recursive) mutex is (at time of writing) always locked already here
+    std::scoped_lock aGuard(m_aMutex);
     if( enable )
     {
         m_updateCheckRunning = true;