target: remove memory leaks
Found by clang. Change-Id: Ifb25dca52f8d9e8e46a35f0947a7239f26eb3757 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/2067 Tested-by: jenkins Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com> Reviewed-by: Jörg Wunsch <openocd@uriah.heep.sax.de>
This commit is contained in:
parent
0cb9778368
commit
0f566ae1a7
|
@ -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);
|
retval = dap_setup_accessport_tar(dap, address);
|
||||||
if (retval != ERROR_OK)
|
if (retval != ERROR_OK) {
|
||||||
|
free(read_buf);
|
||||||
return retval;
|
return retval;
|
||||||
|
}
|
||||||
|
|
||||||
/* Queue up all reads. Each read will store the entire DRW word in the read buffer. How many
|
/* 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
|
* useful bytes it contains, and their location in the word, depends on the type of transfer
|
||||||
|
|
|
@ -704,18 +704,25 @@ static int jim_nds32_bulk_write(Jim_Interp *interp, int argc, Jim_Obj * const *a
|
||||||
return e;
|
return e;
|
||||||
|
|
||||||
uint32_t *data = malloc(count * sizeof(uint32_t));
|
uint32_t *data = malloc(count * sizeof(uint32_t));
|
||||||
|
if (data == NULL)
|
||||||
|
return JIM_ERR;
|
||||||
|
|
||||||
jim_wide i;
|
jim_wide i;
|
||||||
for (i = 0; i < count; i++) {
|
for (i = 0; i < count; i++) {
|
||||||
jim_wide tmp;
|
jim_wide tmp;
|
||||||
e = Jim_GetOpt_Wide(&goi, &tmp);
|
e = Jim_GetOpt_Wide(&goi, &tmp);
|
||||||
if (e != JIM_OK)
|
if (e != JIM_OK) {
|
||||||
|
free(data);
|
||||||
return e;
|
return e;
|
||||||
|
}
|
||||||
data[i] = (uint32_t)tmp;
|
data[i] = (uint32_t)tmp;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* all args must be consumed */
|
/* all args must be consumed */
|
||||||
if (goi.argc != 0)
|
if (goi.argc != 0) {
|
||||||
|
free(data);
|
||||||
return JIM_ERR;
|
return JIM_ERR;
|
||||||
|
}
|
||||||
|
|
||||||
struct target *target = Jim_CmdPrivData(goi.interp);
|
struct target *target = Jim_CmdPrivData(goi.interp);
|
||||||
int result;
|
int result;
|
||||||
|
|
|
@ -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)
|
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)
|
if (target->tap == NULL)
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
|
|
||||||
|
struct or1k_common *or1k = calloc(1, sizeof(struct or1k_common));
|
||||||
|
|
||||||
target->arch_info = or1k;
|
target->arch_info = or1k;
|
||||||
|
|
||||||
or1k_create_reg_list(target);
|
or1k_create_reg_list(target);
|
||||||
|
|
|
@ -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) */
|
/* receive data from target (count times 32-bit words in host endianness) */
|
||||||
buf32 = malloc(4 * count);
|
buf32 = malloc(4 * count);
|
||||||
retval = xscale_receive(target, buf32, count);
|
retval = xscale_receive(target, buf32, count);
|
||||||
if (retval != ERROR_OK)
|
if (retval != ERROR_OK) {
|
||||||
|
free(buf32);
|
||||||
return retval;
|
return retval;
|
||||||
|
}
|
||||||
|
|
||||||
/* extract data from host-endian buffer into byte stream */
|
/* extract data from host-endian buffer into byte stream */
|
||||||
for (i = 0; i < count; i++) {
|
for (i = 0; i < count; i++) {
|
||||||
|
|
Loading…
Reference in New Issue