From 0edb29095124acf39f532909710fef4642378136 Mon Sep 17 00:00:00 2001 From: YAMAMOTO Takashi Date: Thu, 27 Jan 2022 13:22:37 +0900 Subject: [PATCH] symtab_findbyname: just retun NULL for NULL symtab The condition is not fatal at all. It's better to let the caller handle the failure. --- libs/libc/symtab/symtab_findbyname.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libs/libc/symtab/symtab_findbyname.c b/libs/libc/symtab/symtab_findbyname.c index d3bd0d01938..7efa80c5ead 100644 --- a/libs/libc/symtab/symtab_findbyname.c +++ b/libs/libc/symtab/symtab_findbyname.c @@ -60,6 +60,12 @@ symtab_findbyname(FAR const struct symtab_s *symtab, int cmp; #endif + if (symtab == NULL) + { + DEBUGASSERT(nsyms == 0); + return NULL; + } + #ifdef CONFIG_SYMTAB_DECORATED if (name[0] == '_') { @@ -67,7 +73,7 @@ symtab_findbyname(FAR const struct symtab_s *symtab, } #endif - DEBUGASSERT(symtab != NULL && name != NULL); + DEBUGASSERT(name != NULL); #ifdef CONFIG_SYMTAB_ORDEREDBYNAME while (low < high)