kconfig: fix infinite loop when expanding a macro at the end of file
authorMasahiro Yamada <masahiroy@kernel.org>
Fri, 2 Feb 2024 15:57:59 +0000 (00:57 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Mon, 19 Feb 2024 09:20:40 +0000 (18:20 +0900)
A macro placed at the end of a file with no newline causes an infinite
loop.

[Test Kconfig]
  $(info,hello)
  \ No newline at end of file

I realized that flex-provided input() returns 0 instead of EOF when it
reaches the end of a file.

Fixes: 104daea149c4 ("kconfig: reference environment variables directly and remove 'option env='")
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
scripts/kconfig/lexer.l

index f93b535a080c00f03b6a08fc09d7acb9e70abb2c..5f1bc33203073ec7748966f96fce7afae728a854 100644 (file)
@@ -303,8 +303,11 @@ static char *expand_token(const char *in, size_t n)
        new_string();
        append_string(in, n);
 
-       /* get the whole line because we do not know the end of token. */
-       while ((c = input()) != EOF) {
+       /*
+        * get the whole line because we do not know the end of token.
+        * input() returns 0 (not EOF!) when it reachs the end of file.
+        */
+       while ((c = input()) != 0) {
                if (c == '\n') {
                        unput(c);
                        break;