[lkml]   [2019]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
Subject[PATCH AUTOSEL 3.18 10/19] kconfig: fix memory leak when EOF is encountered in quotation
From: Masahiro Yamada <>

[ Upstream commit fbac5977d81cb2b2b7e37b11c459055d9585273c ]

An unterminated string literal followed by new line is passed to the
parser (with "multi-line strings not supported" warning shown), then
handled properly there.

On the other hand, an unterminated string literal at end of file is
never passed to the parser, then results in memory leak.

[Test Code]

----------(Kconfig begin)----------
source ""

config A
bool "a"
-----------(Kconfig end)-----------

--------( begin)--------
config B
bool "b\No new line at end of file
---------( end)---------

[Summary from Valgrind]

Before the fix:

definitely lost: 16 bytes in 1 blocks

After the fix:

definitely lost: 0 bytes in 0 blocks

Eliminate the memory leak path by handling this case. Of course, such
a Kconfig file is wrong already, so I will add an error message later.

Signed-off-by: Masahiro Yamada <>
Signed-off-by: Sasha Levin <>
scripts/kconfig/zconf.l | 2 ++
1 file changed, 2 insertions(+)

diff --git a/scripts/kconfig/zconf.l b/scripts/kconfig/zconf.l
index 6c62d93b4ffb..485b19a461f8 100644
--- a/scripts/kconfig/zconf.l
+++ b/scripts/kconfig/zconf.l
@@ -180,6 +180,8 @@ n [A-Za-z0-9_]
<<EOF>> {
+ yylval.string = text;
+ return T_WORD_QUOTE;

 \ /
  Last update: 2019-01-08 20:37    [W:0.068 / U:1.832 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site