From 5807555f9a0afb94ad92535dd2933ee4dbd25088 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Wed, 11 May 2011 15:57:07 -0700 Subject: [PATCH] Don't try to compute I/O for empty sub buses. I am not sure if the sub bus being 0 is a problem, or if the assumption there has to be at least one non empty link is just wrong. It certainly does not hurt to add a small consistency check in either case. Change-Id: I098446deef96a8baae26a7ca1ddd96e626a06dc5 Signed-off-by: Stefan Reinauer Reviewed-on: http://review.coreboot.org/693 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich --- src/devices/device_util.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/devices/device_util.c b/src/devices/device_util.c index 9081a36ea2..84fb11505c 100644 --- a/src/devices/device_util.c +++ b/src/devices/device_util.c @@ -583,6 +583,8 @@ void search_bus_resources(struct bus *bus, unsigned long type_mask, if (subbus->link_num == IOINDEX_SUBTRACTIVE_LINK(res->index)) break; + if (!subbus) /* Why can subbus be NULL? */ + break; search_bus_resources(subbus, type_mask, type, search, gp); continue;