This is an automated email from Gerrit. Spencer Oliver ([email protected]) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/199
-- gerrit commit 467db44be91a594274400845227987d06f544484 Author: Philip Nye <[email protected]> Date: Mon Nov 14 19:44:32 2011 +0000 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 <[email protected]> diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index 7c55e7c..cbc2b40 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 -- ------------------------------------------------------------------------------ RSA(R) Conference 2012 Save $700 by Nov 18 Register now http://p.sf.net/sfu/rsa-sfdev2dev1 _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
