[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openocd-development
Subject:    [OpenOCD-devel] [PATCH]: 17a9ac5 target: remove memory leaks
From:       gerrit () openocd ! zylin ! com
Date:       2014-03-28 11:28:39
Message-ID: 20140328112839.DFCE1242D1 () openocd ! zylin ! com
[Download RAW message or body]

This is an automated email from Gerrit.

Spencer Oliver (spen@spen-soft.co.uk) just uploaded a new patch set to Gerrit, which \
you can find at http://openocd.zylin.com/2067

-- gerrit

commit 17a9ac55d73d1f34183e8ded9ff1dcd18f396c43
Author: Spencer Oliver <spen@spen-soft.co.uk>
Date:   Fri Mar 28 11:27:48 2014 +0000

    target: remove memory leaks
    
    Found by clang.
    
    Change-Id: Ifb25dca52f8d9e8e46a35f0947a7239f26eb3757
    Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>

diff --git a/src/target/arm_adi_v5.c b/src/target/arm_adi_v5.c
index 4a1e42c..4bfa7f7 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 8970fd7..faf9e0a 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 a7b3ed2..f26a016 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 99f67af..e88a231 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++) {

-- 

------------------------------------------------------------------------------
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel


[prev in list] [next in list] [prev in thread] [next in thread]