commit 35208459698f634f399f859f33e808d1a00946e5
author Patrick Williams <patrick@stwcx.xyz> Fri Jun 24 06:12:15 2022 -0500
committer Patrick Williams <patrick@stwcx.xyz> Fri Jun 24 06:12:15 2022 -0500
tree 213a87dcc1af797640080bc320e2c5f96cc317a6
parent 7b7273ebdc50054fe9c2140252b439e168e56c63

appcommands: handle system result for sled-cycle

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib62cc7bae69e44f5fda40c846ec722d1fc0e652b

src/appcommands.cpp

diff --git a/src/appcommands.cpp b/src/appcommands.cpp
index ce5c13b..a4aa3cc 100644
--- a/src/appcommands.cpp
+++ b/src/appcommands.cpp
@@ -136,21 +136,26 @@
 {
     uint8_t* req = reinterpret_cast<uint8_t*>(request);
     std::string mfrTest = "sled-cycle";
+    ipmi_ret_t rc = IPMI_CC_OK;
 
     if (!memcmp(req, mfrTest.data(), mfrTest.length()) &&
         (*data_len == mfrTest.length()))
     {
         /* sled-cycle the BMC */
-        system("/usr/sbin/power-util sled-cycle");
+        auto ret = system("/usr/sbin/power-util sled-cycle");
+        if (ret)
+        {
+            rc = IPMI_CC_UNSPECIFIED_ERROR;
+        }
     }
     else
     {
-        return IPMI_CC_SYSTEM_INFO_PARAMETER_NOT_SUPPORTED;
+        rc = IPMI_CC_SYSTEM_INFO_PARAMETER_NOT_SUPPORTED;
     }
 
     *data_len = 0;
 
-    return IPMI_CC_OK;
+    return rc;
 }
 
 //----------------------------------------------------------------------