[HICN-460] Check of parchashmap before creating iterator

Change-Id: I559e86e269026be601a5735f77bbcfac40f24694
Signed-off-by: Angelo Mantellini <angelo.mantellini@cisco.com>

1 files changed, 18 insertions, 16 deletions

diff --git a/hicn-light/src/hicn/core/mapme.c b/hicn-light/src/hicn/core/mapme.c
index af332ede2..a0a34e8ce 100644
--- a/hicn-light/src/hicn/core/mapme.c
+++ b/hicn-light/src/hicn/core/mapme.c
@@ -351,28 +351,30 @@ static bool mapme_setFacePending(const MapMe *mapme, const Name *name,
      * It is likely we cannot iterator and remove elements from the hashmap at
      * the same time, so we proceed in two steps
      */
-    NumberSet * conns = numberSet_Create();
+    if (parcHashMap_Size(TFIB(fibEntry)->nexthops) > 0) {
 
-    PARCIterator *it = parcHashMap_CreateKeyIterator(TFIB(fibEntry)->nexthops);
-    while (parcIterator_HasNext(it)) {
-      PARCUnsigned *cid = parcIterator_Next(it);
-      unsigned conn_id = parcUnsigned_GetUnsigned(cid);
-      numberSet_Add(conns, conn_id);
-    }
-    parcIterator_Release(&it);
+      NumberSet * conns = numberSet_Create();
+
+      PARCIterator *it = parcHashMap_CreateKeyIterator(TFIB(fibEntry)->nexthops);
+      while (parcIterator_HasNext(it)) {
+        PARCUnsigned *cid = parcIterator_Next(it);
+        unsigned conn_id = parcUnsigned_GetUnsigned(cid);
+        numberSet_Add(conns, conn_id);
+      }
+      parcIterator_Release(&it);
 
-    for (size_t i = 0; i < numberSet_Length(conns); i++) {
-      unsigned conn_id = numberSet_GetItem(conns, i);
-      PARCEventTimer *oldTimer = (PARCEventTimer *)mapmeTFIB_Get(TFIB(fibEntry), conn_id);
-      if (oldTimer)
+      for (size_t i = 0; i < numberSet_Length(conns); i++) {
+        unsigned conn_id = numberSet_GetItem(conns, i);
+        PARCEventTimer *oldTimer = (PARCEventTimer *)mapmeTFIB_Get(TFIB(fibEntry), conn_id);
+        if (oldTimer)
           parcEventTimer_Stop(oldTimer);
-      mapmeTFIB_Remove(TFIB(fibEntry), conn_id);
-    }
+        mapmeTFIB_Remove(TFIB(fibEntry), conn_id);
+      }
 
-    numberSet_Release(&conns);
+      numberSet_Release(&conns);
+    }
   }
 
-
   // NOTE
   // - at producer, send always true, we always send something reliably so we
   // set the timer.