mbed_error.c: Fixed another bug of possible stack overflow.

andrewc-arm · andrewc-arm · commit c257c5f0c751 · 2019-09-06T16:49:09.000+08:00
diff --git a/platform/source/mbed_error.c b/platform/source/mbed_error.c
@@ -509,7 +509,7 @@ static void print_stack_dump_core(uint32_t stack_start, uint32_t stack_size, uin
         mbed_error_printf("\n0x%08" PRIX32 ":", st);
         for (int i = 0; i < STACK_DUMP_WIDTH; i++) {
             uint32_t st_cur = st + i * sizeof(int);
-            if (st_cur > st_end) {
+            if (st_cur >= st_end) {
                 break;
             }
             uint32_t st_val = *((uint32_t *)st_cur);

Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,7 @@ static void print_stack_dump_core(uint32_t stack_start, uint32_t stack_size, uin`
`509`	`509`	`mbed_error_printf("\n0x%08" PRIX32 ":", st);`
`510`	`510`	`for (int i = 0; i < STACK_DUMP_WIDTH; i++) {`
`511`	`511`	`uint32_t st_cur = st + i * sizeof(int);`
`512`		`- if (st_cur > st_end) {`
	`512`	`+ if (st_cur >= st_end) {`
`513`	`513`	`break;`
`514`	`514`	`}`
`515`	`515`	`uint32_t st_val = ((uint32_t )st_cur);`