Merge pull request micropython#1119 from tannewt/memory_free_fix

tannewt · web-flow · commit 92ed5d7bf223 · 2018-08-16T11:21:34.000-07:00
Fix crash due to unsigned index and 0 boundary loop.
diff --git a/supervisor/shared/memory.c b/supervisor/shared/memory.c
@@ -43,7 +43,7 @@ void memory_init(void) {
 }
 
 void free_memory(supervisor_allocation* allocation) {
-    uint8_t index = 0;
+    int32_t index = 0;
     bool found = false;
     for (index = 0; index < CIRCUITPY_SUPERVISOR_ALLOC_COUNT; index++) {
         found = allocation == &allocations[index];

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ void memory_init(void) {`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`void free_memory(supervisor_allocation* allocation) {`
`46`		`- uint8_t index = 0;`
	`46`	`+ int32_t index = 0;`
`47`	`47`	`bool found = false;`
`48`	`48`	`for (index = 0; index < CIRCUITPY_SUPERVISOR_ALLOC_COUNT; index++) {`
`49`	`49`	`found = allocation == &allocations[index];`