target: remove handle_profile_command memory leak
COMMAND_PARSE_NUMBER may return, thus any memory allocated may not be freed, simple reorder fixes the issue. Change-Id: I0ce444a5b032f5c49b6d33a03a8c0b71cad49c8c Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/2065 Tested-by: jenkins Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
This commit is contained in:
parent
8b1fabd8e0
commit
e22bad797f
|
@ -3620,14 +3620,15 @@ COMMAND_HANDLER(handle_profile_command)
|
||||||
uint32_t offset;
|
uint32_t offset;
|
||||||
uint32_t num_of_sampels;
|
uint32_t num_of_sampels;
|
||||||
int retval = ERROR_OK;
|
int retval = ERROR_OK;
|
||||||
|
|
||||||
|
COMMAND_PARSE_NUMBER(u32, CMD_ARGV[0], offset);
|
||||||
|
|
||||||
uint32_t *samples = malloc(sizeof(uint32_t) * MAX_PROFILE_SAMPLE_NUM);
|
uint32_t *samples = malloc(sizeof(uint32_t) * MAX_PROFILE_SAMPLE_NUM);
|
||||||
if (samples == NULL) {
|
if (samples == NULL) {
|
||||||
LOG_ERROR("No memory to store samples.");
|
LOG_ERROR("No memory to store samples.");
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
}
|
}
|
||||||
|
|
||||||
COMMAND_PARSE_NUMBER(u32, CMD_ARGV[0], offset);
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Some cores let us sample the PC without the
|
* Some cores let us sample the PC without the
|
||||||
* annoying halt/resume step; for example, ARMv7 PCSR.
|
* annoying halt/resume step; for example, ARMv7 PCSR.
|
||||||
|
|
Loading…
Reference in New Issue