Improve and simplify handle_bp_command and handle_rbp_command:
- Bug fix: return syntax error if remove called without one argument. - Use parse_u32 to ensure address and length arguments parse properly. git-svn-id: svn://svn.berlios.de/openocd/trunk@2235 b42882b7-edfa-0310-969c-e2dbd0fdcd60
This commit is contained in:
parent
16c77cf3ca
commit
d0a300e0a0
|
@ -2541,8 +2541,15 @@ static int handle_bp_command(struct command_context_s *cmd_ctx,
|
|||
return ERROR_COMMAND_SYNTAX_ERROR;
|
||||
}
|
||||
|
||||
u32 addr = strtoul(args[0], NULL, 0);
|
||||
u32 length = strtoul(args[1], NULL, 0);
|
||||
u32 addr;
|
||||
int retval = parse_u32(args[0], &addr);
|
||||
if (ERROR_OK != retval)
|
||||
return retval;
|
||||
|
||||
u32 length;
|
||||
retval = parse_u32(args[1], &length);
|
||||
if (ERROR_OK != retval)
|
||||
return retval;
|
||||
|
||||
int hw = BKPT_SOFT;
|
||||
if (argc == 3)
|
||||
|
@ -2558,10 +2565,16 @@ static int handle_bp_command(struct command_context_s *cmd_ctx,
|
|||
|
||||
static int handle_rbp_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
|
||||
{
|
||||
target_t *target = get_current_target(cmd_ctx);
|
||||
if (argc != 1)
|
||||
return ERROR_COMMAND_SYNTAX_ERROR;
|
||||
|
||||
if (argc > 0)
|
||||
breakpoint_remove(target, strtoul(args[0], NULL, 0));
|
||||
u32 addr;
|
||||
int retval = parse_u32(args[0], &addr);
|
||||
if (ERROR_OK != retval)
|
||||
return retval;
|
||||
|
||||
target_t *target = get_current_target(cmd_ctx);
|
||||
breakpoint_remove(target, addr);
|
||||
|
||||
return ERROR_OK;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue