diff options
author | Kostya Kortchinsky <kostyak@google.com> | 2016-09-20 22:17:59 +0000 |
---|---|---|
committer | Kostya Kortchinsky <kostyak@google.com> | 2016-09-20 22:17:59 +0000 |
commit | 9a0a02984a9231b7340aaa02fbc46e42c1d31587 (patch) | |
tree | 3c1d379059b78431bc10b86fff242bcedab37f17 /cmake/Modules | |
parent | c49eacfbb518e8703d51c2230c53cd29f2a15b14 (diff) |
[scudo] Fix a bug in the new Secondary Allocator
Summary:
GetActuallyAllocatedSize() was not accounting for the last page of the mapping
being a guard page, and was returning the wrong number of actually allocated
bytes, which in turn would mess up with the realloc logic. Current tests didn't
find this as the size exercised was only serviced by the Primary.
Correct the issue by subtracting PageSize, and update the realloc test to
exercise paths in both the Primary and the Secondary.
Reviewers: kcc
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D24787
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@282030 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'cmake/Modules')
0 files changed, 0 insertions, 0 deletions