[PATCH] D39303: [scudo] Remove comment about security of the 32-bit allocator

[Kostya Kortchinsky via Phabricator via llvm-commits]

cryptoad created this revision.

The 32-bit allocator is now on par with the 64-bit in terms of security (chunks
randomization is done, batches separation is done).

Unless objection, the comment can go away.


https://reviews.llvm.org/D39303

Files:
  lib/scudo/scudo_allocator.h


Index: lib/scudo/scudo_allocator.h
===================================================================
--- lib/scudo/scudo_allocator.h
+++ lib/scudo/scudo_allocator.h
@@ -77,9 +77,6 @@
 };
 typedef SizeClassAllocator64<AP64> PrimaryAllocator;
 #else
-// Currently, the 32-bit Sanitizer allocator has not yet benefited from all the
-// security improvements brought to the 64-bit one. This makes the 32-bit
-// version of Scudo slightly less toughened.
 static const uptr NumRegions = SANITIZER_MMAP_RANGE_SIZE >> RegionSizeLog;
 # if SANITIZER_WORDSIZE == 32
 typedef FlatByteMap<NumRegions> ByteMap;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39303.120300.patch
Type: text/x-patch
Size: 601 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171025/75b1e663/attachment.bin>