From b7125c369c4e5bde86d6688cbd1b7a87384d3d24 Mon Sep 17 00:00:00 2001 From: Antonio Borneo Date: Sat, 25 Jun 2022 12:40:24 +0200 Subject: [PATCH] target: esp_xtensa_smp: fix clang report The variable 'smp_break' is only set and used in case of smp. But clang cannot track if 'target->smp' get modified between the set and the use of 'smp_break', so it consider possible to use 'smp_break' uninitialized. Initialize 'smp_break' to silent clang. Change-Id: Ifa25a84fe9ffa25b8b58d7920ec77994c3b7ebfe Signed-off-by: Antonio Borneo Fixes: 77287b8d47b4 ("target: add Espressif ESP32 basic support") Reviewed-on: https://review.openocd.org/c/openocd/+/7050 Reviewed-by: Erhan Kurubas Tested-by: jenkins --- src/target/espressif/esp_xtensa_smp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/target/espressif/esp_xtensa_smp.c b/src/target/espressif/esp_xtensa_smp.c index 1c36a295a..d51906bef 100644 --- a/src/target/espressif/esp_xtensa_smp.c +++ b/src/target/espressif/esp_xtensa_smp.c @@ -386,7 +386,7 @@ int esp_xtensa_smp_step(struct target *target, int handle_breakpoints) { int res; - uint32_t smp_break; + uint32_t smp_break = 0; struct esp_xtensa_smp_common *esp_xtensa_smp = target_to_esp_xtensa_smp(target); if (target->smp) {