2023-06-16 01:55:06

by Tiezhu Yang

[permalink] [raw]
Subject: [PATCH 2/2] module: Make is_valid_name() return bool

The return value of is_valid_name() is true or false,
so change its type to reflect that.

Signed-off-by: Tiezhu Yang <[email protected]>
---
scripts/mod/modpost.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
index d4531d0..e079418 100644
--- a/scripts/mod/modpost.c
+++ b/scripts/mod/modpost.c
@@ -1122,12 +1122,12 @@ static int secref_whitelist(const struct sectioncheck *mismatch,
* only by merging __exit and __init sections into __text, bloating
* the kernel (which is especially evil on embedded platforms).
*/
-static inline int is_valid_name(struct elf_info *elf, Elf_Sym *sym)
+static inline bool is_valid_name(struct elf_info *elf, Elf_Sym *sym)
{
const char *name = elf->strtab + sym->st_name;

if (!name || !strlen(name))
- return 0;
+ return false;
return !is_mapping_symbol(name);
}

--
2.1.0