diff options
| author | Naveen Kumar Chaudhary <[email protected]> | 2026-07-10 20:52:15 +0530 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-07-23 15:16:09 -0600 |
| commit | f7b525a8bc5396b84cf873bdfa5750731de398e9 (patch) | |
| tree | f655f6276c3f82d6871ceda990a6ca0487afc812 | |
| parent | 0ccd0a939d7d20c5a754a88c52450533932851a1 (diff) | |
cmd: mbr: return false instead of NULL from bool found_key()
found_key() is declared to return bool but returns NULL when strdup()
fails. NULL is a pointer constant; while it happens to convert to
zero (i.e. false) it is a type mismatch that trips stricter
compilers/static analysers.
Return false to match the declared return type.
Signed-off-by: Naveen Kumar Chaudhary <[email protected]>
| -rw-r--r-- | cmd/mbr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/mbr.c b/cmd/mbr.c index 7fe6c9e103a..b282b68d50e 100644 --- a/cmd/mbr.c +++ b/cmd/mbr.c @@ -73,7 +73,7 @@ static bool found_key(const char *str, const char *key) strcopy = strdup(str); if (!strcopy) - return NULL; + return false; s = strcopy; while (s) { |