diff --git a/manager/lamptest/lamptest.cpp b/manager/lamptest/lamptest.cpp
index f91a085..f7f9e90 100644
--- a/manager/lamptest/lamptest.cpp
+++ b/manager/lamptest/lamptest.cpp
@@ -246,7 +246,7 @@
 void LampTest::timeOutHandler()
 {
     // set the Asserted property of lamp test to false
-    if (!groupObj)
+    if (groupObj == nullptr)
     {
         lg2::error("the Group object is nullptr");
         throw std::runtime_error("the Group object is nullptr");
diff --git a/manager/manager.cpp b/manager/manager.cpp
index 3dfa92a..df65426 100644
--- a/manager/manager.cpp
+++ b/manager/manager.cpp
@@ -273,7 +273,7 @@
         lg2::debug("De-Asserting LED, NAME = {NAME}, ACTION = {ACTION}", "NAME",
                    it.name, "ACTION", it.action);
         if (drivePhysicalLED(objPath, Layout::Action::Off, it.dutyOn,
-                             it.period))
+                             it.period) != 0)
         {
             failedLedsDeAssert.insert(it);
         }
@@ -284,7 +284,7 @@
         std::string objPath = std::string(phyLedPath) + it.name;
         lg2::debug("Asserting LED, NAME = {NAME}, ACTION = {ACTION}", "NAME",
                    it.name, "ACTION", it.action);
-        if (drivePhysicalLED(objPath, it.action, it.dutyOn, it.period))
+        if (drivePhysicalLED(objPath, it.action, it.dutyOn, it.period) != 0)
         {
             failedLedsAssert.insert(it);
         }