sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

libpayload: Fix the format string of the assert macro.

Browse Source 
The assert macro in libpayload was using a format string which printed the
line number with %s. The line number came from the __LINE__ predefined macro
which resolves to an integer constant.

Change-Id: I0e00d42a1569802137cf440af3061d7f397fdd27
Signed-off-by: Gabe Black <gabeblack@google.com>
Reviewed-on: http://review.coreboot.org/1730
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
This commit is contained in:
Gabe Black
2012-10-02 00:28:33 -07:00
committed by Stefan Reinauer
parent 3b84086e3d
commit 5c27e82073
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
payloads/libpayload/include/assert.h
View File

@ -36,5 +36,11 @@
// Heisenbugs appear if statement has side-effects. This could be worked around but does the standard allow for that? // Heisenbugs appear if statement has side-effects. This could be worked around but does the standard allow for that?
#define assert(statement) #define assert(statement)
#else #else
#define assert(statement) if ((statement) == 0) { fprintf(stderr, "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 %d\n", \
__FILE__, __FUNCTION__, __LINE__); \
abort(); \
}
#endif #endif