[PATCH] kconfig: default to zero if int/hex symbol lacks default property

Yoann Congal yoann.congal at smile.fr
Sun May 5 15:53:53 CEST 2024


From: Masahiro Yamada <masahiroy at kernel.org>

This is a cherry-pick from the kernel commit:
6262afa10ef7c (kconfig: default to zero if int/hex symbol lacks default property, 2023-11-26)

When a default property is missing in an int or hex symbol, it defaults
to an empty string, which is not a valid symbol value.

It results in an incorrect .config, and can also lead to an infinite
loop in scripting.

Use "0" for int and "0x0" for hex as a default value.

Signed-off-by: Masahiro Yamada <masahiroy at kernel.org>
Reviewed-by: Yoann Congal <yoann.congal at smile.fr>

Signed-off-by: Yoann Congal <yoann.congal at smile.fr>
---
Added context that was not in the upstream commit:
The infinite loop case happens with a configuration defined like this
(a hex config without a valid default value):
  config HEX_TEST
  	hex "Hex config without default"

And using:
  $ make oldconfig < /dev/null
  scripts/kconfig/conf  --oldconfig Kconfig
  *
  * General setup
  *

  Error in reading or end of file.

  Error in reading or end of file.
  Hex config without default (HEX_TEST) [] (NEW)

  Error in reading or end of file.
  Hex config without default (HEX_TEST) [] (NEW)
  # This loops forever

NB: Scripted config manipulation often call make with /dev/null as
stdin (Yocto recipe, CI build, ...)

This was discovered when working on Yocto bug:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=14136
---
 scripts/kconfig/symbol.c | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c
index 703b9b899ee..b4c51de3f20 100644
--- a/scripts/kconfig/symbol.c
+++ b/scripts/kconfig/symbol.c
@@ -343,7 +343,11 @@ void sym_calc_value(struct symbol *sym)
 
 	switch (sym->type) {
 	case S_INT:
+		newval.val = "0";
+		break;
 	case S_HEX:
+		newval.val = "0x0";
+		break;
 	case S_STRING:
 		newval = symbol_empty.curr;
 		break;
@@ -753,15 +757,17 @@ const char *sym_get_string_default(struct symbol *sym)
 		case yes: return "y";
 		}
 	case S_INT:
+		if (!str[0])
+			str = "0";
+		break;
 	case S_HEX:
-		return str;
-	case S_STRING:
-		return str;
-	case S_OTHER:
-	case S_UNKNOWN:
+		if (!str[0])
+			str = "0x0";
+		break;
+	default:
 		break;
 	}
-	return "";
+	return str;
 }
 
 const char *sym_get_string_value(struct symbol *sym)
-- 
2.39.2



More information about the U-Boot mailing list