Patch "perf expr: Initialize is_test value in expr__ctx_new()" has been added to the 6.12-stable tree

[Sasha Levin <sashal@xxxxxxxxxx>]

This is a note to let you know that I've just added the patch titled

    perf expr: Initialize is_test value in expr__ctx_new()

to the 6.12-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     perf-expr-initialize-is_test-value-in-expr__ctx_new.patch
and it can be found in the queue-6.12 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 4e2df0c82741b51ef4a31afef51524ea6bee9c55
Author: Levi Yun <yeoreum.yun@xxxxxxx>
Date:   Fri Nov 8 14:34:25 2024 +0000

    perf expr: Initialize is_test value in expr__ctx_new()
    
    [ Upstream commit 1d18ebcfd302a2005b83ae5f13df223894d19902 ]
    
    When expr_parse_ctx is allocated by expr_ctx_new(),
    expr_scanner_ctx->is_test isn't initialize, so it has garbage value.
    this can affects the result of expr__parse() return when it parses
    non-exist event literal according to garbage value.
    
    Use calloc instead of malloc in expr_ctx_new() to fix this.
    
    Fixes: 3340a08354ac286e ("perf pmu-events: Fix testing with JEVENTS_ARCH=all")
    Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>
    Reviewed-by: James Clark <james.clark@xxxxxxxxxx>
    Signed-off-by: Levi Yun <yeoreum.yun@xxxxxxx>
    Cc: Mark Rutland <mark.rutland@xxxxxxx>
    Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
    Link: https://lore.kernel.org/r/20241108143424.819126-1-yeoreum.yun@xxxxxxx
    Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/tools/perf/util/expr.c b/tools/perf/util/expr.c
index b2536a59c44e6..90c6ce2212e4f 100644
--- a/tools/perf/util/expr.c
+++ b/tools/perf/util/expr.c
@@ -288,7 +288,7 @@ struct expr_parse_ctx *expr__ctx_new(void)
 {
 	struct expr_parse_ctx *ctx;
 
-	ctx = malloc(sizeof(struct expr_parse_ctx));
+	ctx = calloc(1, sizeof(struct expr_parse_ctx));
 	if (!ctx)
 		return NULL;
 
@@ -297,9 +297,6 @@ struct expr_parse_ctx *expr__ctx_new(void)
 		free(ctx);
 		return NULL;
 	}
-	ctx->sctx.user_requested_cpu_list = NULL;
-	ctx->sctx.runtime = 0;
-	ctx->sctx.system_wide = false;
 
 	return ctx;
 }