From: Philip Nye Date: Mon, 14 Nov 2011 19:44:32 +0000 (+0000) Subject: gdb: fix multi core gdb issue X-Git-Tag: v0.6.0-rc1~419 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=950f2405197661974225a578630e44667639ccb6;p=openocd gdb: fix multi core gdb issue gdb_memory_map() correctly calculates the target specific number of flash banks, but then uses the total number (all targets) instead of the target specific number to construct its GDB response, causing a crash. Change-Id: I3f8639b3e90303a59753ebe140ce4fff96fd5db0 Signed-off-by: Spencer Oliver Reviewed-on: http://openocd.zylin.com/199 Tested-by: jenkins Reviewed-by: Andreas Fritiofson --- diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index 7c55e7cd..cbc2b401 100644 --- a/src/server/gdb_server.c +++ b/src/server/gdb_server.c @@ -1741,7 +1741,7 @@ static int gdb_memory_map(struct connection *connection, qsort(banks, target_flash_banks, sizeof(struct flash_bank *), compare_bank); - for (i = 0; i < flash_get_bank_count(); i++) { + for (i = 0; i < target_flash_banks; i++) { int j; unsigned sector_size = 0; uint32_t start; @@ -2451,7 +2451,7 @@ static int gdb_target_start(struct target *target, const char *port) { curr = head->target; if (curr != target) curr->gdb_service = gdb_service; - head = head->next; + head = head->next; } } return ret; @@ -2462,7 +2462,7 @@ static int gdb_target_add_one(struct target *target) /* one gdb instance per smp list */ if ((target->smp) && (target->gdb_service)) return ERROR_OK; int retval = gdb_target_start(target, gdb_port_next); - if (retval == ERROR_OK) + if (retval == ERROR_OK) { long portnumber; /* If we can parse the port number