diff options
| author | Spencer Oliver <spen@spen-soft.co.uk> | 2014-03-28 11:27:48 +0000 |
|---|---|---|
| committer | Paul Fertser <fercerpav@gmail.com> | 2014-03-29 08:04:55 +0000 |
| commit | 0f566ae1a78b054328de2123ff36f93bc5b8fd93 (patch) | |
| tree | fd8cde9ec40f60839d8c931dfed1cf407ba23d23 /src/target/nds32_cmd.c | |
| parent | 0cb9778368ddda6dc193752034de6f8aafeb2454 (diff) | |
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>
Diffstat (limited to 'src/target/nds32_cmd.c')
| -rw-r--r-- | src/target/nds32_cmd.c | 11 |
1 files changed, 9 insertions, 2 deletions
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; |