diff options
| author | Tomas Vanek <vanekt@fbl.cz> | 2018-12-18 17:17:33 +0100 |
|---|---|---|
| committer | Matthias Welwarsky <matthias@welwarsky.de> | 2019-02-07 07:51:50 +0000 |
| commit | 7a3eec2b4d9dbb9533acfb271dbe91afa0727c8e (patch) | |
| tree | 8b8d19f57f6f12dec1c1c0b728f04f1c6309bbbf /src/target/armv7m.c | |
| parent | 0d48104e03320112db1e8825d0700ea8be735cef (diff) | |
| download | riscv-openocd-7a3eec2b4d9dbb9533acfb271dbe91afa0727c8e.zip riscv-openocd-7a3eec2b4d9dbb9533acfb271dbe91afa0727c8e.tar.gz riscv-openocd-7a3eec2b4d9dbb9533acfb271dbe91afa0727c8e.tar.bz2 | |
target algo: do not write reg_param if direction is PARAM_IN
Without this change xxx_start_algorithm() writes all register
parameters no matter of their direction. It usually results
in writing of uninitialized reg_params[].value - possibly
reported by valgrind.
While on it fix the wrong parameter direction in
kinetis_disable_wdog_algo(). This bug did not have any
impact because of unconditional write of reg_params.
Change-Id: Ia9c6a7b37f77d5eb6e5f5463012dddd50471742b
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-on: http://openocd.zylin.com/4813
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-by: Matthias Welwarsky <matthias@welwarsky.de>
Diffstat (limited to 'src/target/armv7m.c')
| -rw-r--r-- | src/target/armv7m.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/target/armv7m.c b/src/target/armv7m.c index a1962fe..ecca0e7 100644 --- a/src/target/armv7m.c +++ b/src/target/armv7m.c @@ -388,6 +388,9 @@ int armv7m_start_algorithm(struct target *target, } for (int i = 0; i < num_reg_params; i++) { + if (reg_params[i].direction == PARAM_IN) + continue; + struct reg *reg = register_get_by_name(armv7m->arm.core_cache, reg_params[i].reg_name, 0); /* uint32_t regvalue; */ |