This is an automated email from Gerrit.

"Tarek BOCHKATI <tarek.bouchk...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/6861

-- gerrit

commit db1c34842bdd8c82e7198ab4d02e6f392635e3ed
Author: Tarek BOCHKATI <tarek.bouchk...@gmail.com>
Date:   Mon Feb 28 10:29:44 2022 +0100

    flash/stm32l4x: fix maybe-uninitialized variable error
    
    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.bouchk...@gmail.com>

diff --git a/src/flash/nor/stm32l4x.c b/src/flash/nor/stm32l4x.c
index fa8924ed38..fd0338899f 100644
--- a/src/flash/nor/stm32l4x.c
+++ b/src/flash/nor/stm32l4x.c
@@ -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 */

-- 

Reply via email to