ADHOC/replicant-frameworks_native
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge "Revert "Add a compatibility symbol for the MemoryBase constructor.""

Browse Source
This commit is contained in:
Elliott Hughes 2013-03-14 02:56:48 +00:00 committed by Gerrit Code Review
commit d8151306f5
3 changed files with 3 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
include/binder/MemoryBase.h
View File

@ -40,9 +40,6 @@ protected:
const sp<IMemoryHeap>& getHeap() const { return mHeap; }
private:
// Backwards compatibility for libdatabase_sqlcipher (http://b/8253769).
MemoryBase(const sp<IMemoryHeap>& heap, long offset, unsigned int size);
size_t mSize;
ssize_t mOffset;
sp<IMemoryHeap> mHeap;

6
libs/binder/IMemory.cpp
View File

@ -298,11 +298,11 @@ void BpMemoryHeap::assertReallyMapped() const
uint32_t flags = reply.readInt32();
uint32_t offset = reply.readInt32();
ALOGE_IF(err, "binder=%p transaction failed fd=%d, size=%d, err=%d (%s)",
ALOGE_IF(err, "binder=%p transaction failed fd=%d, size=%ld, err=%d (%s)",
asBinder().get(), parcel_fd, size, err, strerror(-err));
int fd = dup( parcel_fd );
ALOGE_IF(fd==-1, "cannot dup fd=%d, size=%d, err=%d (%s)",
ALOGE_IF(fd==-1, "cannot dup fd=%d, size=%ld, err=%d (%s)",
parcel_fd, size, err, strerror(errno));
int access = PROT_READ;
@ -315,7 +315,7 @@ void BpMemoryHeap::assertReallyMapped() const
mRealHeap = true;
mBase = mmap(0, size, access, MAP_SHARED, fd, offset);
if (mBase == MAP_FAILED) {
ALOGE("cannot map BpMemoryHeap (binder=%p), size=%d, fd=%d (%s)",
ALOGE("cannot map BpMemoryHeap (binder=%p), size=%ld, fd=%d (%s)",
asBinder().get(), size, fd, strerror(errno));
close(fd);
} else {

9
libs/binder/MemoryBase.cpp
View File

@ -14,7 +14,6 @@
* limitations under the License.
*/
#define LOG_TAG "MemoryBase"
#include <stdlib.h>
#include <stdint.h>
@ -32,14 +31,6 @@ MemoryBase::MemoryBase(const sp<IMemoryHeap>& heap,
{
}
MemoryBase::MemoryBase(const sp<IMemoryHeap>& heap,
long offset, unsigned int size)
: mSize(size), mOffset(offset), mHeap(heap)
{
ALOGW("Using temporary compatibility workaround for usage of MemoryBase "
"private API. Please fix your application!");
}
sp<IMemoryHeap> MemoryBase::getMemory(ssize_t* offset, size_t* size) const
{
if (offset) *offset = mOffset;