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

This reverts commit 7f974ff2f3

Change-Id: I28ebfc5ac88bd025471b613aa7958d23749e6991
This commit is contained in:
Elliott Hughes 2013-03-14 02:56:34 +00:00 committed by Gerrit Code Review
parent 7f974ff2f3
commit 6dbf07d7d5
3 changed files with 3 additions and 15 deletions

View File

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

View File

@ -298,11 +298,11 @@ void BpMemoryHeap::assertReallyMapped() const
uint32_t flags = reply.readInt32(); uint32_t flags = reply.readInt32();
uint32_t offset = 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)); asBinder().get(), parcel_fd, size, err, strerror(-err));
int fd = dup( parcel_fd ); 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)); parcel_fd, size, err, strerror(errno));
int access = PROT_READ; int access = PROT_READ;
@ -315,7 +315,7 @@ void BpMemoryHeap::assertReallyMapped() const
mRealHeap = true; mRealHeap = true;
mBase = mmap(0, size, access, MAP_SHARED, fd, offset); mBase = mmap(0, size, access, MAP_SHARED, fd, offset);
if (mBase == MAP_FAILED) { 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)); asBinder().get(), size, fd, strerror(errno));
close(fd); close(fd);
} else { } else {

View File

@ -14,7 +14,6 @@
* limitations under the License. * limitations under the License.
*/ */
#define LOG_TAG "MemoryBase"
#include <stdlib.h> #include <stdlib.h>
#include <stdint.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 sp<IMemoryHeap> MemoryBase::getMemory(ssize_t* offset, size_t* size) const
{ {
if (offset) *offset = mOffset; if (offset) *offset = mOffset;