commit 497f27ed9df2e53e81ffc631c0c66055e319e523
author Urvang Joshi <urvang@google.com> Tue Jul 26 12:02:37 2016 -0700
committer Yaowu Xu <yaowu@google.com> Tue Sep 06 21:27:20 2016 +0000
tree 619d08b36d9d198c3aab75c51a613e2a8858f353
parent 73a3fd47101ececb77af984756ee8d5977d9cafa

aom_realloc correction.

aom_realloc was allocating 1 byte more than needed every time.
Fixed this, and took this opportunity to do a small refactoring.

Change-Id: I38fcb62b698894acbbab43466c1decd12f906789

aom_mem/aom_mem.c

diff --git a/aom_mem/aom_mem.c b/aom_mem/aom_mem.c
index 35b286b..38350c6 100644
--- a/aom_mem/aom_mem.c
+++ b/aom_mem/aom_mem.c
@@ -18,6 +18,10 @@
 #include "include/aom_mem_intrnl.h"
 #include "aom/aom_integer.h"
 
+static inline size_t GetAlignedMallocSize(size_t size, size_t align) {
+  return size + align - 1 + ADDRESS_STORAGE_SIZE;
+}
+
 static inline size_t *GetMallocAddressLocation(void *const mem) {
   return ((size_t *)mem) - 1;
 }
@@ -35,7 +39,8 @@
 
 void *aom_memalign(size_t align, size_t size) {
   void *x = NULL;
-  void *const addr = malloc(size + align - 1 + ADDRESS_STORAGE_SIZE);
+  const size_t aligned_size = GetAlignedMallocSize(size, align);
+  void *const addr = malloc(aligned_size);
   if (addr) {
     x = align_addr((unsigned char *)addr + ADDRESS_STORAGE_SIZE, (int)align);
     SetActualMallocAddress(x, addr);
@@ -69,8 +74,9 @@
     aom_free(memblk);
   else {
     void *addr = GetActualMallocAddress(memblk);
+    const size_t aligned_size = GetAlignedMallocSize(size, DEFAULT_ALIGNMENT);
     memblk = NULL;
-    addr = realloc(addr, size + DEFAULT_ALIGNMENT + ADDRESS_STORAGE_SIZE);
+    addr = realloc(addr, aligned_size);
     if (addr) {
       new_addr = align_addr((unsigned char *)addr + ADDRESS_STORAGE_SIZE,
                             DEFAULT_ALIGNMENT);