commit 196e3089a212e99d0b2cd468d395deb7fda8307d
author Rekha Aparna <vrekhaaparna@ibm.com> Mon Sep 08 20:40:35 2025 -0500
committer Rekha Aparna <vrekhaaparna@ibm.com> Tue Sep 09 01:30:28 2025 -0500
tree 22ddc21ea5ea286c763197bc4f3f317594cd6645
parent b00b92affa78574fc608e21a1612db19592b2d6b

Set error code for isBackupAndRestoreRequired API

This commit updates isBackupAndRestoreRequired API to set error code in
case of error. This helps caller of API to take action based on the
error code returned from the API.

Change-Id: I78e1c287762c399880ed2d88c5833d9d6cf1701b
Signed-off-by: Rekha Aparna <vrekhaaparna@ibm.com>

vpd-manager/src/worker.cpp

diff --git a/vpd-manager/src/worker.cpp b/vpd-manager/src/worker.cpp
index 7a4ce00..55e6709 100644
--- a/vpd-manager/src/worker.cpp
+++ b/vpd-manager/src/worker.cpp
@@ -424,10 +424,20 @@
 
         setJsonSymbolicLink(systemJson);
 
-        if (isSystemVPDOnDBus() &&
-            jsonUtility::isBackupAndRestoreRequired(m_parsedJson))
+        if (isSystemVPDOnDBus())
         {
-            performBackupAndRestore(parsedVpdMap);
+            uint16_t l_errCode = 0;
+            if (jsonUtility::isBackupAndRestoreRequired(m_parsedJson,
+                                                        l_errCode))
+            {
+                performBackupAndRestore(parsedVpdMap);
+            }
+            else if (l_errCode)
+            {
+                logging::logMessage(
+                    "Failed to check if backup and restore required. Reason : " +
+                    vpdSpecificUtility::getErrCodeMsg(l_errCode));
+            }
         }
 
         // proceed to publish system VPD.