From 0f566ae1a78b054328de2123ff36f93bc5b8fd93 Mon Sep 17 00:00:00 2001 From: Spencer Oliver Date: Fri, 28 Mar 2014 11:27:48 +0000 Subject: [PATCH] target: remove memory leaks MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Found by clang. Change-Id: Ifb25dca52f8d9e8e46a35f0947a7239f26eb3757 Signed-off-by: Spencer Oliver Reviewed-on: http://openocd.zylin.com/2067 Tested-by: jenkins Reviewed-by: Andreas Fritiofson Reviewed-by: Jörg Wunsch --- src/target/arm_adi_v5.c | 4 +++- src/target/nds32_cmd.c | 11 +++++++++-- src/target/openrisc/or1k.c | 4 ++-- src/target/xscale.c | 4 +++- 4 files changed, 17 insertions(+), 6 deletions(-) diff --git a/src/target/arm_adi_v5.c b/src/target/arm_adi_v5.c index 4a1e42c0..4bfa7f79 100644 --- a/src/target/arm_adi_v5.c +++ b/src/target/arm_adi_v5.c @@ -409,8 +409,10 @@ int mem_ap_read(struct adiv5_dap *dap, uint8_t *buffer, uint32_t size, uint32_t } retval = dap_setup_accessport_tar(dap, address); - if (retval != ERROR_OK) + if (retval != ERROR_OK) { + free(read_buf); return retval; + } /* Queue up all reads. Each read will store the entire DRW word in the read buffer. How many * useful bytes it contains, and their location in the word, depends on the type of transfer diff --git a/src/target/nds32_cmd.c b/src/target/nds32_cmd.c index 8970fd7e..faf9e0ae 100644 --- a/src/target/nds32_cmd.c +++ b/src/target/nds32_cmd.c @@ -704,18 +704,25 @@ static int jim_nds32_bulk_write(Jim_Interp *interp, int argc, Jim_Obj * const *a return e; uint32_t *data = malloc(count * sizeof(uint32_t)); + if (data == NULL) + return JIM_ERR; + jim_wide i; for (i = 0; i < count; i++) { jim_wide tmp; e = Jim_GetOpt_Wide(&goi, &tmp); - if (e != JIM_OK) + if (e != JIM_OK) { + free(data); return e; + } data[i] = (uint32_t)tmp; } /* all args must be consumed */ - if (goi.argc != 0) + if (goi.argc != 0) { + free(data); return JIM_ERR; + } struct target *target = Jim_CmdPrivData(goi.interp); int result; diff --git a/src/target/openrisc/or1k.c b/src/target/openrisc/or1k.c index a7b3ed27..f26a0162 100644 --- a/src/target/openrisc/or1k.c +++ b/src/target/openrisc/or1k.c @@ -1169,11 +1169,11 @@ static int or1k_init_target(struct command_context *cmd_ctx, static int or1k_target_create(struct target *target, Jim_Interp *interp) { - struct or1k_common *or1k = calloc(1, sizeof(struct or1k_common)); - if (target->tap == NULL) return ERROR_FAIL; + struct or1k_common *or1k = calloc(1, sizeof(struct or1k_common)); + target->arch_info = or1k; or1k_create_reg_list(target); diff --git a/src/target/xscale.c b/src/target/xscale.c index 99f67aff..e88a2311 100644 --- a/src/target/xscale.c +++ b/src/target/xscale.c @@ -1821,8 +1821,10 @@ static int xscale_read_memory(struct target *target, uint32_t address, /* receive data from target (count times 32-bit words in host endianness) */ buf32 = malloc(4 * count); retval = xscale_receive(target, buf32, count); - if (retval != ERROR_OK) + if (retval != ERROR_OK) { + free(buf32); return retval; + } /* extract data from host-endian buffer into byte stream */ for (i = 0; i < count; i++) { -- 2.39.5