perfmon: fix coverity warning 28/34528/2
authorKlement Sekera <ksekera@cisco.com>
Tue, 16 Nov 2021 11:32:59 +0000 (12:32 +0100)
committerOle Tr�an <otroan@employees.org>
Tue, 16 Nov 2021 16:19:40 +0000 (16:19 +0000)
Check for possible hash lookup failure to avoid NULL dereference.

Type: fix
Fixes: e15c999c30
Signed-off-by: Klement Sekera <ksekera@cisco.com>
Change-Id: Ib806b4d124be26fbccf36fe9d19af1aec63f487b

src/plugins/perfmon/intel/bundle/iio_bw.c

index 9aaca42..66c550f 100644 (file)
@@ -126,9 +126,16 @@ format_stack_socket (u8 *s, va_list *va)
     }
 
   uword *pu = hash_get (h, root_bus.bus);
-  e = pool_elt_at_index (p, (index_t) pu[0]);
+  if (pu)
+    {
+      e = pool_elt_at_index (p, (index_t) pu[0]);
 
-  s = format (s, "IIO%u/%u", e->socket_id, e->iio_unit_id);
+      s = format (s, "IIO%u/%u", e->socket_id, e->iio_unit_id);
+    }
+  else
+    {
+      s = format (s, "[ERR: hash lookup for bus '%u' failed]", root_bus.bus);
+    }
   return s;
 }