This is an automated email from Gerrit.

"Tomas Vanek <van...@fbl.cz>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/7238

-- gerrit

commit 3a60717aec62f31b9d9ca7d29a1d0b2c17068cee
Author: Tomas Vanek <van...@fbl.cz>
Date:   Sat Oct 1 17:00:51 2022 +0200

    target/armv7m: check error in armv7m_restore_context()
    
    Return error if arm.write_core_reg() fails.
    
    Change-Id: Ide8f5aa5958532b202dc9f5e13d3250a706d832d
    Signed-off-by: Tomas Vanek <van...@fbl.cz>

diff --git a/src/target/armv7m.c b/src/target/armv7m.c
index 3979887df2..5d93ec3338 100644
--- a/src/target/armv7m.c
+++ b/src/target/armv7m.c
@@ -182,8 +182,11 @@ int armv7m_restore_context(struct target *target)
        for (i = cache->num_regs - 1; i >= 0; i--) {
                struct reg *r = &cache->reg_list[i];
 
-               if (r->exist && r->dirty)
-                       armv7m->arm.write_core_reg(target, r, i, ARM_MODE_ANY, 
r->value);
+               if (r->exist && r->dirty) {
+                       int retval = armv7m->arm.write_core_reg(target, r, i, 
ARM_MODE_ANY, r->value);
+                       if (retval != ERROR_OK)
+                               return retval;
+               }
        }
 
        return ERROR_OK;

-- 

Reply via email to