rewrite broken null pointer check code by reducing scope
of variable.
Change-Id: I8254f6849b187e5c9cd083053cdc11973c6fe339
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Reviewed-on: http://openocd.zylin.com/142
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
static struct debug_msg_receiver* find_debug_msg_receiver(struct command_context *cmd_ctx, struct target *target)
{
int do_all_targets = 0;
static struct debug_msg_receiver* find_debug_msg_receiver(struct command_context *cmd_ctx, struct target *target)
{
int do_all_targets = 0;
- struct debug_msg_receiver **p = &target->dbgmsg;
/* if no target has been specified search all of them */
/* if no target has been specified search all of them */
- if (target == NULL)
- {
/* if no targets haven been specified */
if (all_targets == NULL)
return NULL;
/* if no targets haven been specified */
if (all_targets == NULL)
return NULL;
+ /* so we target != null */
+ struct debug_msg_receiver **p = &target->dbgmsg;
+ do {
while (*p)
{
if ((*p)->cmd_ctx == cmd_ctx)
while (*p)
{
if ((*p)->cmd_ctx == cmd_ctx)