[coreboot] New patch to review for coreboot: 835e0e6 mptable: Fix BUS type determination

[Sven Schnelle (svens@stackframe.org)]

Sven Schnelle (svens at stackframe.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1112

-gerrit

commit 835e0e6e494b719a8b66dd3cfd27278401cf9707
Author: Sven Schnelle <svens at stackframe.org>
Date:   Wed Jun 20 10:01:47 2012 +0200

    mptable: Fix BUS type determination
    
    Change-Id: I7268b35671f6629601fa3b2a589054b8c5da5d78
    Signed-off-by: Sven Schnelle <svens at stackframe.org>
---
 util/mptable/mptable.c |   19 +++++++++++++++++--
 1 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/util/mptable/mptable.c b/util/mptable/mptable.c
index 80d4742..1aabe17 100644
--- a/util/mptable/mptable.c
+++ b/util/mptable/mptable.c
@@ -50,6 +50,7 @@
 #include <unistd.h>
 #include <stdint.h>
 
+#define ARRAY_SIZE(_x) (sizeof(_x) / sizeof(_x[0]))
 #define SEP_LINE \
 "\n-------------------------------------------------------------------------------\n"
 
@@ -982,7 +983,7 @@ static void busEntry(void)
 {
 	char name[8];
 	BusEntry entry;
-
+	int i;
 	/* read it into local memory */
 	readEntry(&entry, sizeof(entry));
 
@@ -997,7 +998,21 @@ static void busEntry(void)
 	}
 
 	memset(name, '\0', sizeof(name));
-	strncpy(name, (char *)entry.busType, 6);
+	for(i = 0; i < 6; i++) {
+		switch(entry.busType[i]) {
+		case ' ':
+		case '\0':
+			break;
+		default:
+			name[i] = entry.busType[i];
+			break;
+		}
+	}
+
+	if (entry.busID > ARRAY_SIZE(busses)) {
+		fprintf(stderr, "busses array to small!\n");
+		exit(1);
+	}
 
 	busses[entry.busID] = lookupBusType(name);
 	printf("\tsmp_write_bus(mc, %d, \"", entry.busID);