If we have it, why not use it?
Signed-off-by: Patrick Georgi <[email protected]>
---
payloads/libpayload/include/assert.h | 3 ++-
payloads/libpayload/libc/lib.c | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/payloads/libpayload/include/assert.h
b/payloads/libpayload/include/assert.h
index fda4bc0..9968504 100644
--- a/payloads/libpayload/include/assert.h
+++ b/payloads/libpayload/include/assert.h
@@ -28,6 +28,7 @@
*/
#include <stdlib.h>
+#include <stdio.h>
// assert's existence depends on NDEBUG state on _last_ inclusion of assert.h,
// so don't guard this against double-includes.
@@ -35,5 +36,5 @@
// Heisenbugs appear if statement has side-effects. This could be worked
around but does the standard allow for that?
#define assert(statement)
#else
-#define assert(statement) if ((statement) == 0) { printf("assertion failed in
file %s, function %s(), line %s\n", __FILE__, __FUNCTION__, __LINE__); abort();
}
+#define assert(statement) if ((statement) == 0) { fprintf(stderr, "assertion
failed in file %s, function %s(), line %s\n", __FILE__, __FUNCTION__,
__LINE__); abort(); }
#endif
diff --git a/payloads/libpayload/libc/lib.c b/payloads/libpayload/libc/lib.c
index baccc07..c8539cb 100644
--- a/payloads/libpayload/libc/lib.c
+++ b/payloads/libpayload/libc/lib.c
@@ -109,7 +109,7 @@ u8 hex2bin(u8 h)
*/
void fatal(const char *msg)
{
- printf("%s",msg);
+ fprintf(stderr, "%s",msg);
halt();
}
--
1.7.1
--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot
