From 5c13495a6c7cfd6fbd9c289c4280c24770293701 Mon Sep 17 00:00:00 2001 From: Gmer4Lfe Date: Wed, 20 May 2026 17:55:18 -0400 Subject: [PATCH] Add missing acquire_lock calls to arr_sync, bulk_permissions_repair, container_data_export MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit arr_sync.sh had acquire_lock documented in OPERATIONAL SAFEGUARDS header but the call was never present in the code — two sync instances could run simultaneously. bulk_permissions_repair.sh and container_data_export.sh had no lock at all. All three are now consistent with every other script in Media/ and Tools/. --- Media/arr_sync.sh | 2 ++ Tools/bulk_permissions_repair.sh | 2 ++ Tools/container_data_export.sh | 2 ++ 3 files changed, 6 insertions(+) diff --git a/Media/arr_sync.sh b/Media/arr_sync.sh index d60962e..2726da7 100755 --- a/Media/arr_sync.sh +++ b/Media/arr_sync.sh @@ -162,6 +162,8 @@ if [[ "$EUID" -ne 0 ]]; then exit 1 fi +acquire_lock + for _tool in curl jq; do if ! command -v "$_tool" >/dev/null 2>&1; then error "$_tool not found — required for arr API calls" diff --git a/Tools/bulk_permissions_repair.sh b/Tools/bulk_permissions_repair.sh index bc29783..a7a97bd 100644 --- a/Tools/bulk_permissions_repair.sh +++ b/Tools/bulk_permissions_repair.sh @@ -96,6 +96,8 @@ if [[ "$EUID" -ne 0 ]]; then exit 1 fi +acquire_lock + validate_unraid_cmd \ "/usr/local/emhttp/plugins/dynamix/scripts/notify" \ "" "" \ diff --git a/Tools/container_data_export.sh b/Tools/container_data_export.sh index eac59c8..f352ac1 100644 --- a/Tools/container_data_export.sh +++ b/Tools/container_data_export.sh @@ -86,6 +86,8 @@ if [[ "$EUID" -ne 0 ]]; then exit 1 fi +acquire_lock + validate_unraid_cmd \ "/usr/local/emhttp/plugins/dynamix/scripts/notify" \ "" "" \