[PATCH]: e0475cc978 target/riscv: do not set DTM_DTMCS_VERSION_UNKNOWN on examine fail

[<ge...@op...>]

This is an automated email from Gerrit.

"Tomas Vanek <va...@fb...>" 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 <va...@fb...>
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 <va...@fb...>
    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;
 

--