starts_with() is declared as returning bool, but the null-argument guard returned NULL (a pointer cast to bool) instead of false. While harmless on most platforms where NULL and false both have value zero, it is technically incorrect.
Signed-off-by: Heinrich Schuchardt <[email protected]> --- v2: new patch --- lib/efi_loader/smbiosdump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader/smbiosdump.c b/lib/efi_loader/smbiosdump.c index d7f2ba30a95..974728a43af 100644 --- a/lib/efi_loader/smbiosdump.c +++ b/lib/efi_loader/smbiosdump.c @@ -199,7 +199,7 @@ static u16 *skip_whitespace(u16 *pos) static bool starts_with(u16 *string, u16 *keyword) { if (!string || !keyword) - return NULL; + return false; for (; *keyword; ++string, ++keyword) { if (*string != *keyword) -- 2.53.0
