Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(298)

Side by Side Diff: third_party/tcmalloc/chromium/src/system-alloc.cc

Issue 12093035: TCMalloc: support userland ASLR on Linux (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ifdef PRNG code Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« base/security_unittest.cc ('K') | « base/security_unittest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2005, Google Inc. 1 // Copyright (c) 2005, Google Inc.
2 // All rights reserved. 2 // All rights reserved.
3 // 3 //
4 // Redistribution and use in source and binary forms, with or without 4 // Redistribution and use in source and binary forms, with or without
5 // modification, are permitted provided that the following conditions are 5 // modification, are permitted provided that the following conditions are
6 // met: 6 // met:
7 // 7 //
8 // * Redistributions of source code must retain the above copyright 8 // * Redistributions of source code must retain the above copyright
9 // notice, this list of conditions and the following disclaimer. 9 // notice, this list of conditions and the following disclaimer.
10 // * Redistributions in binary form must reproduce the above 10 // * Redistributions in binary form must reproduce the above
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 // Check that no bit is set at position ADDRESS_BITS or higher. 93 // Check that no bit is set at position ADDRESS_BITS or higher.
94 template <int ADDRESS_BITS> bool CheckAddressBits(uintptr_t ptr) { 94 template <int ADDRESS_BITS> bool CheckAddressBits(uintptr_t ptr) {
95 return (ptr >> ADDRESS_BITS) == 0; 95 return (ptr >> ADDRESS_BITS) == 0;
96 } 96 }
97 97
98 // Specialize for the bit width of a pointer to avoid undefined shift. 98 // Specialize for the bit width of a pointer to avoid undefined shift.
99 template <> bool CheckAddressBits<8 * sizeof(void*)>(uintptr_t ptr) { 99 template <> bool CheckAddressBits<8 * sizeof(void*)>(uintptr_t ptr) {
100 return true; 100 return true;
101 } 101 }
102 102
103 #if (defined(OS_LINUX) || defined(OS_CHROMEOS)) && defined(__x86_64__)
104 #define ASLR_IS_SUPPORTED
105 #endif
106
107 #if defined(ASLR_IS_SUPPORTED)
108 // From libdieharder, public domain library by Bob Jenkins (rngav.c).
109 // Described at http://burtleburtle.net/bob/rand/smallprng.html.
110 // Not cryptographically secure, but good enough for what we need.
111 typedef uint32_t u4;
112 typedef struct ranctx { u4 a; u4 b; u4 c; u4 d; } ranctx;
jar (doing other things) 2013/01/30 17:20:48 nit: Please reformat the structure definition with
jln (very slow on Chromium) 2013/01/30 20:11:33 Done.
113
114 #define rot(x,k) (((x)<<(k))|((x)>>(32-(k))))
115
116 u4 ranval(ranctx* x) {
117 /* xxx: the generator being tested */
118 u4 e = x->a - rot(x->b, 27);
jar (doing other things) 2013/01/30 17:20:48 I was bothered by the use of single character name
119 x->a = x->b ^ rot(x->c, 17);
120 x->b = x->c + x->d;
121 x->c = x->d + e;
122 x->d = e + x->a;
123 return x->d;
124 }
125
126 void raninit(ranctx* x, u4 seed) {
127 u4 i;
128 x->a = 0xf1ea5eed, x->b = x->c = x->d = seed;
jar (doing other things) 2013/01/30 17:20:48 nit: Please reformat that line to comply with styl
jln (very slow on Chromium) 2013/01/30 20:11:33 Done.
129 for (i=0; i<20; ++i) {
130 (void)ranval(x);
131 }
132 }
133
134 #endif // defined(ASLR_IS_SUPPORTED)
135
136 // Give a random "hint" that is suitable for use with mmap(). This cannot make
137 // mmap fail, as the kernel will simply not follow the hint if it can't.
138 // However, this will create address space fragmentation. Currently, we only
139 // implement it on x86_64, where we have a 47 bits userland address space and
140 // fragmentation is not an issue.
141 void* GetRandomAddrHint() {
142 #if !defined(ASLR_IS_SUPPORTED)
143 return NULL;
144 #else
145 // Note: we are protected by the general TCMalloc_SystemAlloc spinlock. Given
146 // the nature of what we're doing, it wouldn't be critical if we weren't for
147 // ctx, but it is for the "initialized" variable.
148 // It's nice to share the state between threads, because scheduling will add
149 // some randomness to the succession of ranval() calls.
150 static ranctx ctx;
151 static bool initialized = false;
152 if (!initialized) {
153 initialized = true;
154 // We really want this to be a stack variable and don't want any compiler
155 // optimization. We're using its address as a poor-man source of
156 // randomness.
157 volatile char c;
158 // Pre-initialize our seed with a "random" address in case /dev/urandom is
159 // not available.
160 uint32_t seed = (reinterpret_cast<uint64_t>(&c) >> 32) ^
161 reinterpret_cast<uint64_t>(&c);
162 int urandom_fd = open("/dev/urandom", O_RDONLY);
163 if (urandom_fd >= 0) {
164 ssize_t len;
165 len = read(urandom_fd, &seed, sizeof(seed));
166 ASSERT(len == sizeof(seed));
167 int ret = close(urandom_fd);
168 ASSERT(ret == 0);
169 }
170 raninit(&ctx, seed);
171 }
172 uint64_t random_address = (static_cast<uint64_t>(ranval(&ctx)) << 32) |
173 ranval(&ctx);
174 // If the kernel cannot honor the hint in arch_get_unmapped_area_topdown, it
175 // will simply ignore it. So we give a hint that has a good chance of
176 // working.
177 // The mmap top-down allocator will normally allocate below TASK_SIZE - gap,
178 // with a gap that depends on the max stack size. See x86/mm/mmap.c. We
179 // should make allocations that are below this area, which would be
180 // 0x7ffbf8000000.
181 // We use 0x3ffffffff000 as the mask so that we only "pollute" half of the
182 // address space. In the unlikely case where fragmentation would become an
183 // issue, the kernel will still have another half to use.
184 // A a bit-wise "and" won't bias our random distribution.
185 random_address &= 0x3ffffffff000ULL;
186 return reinterpret_cast<void*>(random_address);
187 #endif // ASLR_IS_SUPPORTED
188 }
189
103 } // Anonymous namespace to avoid name conflicts on "CheckAddressBits". 190 } // Anonymous namespace to avoid name conflicts on "CheckAddressBits".
104 191
105 COMPILE_ASSERT(kAddressBits <= 8 * sizeof(void*), 192 COMPILE_ASSERT(kAddressBits <= 8 * sizeof(void*),
106 address_bits_larger_than_pointer_size); 193 address_bits_larger_than_pointer_size);
107 194
108 // Structure for discovering alignment 195 // Structure for discovering alignment
109 union MemoryAligner { 196 union MemoryAligner {
110 void* p; 197 void* p;
111 double d; 198 double d;
112 size_t s; 199 size_t s;
(...skipping 19 matching lines...) Expand all
132 EnvToInt("TCMALLOC_DEVMEM_LIMIT", 0), 219 EnvToInt("TCMALLOC_DEVMEM_LIMIT", 0),
133 "Physical memory limit location in MB for /dev/mem allocation." 220 "Physical memory limit location in MB for /dev/mem allocation."
134 " Setting this to 0 means no limit."); 221 " Setting this to 0 means no limit.");
135 DEFINE_bool(malloc_skip_sbrk, 222 DEFINE_bool(malloc_skip_sbrk,
136 EnvToBool("TCMALLOC_SKIP_SBRK", false), 223 EnvToBool("TCMALLOC_SKIP_SBRK", false),
137 "Whether sbrk can be used to obtain memory."); 224 "Whether sbrk can be used to obtain memory.");
138 DEFINE_bool(malloc_skip_mmap, 225 DEFINE_bool(malloc_skip_mmap,
139 EnvToBool("TCMALLOC_SKIP_MMAP", false), 226 EnvToBool("TCMALLOC_SKIP_MMAP", false),
140 "Whether mmap can be used to obtain memory."); 227 "Whether mmap can be used to obtain memory.");
141 228
229 DEFINE_bool(malloc_random_allocator,
230 #if defined(ASLR_IS_SUPPORTED)
231 EnvToBool("TCMALLOC_ASLR", true),
232 #else
233 EnvToBool("TCMALLOC_ASLR", false),
234 #endif
235 "Whether to randomize the address space via mmap().");
236
142 // static allocators 237 // static allocators
143 class SbrkSysAllocator : public SysAllocator { 238 class SbrkSysAllocator : public SysAllocator {
144 public: 239 public:
145 SbrkSysAllocator() : SysAllocator() { 240 SbrkSysAllocator() : SysAllocator() {
146 } 241 }
147 void* Alloc(size_t size, size_t *actual_size, size_t alignment); 242 void* Alloc(size_t size, size_t *actual_size, size_t alignment);
148 }; 243 };
149 static char sbrk_space[sizeof(SbrkSysAllocator)]; 244 static char sbrk_space[sizeof(SbrkSysAllocator)];
150 245
151 class MmapSysAllocator : public SysAllocator { 246 class MmapSysAllocator : public SysAllocator {
(...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 // Ask for extra memory if alignment > pagesize 392 // Ask for extra memory if alignment > pagesize
298 size_t extra = 0; 393 size_t extra = 0;
299 if (alignment > pagesize) { 394 if (alignment > pagesize) {
300 extra = alignment - pagesize; 395 extra = alignment - pagesize;
301 } 396 }
302 397
303 // Note: size + extra does not overflow since: 398 // Note: size + extra does not overflow since:
304 // size + alignment < (1<<NBITS). 399 // size + alignment < (1<<NBITS).
305 // and extra <= alignment 400 // and extra <= alignment
306 // therefore size + extra < (1<<NBITS) 401 // therefore size + extra < (1<<NBITS)
307 void* result = mmap(NULL, size + extra, 402 void* address_hint = NULL;
403 if (FLAGS_malloc_random_allocator) {
404 address_hint = GetRandomAddrHint();
405 }
406 void* result = mmap(address_hint, size + extra,
308 PROT_READ|PROT_WRITE, 407 PROT_READ|PROT_WRITE,
309 MAP_PRIVATE|MAP_ANONYMOUS, 408 MAP_PRIVATE|MAP_ANONYMOUS,
310 -1, 0); 409 -1, 0);
311 if (result == reinterpret_cast<void*>(MAP_FAILED)) { 410 if (result == reinterpret_cast<void*>(MAP_FAILED)) {
312 return NULL; 411 return NULL;
313 } 412 }
314 413
315 // Adjust the return memory so it is aligned 414 // Adjust the return memory so it is aligned
316 uintptr_t ptr = reinterpret_cast<uintptr_t>(result); 415 uintptr_t ptr = reinterpret_cast<uintptr_t>(result);
317 size_t adjust = 0; 416 size_t adjust = 0;
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
446 SbrkSysAllocator *sbrk = new (sbrk_space) SbrkSysAllocator(); 545 SbrkSysAllocator *sbrk = new (sbrk_space) SbrkSysAllocator();
447 546
448 // In 64-bit debug mode, place the mmap allocator first since it 547 // In 64-bit debug mode, place the mmap allocator first since it
449 // allocates pointers that do not fit in 32 bits and therefore gives 548 // allocates pointers that do not fit in 32 bits and therefore gives
450 // us better testing of code's 64-bit correctness. It also leads to 549 // us better testing of code's 64-bit correctness. It also leads to
451 // less false negatives in heap-checking code. (Numbers are less 550 // less false negatives in heap-checking code. (Numbers are less
452 // likely to look like pointers and therefore the conservative gc in 551 // likely to look like pointers and therefore the conservative gc in
453 // the heap-checker is less likely to misinterpret a number as a 552 // the heap-checker is less likely to misinterpret a number as a
454 // pointer). 553 // pointer).
455 DefaultSysAllocator *sdef = new (default_space) DefaultSysAllocator(); 554 DefaultSysAllocator *sdef = new (default_space) DefaultSysAllocator();
555 // Unfortunately, this code runs before flags are initialized. So
556 // we can't use FLAGS_malloc_random_allocator.
557 #if defined(ASLR_IS_SUPPORTED)
558 // Our only random allocator is mmap.
559 sdef->SetChildAllocator(mmap, 0, mmap_name);
560 #else
456 if (kDebugMode && sizeof(void*) > 4) { 561 if (kDebugMode && sizeof(void*) > 4) {
457 sdef->SetChildAllocator(mmap, 0, mmap_name); 562 sdef->SetChildAllocator(mmap, 0, mmap_name);
458 sdef->SetChildAllocator(sbrk, 1, sbrk_name); 563 sdef->SetChildAllocator(sbrk, 1, sbrk_name);
459 } else { 564 } else {
460 sdef->SetChildAllocator(sbrk, 0, sbrk_name); 565 sdef->SetChildAllocator(sbrk, 0, sbrk_name);
461 sdef->SetChildAllocator(mmap, 1, mmap_name); 566 sdef->SetChildAllocator(mmap, 1, mmap_name);
462 } 567 }
568 #endif // ASLR_IS_SUPPORTED
463 sys_alloc = sdef; 569 sys_alloc = sdef;
464 } 570 }
465 571
466 void* TCMalloc_SystemAlloc(size_t size, size_t *actual_size, 572 void* TCMalloc_SystemAlloc(size_t size, size_t *actual_size,
467 size_t alignment) { 573 size_t alignment) {
468 // Discard requests that overflow 574 // Discard requests that overflow
469 if (size + alignment < size) return NULL; 575 if (size + alignment < size) return NULL;
470 576
471 SpinLockHolder lock_holder(&spinlock); 577 SpinLockHolder lock_holder(&spinlock);
472 578
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
540 } 646 }
541 } 647 }
542 #endif 648 #endif
543 } 649 }
544 650
545 void TCMalloc_SystemCommit(void* start, size_t length) { 651 void TCMalloc_SystemCommit(void* start, size_t length) {
546 // Nothing to do here. TCMalloc_SystemRelease does not alter pages 652 // Nothing to do here. TCMalloc_SystemRelease does not alter pages
547 // such that they need to be re-committed before they can be used by the 653 // such that they need to be re-committed before they can be used by the
548 // application. 654 // application.
549 } 655 }
OLDNEW
« base/security_unittest.cc ('K') | « base/security_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698