From: Ulf Magnusson Date: Sun, 8 Oct 2017 17:35:45 +0000 (+0200) Subject: kconfig: Fix expr_free() E_NOT leak X-Git-Url: https://jxself.org/git/?a=commitdiff_plain;h=c1c9e127824bb32ed33b93898eba5e9d9926899c;p=carl9170fw.git kconfig: Fix expr_free() E_NOT leak Only the E_NOT operand and not the E_NOT node itself was freed, due to accidentally returning too early in expr_free(). Outline of leak: switch (e->type) { ... case E_NOT: expr_free(e->left.expr); return; ... } *Never reached, 'e' leaked* free(e); Fix by changing the 'return' to a 'break'. Summary from Valgrind on 'menuconfig' (ARCH=x86) before the fix: LEAK SUMMARY: definitely lost: 44,448 bytes in 1,852 blocks ... Summary after the fix: LEAK SUMMARY: definitely lost: 1,608 bytes in 67 blocks ... Signed-off-by: Ulf Magnusson Signed-off-by: Masahiro Yamada Signed-off-by: Christian Lamparter --- diff --git a/config/expr.c b/config/expr.c index 8cee597..ee23ebc 100644 --- a/config/expr.c +++ b/config/expr.c @@ -113,7 +113,7 @@ void expr_free(struct expr *e) break; case E_NOT: expr_free(e->left.expr); - return; + break; case E_EQUAL: case E_GEQ: case E_GTH: