commit 1079406cab09090cc11e26d4ac2f1ba9c4930cdb
author Dmitriy Ivanov <dimitry@google.com> Wed May 14 12:52:57 2014 -0700
committer Dmitriy Ivanov <dimitry@google.com> Wed May 14 12:52:57 2014 -0700
tree 1990287fea9a5cc28410dfa1bc3ff3741ad1cfc2
parent c6b5834ceb3df72608a8fe3526a2db337e504512

Fix for linker allocator

Change-Id: I18a3f1a8515657f0deb69266184fb516a27f90e8

linker/linker_allocator.cpp

diff --git a/linker/linker_allocator.cpp b/linker/linker_allocator.cpp
index 60ce1ea..5738090 100644
--- a/linker/linker_allocator.cpp
+++ b/linker/linker_allocator.cpp
@@ -117,8 +117,8 @@
   }
 
   LinkerAllocatorPage* page = page_list_;
-  const uint8_t* page_ptr = reinterpret_cast<const uint8_t*>(page);
   while (page != nullptr) {
+    const uint8_t* page_ptr = reinterpret_cast<const uint8_t*>(page);
     if (block >= (page_ptr + sizeof(page->next)) && block < (page_ptr + PAGE_SIZE)) {
       return page;
     }

linker/tests/linker_allocator_test.cpp

diff --git a/linker/tests/linker_allocator_test.cpp b/linker/tests/linker_allocator_test.cpp
index e3a91c5..0ed8252 100644
--- a/linker/tests/linker_allocator_test.cpp
+++ b/linker/tests/linker_allocator_test.cpp
@@ -96,6 +96,9 @@
     ASSERT_TRUE(allocator.alloc() != nullptr);
   }
 
+  test_struct_larger* ptr_to_free = allocator.alloc();
+  ASSERT_TRUE(ptr_to_free != nullptr);
+  allocator.free(ptr1);
 }
 
 static void protect_all() {