[PATCH] D16805: Cast the fifth arg to mremap to void *
H.J Lu via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 2 07:44:05 PST 2016
hjl.tools created this revision.
hjl.tools added reviewers: kcc, eugenis.
hjl.tools added a subscriber: llvm-commits.
Since the prototype of mremap is
```
void *mremap(void *old_address, size_t old_size, size_t new_size,
int flags, ... /* void *new_address*/);
```
we need to cast new_address to void * when calling mremap. Otherwise,
the wrong value will be passed to mremap on x32.
http://reviews.llvm.org/D16805
Files:
lib/cfi/cfi.cc
Index: lib/cfi/cfi.cc
===================================================================
--- lib/cfi/cfi.cc
+++ lib/cfi/cfi.cc
@@ -153,7 +153,7 @@
if (main_shadow) {
// Update.
void *res = mremap((void *)shadow_, GetShadowSize(), GetShadowSize(),
- MREMAP_MAYMOVE | MREMAP_FIXED, main_shadow);
+ MREMAP_MAYMOVE | MREMAP_FIXED, (void *)main_shadow);
CHECK(res != MAP_FAILED);
} else {
// Initial setup.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16805.46653.patch
Type: text/x-patch
Size: 473 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160202/8e055448/attachment.bin>
More information about the llvm-commits
mailing list