commit 56127ef8dcdb8705e58420c6b99941c88791aec2
author Sunita Kumari <sunitax.kumari@intel.com> Wed Oct 12 05:01:20 2022 +0000
committer Sunita Kumari <sunitax.kumari@intel.com> Thu Oct 13 03:56:19 2022 +0000
tree 91ada686ebe10d93ec6bc07027dbe9e82180ebb3
parent aee6540154f2ac8ea510efacd2785868c1898a7d

[Fix] lpcsnoop service exit unsuccessfully

[Issue] when BMC code reset, the lpcsnoop service exit unsuccessfully.

[Root Cause] return exit code 1 when caught SIGTERM signal.

https://github.com/openbmc/phosphor-host-postd/blob/master/main.cpp#L368
https://github.com/openbmc/sdeventplus/blob/master/src/sdeventplus/event.hpp#L115

Tested:
Service exit successfully

Signed-off-by: Sunita Kumari <sunitax.kumari@intel.com>
Change-Id: I47cc6d31742f1595231fe22e345ea955ad41c377

main.cpp

diff --git a/main.cpp b/main.cpp
index 57897f9..c294a13 100644
--- a/main.cpp
+++ b/main.cpp
@@ -320,7 +320,6 @@
 #endif
 
 #ifndef ENABLE_IPMI_SNOOP
-    int rc = 0;
 
     bool deferSignals = true;
 
@@ -365,7 +364,7 @@
                 ;
 
             // Run and never timeout
-            rc = event.run(std::nullopt);
+            event.run(std::nullopt);
         }
     }
     catch (const std::exception& e)
@@ -378,6 +377,6 @@
         close(postFd);
     }
 
-    return rc;
+    return 0;
 #endif
 }