[LinuxBIOS] r2679 - trunk/LinuxBIOSv2/util/lbtdump

svn at openbios.org svn at openbios.org
Sun May 20 19:10:17 CEST 2007


Author: uwe
Date: 2007-05-20 19:10:17 +0200 (Sun, 20 May 2007)
New Revision: 2679

Modified:
   trunk/LinuxBIOSv2/util/lbtdump/lbtdump.c
Log:
Here is a small fix to prevent a segmentation fault in lbtdump.

The format specifier in the printf statements have been changed from
%08lx to %08llx or similar where uint64_t are being displayed.

Signed-off-by: Ben Hewson <ben at hewson-venieri.com>
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>



Modified: trunk/LinuxBIOSv2/util/lbtdump/lbtdump.c
===================================================================
--- trunk/LinuxBIOSv2/util/lbtdump/lbtdump.c	2007-05-20 16:36:01 UTC (rev 2678)
+++ trunk/LinuxBIOSv2/util/lbtdump/lbtdump.c	2007-05-20 17:10:17 UTC (rev 2679)
@@ -110,30 +110,30 @@
 {
 	if (value >      1024ULL*1024*1024*1024*1024*1024) {
 		value /= 1024ULL*1024*1024*1024*1024*1024;
-		fprintf(stream, "%ldEB", value);
+		fprintf(stream, "%lldEB", value);
 	}
 	else if (value > 1024ULL*1024*1024*1024*1024) {
 		value /= 1024ULL*1024*1024*1024*1024;
-		fprintf(stream, "%ldPB", value);
+		fprintf(stream, "%lldPB", value);
 	}
 	else if (value > 1024ULL*1024*1024*1024) {
 		value /= 1024ULL*1024*1024*1024;
-		fprintf(stream, "%ldTB", value);
+		fprintf(stream, "%lldTB", value);
 	}
 	else if (value > 1024ULL*1024*1024) {
 		value /= 1024ULL*1024*1024;
-		fprintf(stream, "%ldGB", value);
+		fprintf(stream, "%lldGB", value);
 	}
 	else if (value > 1024ULL*1024) {
 		value /= 1024ULL*1024;
-		fprintf(stream, "%ldMB", value);
+		fprintf(stream, "%lldMB", value);
 	}
 	else if (value > 1024ULL) {
 		value /= 1024ULL;
-		fprintf(stream, "%ldKB", value);
+		fprintf(stream, "%lldKB", value);
 	}
 	else {
-		fprintf(stream, "%ldB", value);
+		fprintf(stream, "%lldB", value);
 	}
 }
 
@@ -156,7 +156,7 @@
 		default:
 		case 2: mem_type = "reserved"; break;
 		}
-		printf("0x%08lx - 0x%08lx %s (",
+		printf("0x%08llx - 0x%08llx %s (",
 			start, end, mem_type);
 		pretty_print_number(stdout, start);
 		printf(" - ");





More information about the coreboot mailing list