From 0eb37b04c98c39e30776fa267fb6af6678c9e366 Mon Sep 17 00:00:00 2001
From: Mathieu Kardous <mathieu.kardous@silabs.com>
Date: Tue, 9 Jan 2024 15:33:41 -0500
Subject: [PATCH] Revert "[Report Scheduler] Empty node pool handling (#31136)"

This reverts commit bd24a657d4aae54febad477cb558d22686438315.
---
 src/app/reporting/SynchronizedReportSchedulerImpl.cpp | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/app/reporting/SynchronizedReportSchedulerImpl.cpp b/src/app/reporting/SynchronizedReportSchedulerImpl.cpp
index 1620d9d70e6bbc..6524c3f01c5f33 100644
--- a/src/app/reporting/SynchronizedReportSchedulerImpl.cpp
+++ b/src/app/reporting/SynchronizedReportSchedulerImpl.cpp
@@ -176,9 +176,6 @@ void SynchronizedReportSchedulerImpl::TimerFired()
     Timestamp now   = mTimerDelegate->GetCurrentMonotonicTimestamp();
     bool firedEarly = true;
 
-    // If there are no handlers registered, no need to do anything.
-    VerifyOrReturn(mNodesPool.Allocated());
-
     mNodesPool.ForEachActiveObject([now, &firedEarly](ReadHandlerNode * node) {
         if (node->GetMinTimestamp() <= now)
         {
@@ -199,7 +196,8 @@ void SynchronizedReportSchedulerImpl::TimerFired()
         return Loop::Continue;
     });
 
-    if (firedEarly)
+    // If there are no handlers registers, no need to schedule the next report
+    if (mNodesPool.Allocated() && firedEarly)
     {
         // If we fired the timer early, we need to recalculate the next report timeout and reschedule the report
         Timeout timeout = Milliseconds32(0);