This is an automated email from Gerrit.

"Tomas Vanek <[email protected]>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/9276

-- gerrit

commit e0475cc97848e9c4abbe87a32dcdb6d88bfd3b23
Author: Tomas Vanek <[email protected]>
Date:   Thu Aug 8 21:03:28 2024 +0200

    target/riscv: do not set DTM_DTMCS_VERSION_UNKNOWN on examine fail
    
    Proper version is necessary for target deinit or future
    re-examinations. Poisoning the version number causes memory leak
    in riscv_deinit_target().
    
    Signed-off-by: Tomas Vanek <[email protected]>
    Change-Id: Ie63ae83626c3fd7498669cb2898309f1c5d4d112

diff --git a/src/target/riscv/riscv.c b/src/target/riscv/riscv.c
index fc59468b7b..c5090f9c42 100644
--- a/src/target/riscv/riscv.c
+++ b/src/target/riscv/riscv.c
@@ -2500,7 +2500,7 @@ static int riscv_examine(struct target *target)
 
        examine_status = tt->examine(target);
        if (examine_status != ERROR_OK)
-               goto examine_fail;
+               return examine_status;
 
        return ERROR_OK;
 

-- 

Reply via email to