diff options
| author | Kostya Kortchinsky <kostyak@google.com> | 2018-07-03 19:03:46 +0000 |
|---|---|---|
| committer | Kostya Kortchinsky <kostyak@google.com> | 2018-07-03 19:03:46 +0000 |
| commit | e64a81475c3e705d35cf74c451ced12ee0dc9ffb (patch) | |
| tree | 7254cbe1ad664302d60a2d51722c4f927ba89edb /compiler-rt/lib | |
| parent | 181aa26eb87070a109ab7173c2d7916a448c2100 (diff) | |
| download | bcm5719-llvm-e64a81475c3e705d35cf74c451ced12ee0dc9ffb.tar.gz bcm5719-llvm-e64a81475c3e705d35cf74c451ced12ee0dc9ffb.zip | |
[scudo] Get rid of builtin-declaration-mismatch warnings
Summary:
The C interceptors were using `SIZE_T` defined in the interception library as
a `__sanitizer::uptr`. On some 32-bit platforms, this lead to the following
warning:
```
warning: declaration of ‘void* malloc(SIZE_T)’ conflicts with built-in declaration ‘void* malloc(unsigned int)’ [-Wbuiltin-declaration-mismatch]
INTERCEPTOR_ATTRIBUTE void *malloc(SIZE_T size) {
```
`__sanitizer::uptr` is indeed defined as an `unsigned long` on those.
So just include `stddef.h` and use `size_t` instead.
Reviewers: alekseyshl
Reviewed By: alekseyshl
Subscribers: delcypher, llvm-commits, #sanitizers
Differential Revision: https://reviews.llvm.org/D48885
llvm-svn: 336221
Diffstat (limited to 'compiler-rt/lib')
| -rw-r--r-- | compiler-rt/lib/scudo/scudo_malloc.cpp | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/compiler-rt/lib/scudo/scudo_malloc.cpp b/compiler-rt/lib/scudo/scudo_malloc.cpp index 6e83be42224..91a77b36582 100644 --- a/compiler-rt/lib/scudo/scudo_malloc.cpp +++ b/compiler-rt/lib/scudo/scudo_malloc.cpp @@ -16,6 +16,8 @@ #include "interception/interception.h" #include "sanitizer_common/sanitizer_platform_interceptors.h" +#include <stddef.h> + using namespace __scudo; extern "C" { @@ -23,24 +25,24 @@ INTERCEPTOR_ATTRIBUTE void free(void *ptr) { scudoDeallocate(ptr, 0, 0, FromMalloc); } -INTERCEPTOR_ATTRIBUTE void *malloc(SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *malloc(size_t size) { return scudoAllocate(size, 0, FromMalloc); } -INTERCEPTOR_ATTRIBUTE void *realloc(void *ptr, SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *realloc(void *ptr, size_t size) { return scudoRealloc(ptr, size); } -INTERCEPTOR_ATTRIBUTE void *calloc(SIZE_T nmemb, SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *calloc(size_t nmemb, size_t size) { return scudoCalloc(nmemb, size); } -INTERCEPTOR_ATTRIBUTE void *valloc(SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *valloc(size_t size) { return scudoValloc(size); } INTERCEPTOR_ATTRIBUTE -int posix_memalign(void **memptr, SIZE_T alignment, SIZE_T size) { +int posix_memalign(void **memptr, size_t alignment, size_t size) { return scudoPosixMemalign(memptr, alignment, size); } @@ -49,28 +51,28 @@ INTERCEPTOR_ATTRIBUTE void cfree(void *ptr) ALIAS("free"); #endif #if SANITIZER_INTERCEPT_MEMALIGN -INTERCEPTOR_ATTRIBUTE void *memalign(SIZE_T alignment, SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *memalign(size_t alignment, size_t size) { return scudoAllocate(size, alignment, FromMemalign); } INTERCEPTOR_ATTRIBUTE -void *__libc_memalign(SIZE_T alignment, SIZE_T size) ALIAS("memalign"); +void *__libc_memalign(size_t alignment, size_t size) ALIAS("memalign"); #endif #if SANITIZER_INTERCEPT_PVALLOC -INTERCEPTOR_ATTRIBUTE void *pvalloc(SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *pvalloc(size_t size) { return scudoPvalloc(size); } #endif #if SANITIZER_INTERCEPT_ALIGNED_ALLOC -INTERCEPTOR_ATTRIBUTE void *aligned_alloc(SIZE_T alignment, SIZE_T size) { +INTERCEPTOR_ATTRIBUTE void *aligned_alloc(size_t alignment, size_t size) { return scudoAlignedAlloc(alignment, size); } #endif #if SANITIZER_INTERCEPT_MALLOC_USABLE_SIZE -INTERCEPTOR_ATTRIBUTE SIZE_T malloc_usable_size(void *ptr) { +INTERCEPTOR_ATTRIBUTE size_t malloc_usable_size(void *ptr) { return scudoMallocUsableSize(ptr); } #endif |

