From d6a47b123a8ddf7db00a1fa267ff3431d2c1081b Mon Sep 17 00:00:00 2001 From: Badri Sampath Date: Fri, 6 May 2022 09:55:05 -0700 Subject: [PATCH] msm_perf: Handle out of bounds case KASAN seems to be flagging that bool would only be 1 byte while %du would want to write 4 bytes leading to out of bounds. Change-Id: I0e43b1103b882bbd61b5a0bef231efcc3d7d1e1b Signed-off-by: Badri Sampath --- drivers/soc/qcom/msm_performance.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/qcom/msm_performance.c b/drivers/soc/qcom/msm_performance.c index a2dcfac2f267..a6cbb8324e8f 100644 --- a/drivers/soc/qcom/msm_performance.c +++ b/drivers/soc/qcom/msm_performance.c @@ -927,7 +927,7 @@ static ssize_t set_core_ctl_register(struct kobject *kobj, bool old_val = core_ctl_register; int ret; - ret = sscanf(buf, "%du", &core_ctl_register); + ret = kstrtobool(buf, &core_ctl_register); if (ret < 0) { pr_err("msm_perf: getting new core_ctl_register failed, ret=%d\n", ret); return ret;