diff options
| author | Dean Sanner <dsanner@us.ibm.com> | 2013-06-28 09:44:49 -0500 |
|---|---|---|
| committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2013-06-28 14:21:37 -0500 |
| commit | 9bd9d3d205199e1aafdf78caddadb1de413df7e7 (patch) | |
| tree | 3a32c2a9b68cdc7990be9d8a056d10d5f0e6ab62 /src/kernel | |
| parent | 9b19b8e279fb596cb22ae59535b1d275ea523309 (diff) | |
| download | blackbird-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.C | 8 |
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; } |

