Fixes: #31672 avoids a deliberate crash on targets which don't suppor… (

#32058) * Fixes: #31672 avoids a deliberate crash on targets which don't support real-time clock * Added clearer error message about checking EVSE certification requirements.
project-chip · Feb 16, 2024 · 5dfaed4 · 5dfaed4
1 parent d116c7c
commit 5dfaed4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/examples/energy-management-app/energy-management-common/src/EnergyEvseDelegateImpl.cpp b/examples/energy-management-app/energy-management-common/src/EnergyEvseDelegateImpl.cpp
@@ -1474,7 +1474,11 @@ CHIP_ERROR GetEpochTS(uint32_t & chipEpoch)
      * This should not be certifiable since getting time is a Mandatory
      * feature of EVSE Cluster
      */
-    VerifyOrDie(err != CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE);
+    if (err == CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE)
+    {
+        ChipLogError(Zcl, "Platform does not support GetClock_RealTimeMS. Check EVSE certification requirements!");
+        return err;
+    }
 
     if (err != CHIP_NO_ERROR)
     {