From 616fe0555226c8144031f08ef0e71590734086a4 Mon Sep 17 00:00:00 2001
From: Eric Sesterhenn <eric.sesterhenn@x41-dsec.de>
Date: Tue, 3 Feb 2026 08:52:31 +0100
Subject: [PATCH] Make sure we always return a value in
 NodeDB::restorePreferences()

In case FScom is not defined there is no return statement. This
moves the return outside of the ifdef to make sure a defined
value is returned.
---
 src/mesh/NodeDB.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesh/NodeDB.cpp b/src/mesh/NodeDB.cpp
index 375bc76e341..bd40a35be26 100644
--- a/src/mesh/NodeDB.cpp
+++ b/src/mesh/NodeDB.cpp
@@ -2171,8 +2171,8 @@ bool NodeDB::restorePreferences(meshtastic_AdminMessage_BackupLocation location,
     } else if (location == meshtastic_AdminMessage_BackupLocation_SD) {
         // TODO: After more mainline SD card support
     }
-    return success;
 #endif
+    return success;
 }
 
 /// Record an error that should be reported via analytics