boot: zephyr: flash_map_extended: Add pointless workaround for clang

nordicjm · nordicjm · commit 572b8fb2cd5c · 2025-04-07T11:16:31.000+01:00
Clang wrongly throws a warning, which will be treated as an error in
twister builds, add pointless workaround to set variable that is
already set by the hook function to avoid this

Signed-off-by: Jamie McCrae &lt;jamie.mccrae@nordicsemi.no&gt;
diff --git a/boot/zephyr/flash_map_extended.c b/boot/zephyr/flash_map_extended.c
@@ -66,7 +66,8 @@ int flash_device_base(uint8_t fd_id, uintptr_t *ret)
  */
 int flash_area_id_from_multi_image_slot(int image_index, int slot)
 {
-    int rc, id;
+    int rc;
+    int id = -1;
 
     rc = BOOT_HOOK_FLASH_AREA_CALL(flash_area_id_from_multi_image_slot_hook,
                                    BOOT_HOOK_REGULAR, image_index, slot, &id);

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,8 @@ int flash_device_base(uint8_t fd_id, uintptr_t *ret)`
`66`	`66`	`*/`
`67`	`67`	`int flash_area_id_from_multi_image_slot(int image_index, int slot)`
`68`	`68`	`{`
`69`		`- int rc, id;`
	`69`	`+ int rc;`
	`70`	`+ int id = -1;`
`70`	`71`
`71`	`72`	`rc = BOOT_HOOK_FLASH_AREA_CALL(flash_area_id_from_multi_image_slot_hook,`
`72`	`73`	`BOOT_HOOK_REGULAR, image_index, slot, &id);`