summaryrefslogtreecommitdiffstats
path: root/src/kernel
diff options
context:
space:
mode:
authorDean Sanner <dsanner@us.ibm.com>2013-06-28 09:44:49 -0500
committerA. Patrick Williams III <iawillia@us.ibm.com>2013-06-28 14:21:37 -0500
commit9bd9d3d205199e1aafdf78caddadb1de413df7e7 (patch)
tree3a32c2a9b68cdc7990be9d8a056d10d5f0e6ab62 /src/kernel
parent9b19b8e279fb596cb22ae59535b1d275ea523309 (diff)
downloadblackbird-hostboot-9bd9d3d205199e1aafdf78caddadb1de413df7e7.tar.gz
blackbird-hostboot-9bd9d3d205199e1aafdf78caddadb1de413df7e7.zip
Fix mailbox memory allocation error on attribute sync
Change-Id: If156faa95769b70731dc77f7b9801ff3ea743556 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/5239 Reviewed-by: Dean Sanner <dsanner@us.ibm.com> Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/kernel')
-rw-r--r--src/kernel/heapmgr.C8
1 files changed, 8 insertions, 0 deletions
diff --git a/src/kernel/heapmgr.C b/src/kernel/heapmgr.C
index d2df127e9..4fdfded57 100644
--- a/src/kernel/heapmgr.C
+++ b/src/kernel/heapmgr.C
@@ -578,6 +578,14 @@ bool HeapManager::_freeBig(void* i_ptr)
}
bc = (big_chunk_t*) (((uint64_t)bc->next) & 0x00000000FFFFFFFF);
}
+
+ // Small allocations are always aligned, hence we exited out at the
+ // beginning of the function. Large allocations are always aligned.
+ // If we did not find a large allocation in the list (result == false)
+ // then either we have a double-free or someone trying to free something
+ // that doesn't belong on the heap.
+ crit_assert(result);
+
return result;
}
OpenPOWER on IntegriCloud