flash/stm32l4x: fix maybe-uninitialized compiler error
authorTarek BOCHKATI <tarek.bouchkati@gmail.com>
Mon, 28 Feb 2022 09:29:44 +0000 (10:29 +0100)
committerTomas Vanek <vanekt@fbl.cz>
Tue, 1 Mar 2022 08:36:52 +0000 (08:36 +0000)
using gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0 we get:
error: ‘retval’ may be used uninitialized in this function

fixes: 13cd75b6ecfd (flash/nor/stm32xx: fix segfault accessing Cortex-M part number)
Change-Id: I897c40c5d2233f50a5385d251ebfa536023e5cf7
Signed-off-by: Tarek BOCHKATI <tarek.bouchkati@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/6861
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
src/flash/nor/stm32l4x.c

index fa8924ed385e12dc144930e7ae37d49eb256f911..fd0338899f461293098f2f3de4d8fa743e8f5211 100644 (file)
@@ -1632,7 +1632,7 @@ err_lock:
 
 static int stm32l4_read_idcode(struct flash_bank *bank, uint32_t *id)
 {
-       int retval;
+       int retval = ERROR_OK;
        struct target *target = bank->target;
 
        /* try reading possible IDCODE registers, in the following order */