diff options
author | Kostya Kortchinsky <kostyak@google.com> | 2016-09-19 21:11:55 +0000 |
---|---|---|
committer | Kostya Kortchinsky <kostyak@google.com> | 2016-09-19 21:11:55 +0000 |
commit | 3beafffcca2eb6306ebf7b68b8069dda95f49492 (patch) | |
tree | 64e5e46c8a36703330e2ed278d235c3d79bb1295 /compiler-rt/lib/scudo/scudo_allocator_secondary.h | |
parent | 233374c4d14e5e0bd6258560f7ba786b37bd12c1 (diff) | |
download | bcm5719-llvm-3beafffcca2eb6306ebf7b68b8069dda95f49492.tar.gz bcm5719-llvm-3beafffcca2eb6306ebf7b68b8069dda95f49492.zip |
[scudo] Modify Scudo to use its own Secondary Allocator
Summary:
The Sanitizer Secondary Allocator was not entirely ideal was Scudo for several
reasons: decent amount of unneeded code, redundant checks already performed by
the front end, unneeded data structures, difficulty to properly protect the
secondary chunks header.
Given that the second allocator is pretty straight forward, Scudo will use its
own, trimming all the unneeded code off of the Sanitizer one. A significant
difference in terms of security is that now each secondary chunk is preceded
and followed by a guard page, thus mitigating overflows into and from the
chunk.
A test was added as well to illustrate the overflow & underflow situations
into the guard pages.
Reviewers: kcc
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D24737
llvm-svn: 281938
Diffstat (limited to 'compiler-rt/lib/scudo/scudo_allocator_secondary.h')
-rw-r--r-- | compiler-rt/lib/scudo/scudo_allocator_secondary.h | 138 |
1 files changed, 138 insertions, 0 deletions
diff --git a/compiler-rt/lib/scudo/scudo_allocator_secondary.h b/compiler-rt/lib/scudo/scudo_allocator_secondary.h new file mode 100644 index 00000000000..450d212d5a0 --- /dev/null +++ b/compiler-rt/lib/scudo/scudo_allocator_secondary.h @@ -0,0 +1,138 @@ +//===-- scudo_allocator_secondary.h -----------------------------*- C++ -*-===// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +/// +/// Scudo Secondary Allocator. +/// This services allocation that are too large to be serviced by the Primary +/// Allocator. It is directly backed by the memory mapping functions of the +/// operating system. +/// +//===----------------------------------------------------------------------===// + +#ifndef SCUDO_ALLOCATOR_SECONDARY_H_ +#define SCUDO_ALLOCATOR_SECONDARY_H_ + +namespace __scudo { + +class ScudoLargeMmapAllocator { + public: + + void Init(bool AllocatorMayReturnNull) { + PageSize = GetPageSizeCached(); + atomic_store(&MayReturnNull, AllocatorMayReturnNull, memory_order_relaxed); + } + + void *Allocate(AllocatorStats *Stats, uptr Size, uptr Alignment) { + // The Scudo frontend prevents us from allocating more than + // MaxAllowedMallocSize, so integer overflow checks would be superfluous. + uptr MapSize = RoundUpTo(Size + sizeof(SecondaryHeader), PageSize); + // Account for 2 guard pages, one before and one after the chunk. + uptr MapBeg = reinterpret_cast<uptr>(MmapNoAccess(MapSize + 2 * PageSize)); + CHECK_NE(MapBeg, ~static_cast<uptr>(0)); + // A page-aligned pointer is assumed after that, so check it now. + CHECK(IsAligned(MapBeg, PageSize)); + MapBeg += PageSize; + CHECK_EQ(MapBeg, reinterpret_cast<uptr>(MmapFixedOrDie(MapBeg, MapSize))); + uptr MapEnd = MapBeg + MapSize; + uptr Ptr = MapBeg + sizeof(SecondaryHeader); + // TODO(kostyak): add a random offset to Ptr. + CHECK_GT(Ptr + Size, MapBeg); + CHECK_LE(Ptr + Size, MapEnd); + SecondaryHeader *Header = getHeader(Ptr); + Header->MapBeg = MapBeg - PageSize; + Header->MapSize = MapSize + 2 * PageSize; + Stats->Add(AllocatorStatAllocated, MapSize); + Stats->Add(AllocatorStatMapped, MapSize); + return reinterpret_cast<void *>(Ptr); + } + + void *ReturnNullOrDie() { + if (atomic_load(&MayReturnNull, memory_order_acquire)) + return nullptr; + ReportAllocatorCannotReturnNull(); + } + + void SetMayReturnNull(bool AllocatorMayReturnNull) { + atomic_store(&MayReturnNull, AllocatorMayReturnNull, memory_order_release); + } + + void Deallocate(AllocatorStats *Stats, void *Ptr) { + SecondaryHeader *Header = getHeader(Ptr); + Stats->Sub(AllocatorStatAllocated, Header->MapSize); + Stats->Sub(AllocatorStatMapped, Header->MapSize); + UnmapOrDie(reinterpret_cast<void *>(Header->MapBeg), Header->MapSize); + } + + uptr TotalMemoryUsed() { + UNIMPLEMENTED(); + } + + bool PointerIsMine(const void *Ptr) { + UNIMPLEMENTED(); + } + + uptr GetActuallyAllocatedSize(void *Ptr) { + SecondaryHeader *Header = getHeader(Ptr); + uptr MapEnd = Header->MapBeg + Header->MapSize; + return MapEnd - reinterpret_cast<uptr>(Ptr); + } + + void *GetMetaData(const void *Ptr) { + UNIMPLEMENTED(); + } + + void *GetBlockBegin(const void *Ptr) { + UNIMPLEMENTED(); + } + + void *GetBlockBeginFastLocked(void *Ptr) { + UNIMPLEMENTED(); + } + + void PrintStats() { + UNIMPLEMENTED(); + } + + void ForceLock() { + UNIMPLEMENTED(); + } + + void ForceUnlock() { + UNIMPLEMENTED(); + } + + void ForEachChunk(ForEachChunkCallback Callback, void *Arg) { + UNIMPLEMENTED(); + } + + private: + // A Secondary allocated chunk header contains the base of the mapping and + // its size. Currently, the base is always a page before the header, but + // we might want to extend that number in the future based on the size of + // the allocation. + struct SecondaryHeader { + uptr MapBeg; + uptr MapSize; + }; + // Check that sizeof(SecondaryHeader) is a multiple of 16. + COMPILER_CHECK((sizeof(SecondaryHeader) & 0xf) == 0); + + SecondaryHeader *getHeader(uptr Ptr) { + return reinterpret_cast<SecondaryHeader*>(Ptr - sizeof(SecondaryHeader)); + } + SecondaryHeader *getHeader(const void *Ptr) { + return getHeader(reinterpret_cast<uptr>(Ptr)); + } + + uptr PageSize; + atomic_uint8_t MayReturnNull; +}; + +} // namespace __scudo + +#endif // SCUDO_ALLOCATOR_SECONDARY_H_ |