diff options
| author | 2016-02-17 19:23:03 +0000 | |
|---|---|---|
| committer | 2016-02-17 19:23:03 +0000 | |
| commit | f9e5e80bc1fe9255fa65b7185514d86beee5952a (patch) | |
| tree | 3e6b8afbb63d57306aa35cc0cf1fe4663ffeab18 | |
| parent | 40ba03fc68e6c234bb8604784716cb2481ad1c67 (diff) | |
| parent | fc598c54d945c48ade547a52a01e2e704df1618b (diff) | |
Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de am: 84f488f354 am: ebdad60d6b
am: fc598c54d9
* commit 'fc598c54d945c48ade547a52a01e2e704df1618b':
Sanity check IMemory access versus underlying mmap
| -rw-r--r-- | libs/binder/IMemory.cpp | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/libs/binder/IMemory.cpp b/libs/binder/IMemory.cpp index e9891a830d..fb8d620b4c 100644 --- a/libs/binder/IMemory.cpp +++ b/libs/binder/IMemory.cpp @@ -26,6 +26,7 @@ #include <sys/mman.h> #include <binder/IMemory.h> +#include <cutils/log.h> #include <utils/KeyedVector.h> #include <utils/threads.h> #include <utils/Atomic.h> @@ -187,15 +188,26 @@ sp<IMemoryHeap> BpMemory::getMemory(ssize_t* offset, size_t* size) const if (heap != 0) { mHeap = interface_cast<IMemoryHeap>(heap); if (mHeap != 0) { - mOffset = o; - mSize = s; + size_t heapSize = mHeap->getSize(); + if (s <= heapSize + && o >= 0 + && (static_cast<size_t>(o) <= heapSize - s)) { + mOffset = o; + mSize = s; + } else { + // Hm. + android_errorWriteWithInfoLog(0x534e4554, + "26877992", -1, NULL, 0); + mOffset = 0; + mSize = 0; + } } } } } if (offset) *offset = mOffset; if (size) *size = mSize; - return mHeap; + return (mSize > 0) ? mHeap : 0; } // --------------------------------------------------------------------------- |