Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d441a0ca authored by Kenny Root's avatar Kenny Root Committed by Android Git Automerger
Browse files

am ceb8465e: am 87bb1857: Merge changes I2337051b,I19b426cb into gingerbread 

Merge commit 'ceb8465e974ba5fa971ffe1b5b74e94a1bd53f97'

* commit 'ceb8465e974ba5fa971ffe1b5b74e94a1bd53f97':
  Tweak ObbFile class
  Allow things that can install packages to set Obb paths
parents 08e74c7e 723499ae

libs/utils/ObbFile.cpp

+25 −13
Original line number Original line Diff line number Diff line
@@ -91,22 +91,24 @@ bool ObbFile::readFrom(const char* filename) 




    fd = ::open(filename, O_RDONLY);

    fd = ::open(filename, O_RDONLY);

    if (fd < 0) {

    if (fd < 0) {

        LOGW("couldn't open file %s: %s", filename, strerror(errno));

        goto out;

        goto out;

    }

    }

    success = readFrom(fd);

    success = readFrom(fd);

    close(fd);

    close(fd);





out:

    if (!success) {

    if (!success) {

        LOGW("failed to read from %s\n", filename);

        LOGW("failed to read from %s (fd=%d)\n", filename, fd);

    }

    }



out:

    return success;

    return success;

}

}





bool ObbFile::readFrom(int fd)

bool ObbFile::readFrom(int fd)

{

{

    if (fd < 0) {

    if (fd < 0) {

        LOGW("failed to read file\n");

        LOGW("attempt to read from invalid fd\n");

        return false;

        return false;

    }

    }
@@ -149,10 +151,16 @@ bool ObbFile::parseObbFile(int fd) 
        footerSize = get4LE((unsigned char*)footer);

        footerSize = get4LE((unsigned char*)footer);

        if (footerSize > (size_t)fileLength - kFooterTagSize

        if (footerSize > (size_t)fileLength - kFooterTagSize

                || footerSize > kMaxBufSize) {

                || footerSize > kMaxBufSize) {

            LOGW("claimed footer size is too large (0x%08lx; file size is 0x%08llx)\n",

            LOGW("claimed footer size is too large (0x%08zx; file size is 0x%08llx)\n",

                    footerSize, fileLength);

                    footerSize, fileLength);

            return false;

            return false;

        }

        }



        if (footerSize < kFooterMinSize) {

            LOGW("claimed footer size is too small (%08zx; minimum size is 0x%x)\n",

                    footerSize, kFooterMinSize);

            return false;

        }

    }

    }





    my_off64_t fileOffset = fileLength - footerSize - kFooterTagSize;

    my_off64_t fileOffset = fileLength - footerSize - kFooterTagSize;
@@ -161,26 +169,22 @@ bool ObbFile::parseObbFile(int fd) 
        return false;

        return false;

    }

    }





    size_t readAmount = kMaxBufSize;

    char* scanBuf = (char*)malloc(footerSize);

    if (readAmount > footerSize)

        readAmount = footerSize;



    char* scanBuf = (char*)malloc(readAmount);

    if (scanBuf == NULL) {

    if (scanBuf == NULL) {

        LOGW("couldn't allocate scanBuf: %s\n", strerror(errno));

        LOGW("couldn't allocate scanBuf: %s\n", strerror(errno));

        return false;

        return false;

    }

    }





    actual = TEMP_FAILURE_RETRY(read(fd, scanBuf, readAmount));

    actual = TEMP_FAILURE_RETRY(read(fd, scanBuf, footerSize));

    // readAmount is guaranteed to be less than kMaxBufSize

    // readAmount is guaranteed to be less than kMaxBufSize

    if (actual != (ssize_t)readAmount) {

    if (actual != (ssize_t)footerSize) {

        LOGI("couldn't read ObbFile footer: %s\n", strerror(errno));

        LOGI("couldn't read ObbFile footer: %s\n", strerror(errno));

        free(scanBuf);

        free(scanBuf);

        return false;

        return false;

    }

    }





#ifdef DEBUG

#ifdef DEBUG

    for (int i = 0; i < readAmount; ++i) {

    for (int i = 0; i < footerSize; ++i) {

        LOGI("char: 0x%02x", scanBuf[i]);

        LOGI("char: 0x%02x", scanBuf[i]);

    }

    }

#endif

#endif
@@ -197,7 +201,8 @@ bool ObbFile::parseObbFile(int fd) 
    uint32_t packageNameLen = get4LE((unsigned char*)scanBuf + kPackageNameLenOffset);

    uint32_t packageNameLen = get4LE((unsigned char*)scanBuf + kPackageNameLenOffset);

    if (packageNameLen <= 0

    if (packageNameLen <= 0

            || packageNameLen > (footerSize - kPackageNameOffset)) {

            || packageNameLen > (footerSize - kPackageNameOffset)) {

        LOGW("bad ObbFile package name length (0x%08x)\n", packageNameLen);

        LOGW("bad ObbFile package name length (0x%04x; 0x%04x possible)\n",

                packageNameLen, footerSize - kPackageNameOffset);

        free(scanBuf);

        free(scanBuf);

        return false;

        return false;

    }

    }
@@ -206,6 +211,11 @@ bool ObbFile::parseObbFile(int fd) 
    mPackageName = String8(const_cast<char*>(packageName), packageNameLen);

    mPackageName = String8(const_cast<char*>(packageName), packageNameLen);





    free(scanBuf);

    free(scanBuf);



#ifdef DEBUG

    LOGI("Obb scan succeeded: packageName=%s, version=%d\n", mPackageName.string(), mVersion);

#endif



    return true;

    return true;

}

}
@@ -234,6 +244,8 @@ bool ObbFile::writeTo(int fd) 
        return false;

        return false;

    }

    }





    my_lseek64(fd, 0, SEEK_END);



    if (mPackageName.size() == 0 || mVersion == -1) {

    if (mPackageName.size() == 0 || mVersion == -1) {

        LOGW("tried to write uninitialized ObbFile data");

        LOGW("tried to write uninitialized ObbFile data");

        return false;

        return false;

libs/utils/tests/ObbFile_test.cpp

+11 −4
Original line number Original line Diff line number Diff line
@@ -22,6 +22,8 @@ 




#include <gtest/gtest.h>

#include <gtest/gtest.h>





#include <fcntl.h>



namespace android {

namespace android {





#define TEST_FILENAME "/test.obb"

#define TEST_FILENAME "/test.obb"
@@ -39,6 +41,11 @@ protected: 
        const int totalLen = strlen(mExternalStorage) + strlen(TEST_FILENAME) + 1;

        const int totalLen = strlen(mExternalStorage) + strlen(TEST_FILENAME) + 1;

        mFileName = new char[totalLen];

        mFileName = new char[totalLen];

        snprintf(mFileName, totalLen, "%s%s", mExternalStorage, TEST_FILENAME);

        snprintf(mFileName, totalLen, "%s%s", mExternalStorage, TEST_FILENAME);



        int fd = ::open(mFileName, O_CREAT | O_TRUNC);

        if (fd < 0) {

            FAIL() << "Couldn't create " << mFileName << " for tests";

        }

    }

    }





    virtual void TearDown() {

    virtual void TearDown() {