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

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

am 6be23712: Merge "Free created FileMap when uncompressing files" into gingerbread 

Merge commit '6be237120e9673e3cfc5243da5bbca287effcf09' into gingerbread-plus-aosp

* commit '6be237120e9673e3cfc5243da5bbca287effcf09':
  Free created FileMap when uncompressing files
parents 0ba08fc3 e6d429c4

libs/utils/ZipFileRO.cpp

+9 −5
Original line number Diff line number Diff line
@@ -636,7 +636,7 @@ bool ZipFileRO::uncompressEntry(ZipEntryRO entry, void* buffer) const 
        memcpy(buffer, ptr, uncompLen);

    } else {

        if (!inflateBuffer(buffer, ptr, uncompLen, compLen))

            goto bail;

            goto unmap;

    }



    if (compLen > kSequentialMin)
@@ -644,6 +644,8 @@ bool ZipFileRO::uncompressEntry(ZipEntryRO entry, void* buffer) const 


    result = true;



unmap:

    file->release();

bail:

    return result;

}
@@ -667,7 +669,7 @@ bool ZipFileRO::uncompressEntry(ZipEntryRO entry, int fd) const 


    getEntryInfo(entry, &method, &uncompLen, &compLen, &offset, NULL, NULL);



    const FileMap* file = createEntryFileMap(entry);

    FileMap* file = createEntryFileMap(entry);

    if (file == NULL) {

        goto bail;

    }
@@ -678,21 +680,23 @@ bool ZipFileRO::uncompressEntry(ZipEntryRO entry, int fd) const 
        ssize_t actual = write(fd, ptr, uncompLen);

        if (actual < 0) {

            LOGE("Write failed: %s\n", strerror(errno));

            goto bail;

            goto unmap;

        } else if ((size_t) actual != uncompLen) {

            LOGE("Partial write during uncompress (%zd of %zd)\n",

                (size_t)actual, (size_t)uncompLen);

            goto bail;

            goto unmap;

        } else {

            LOGI("+++ successful write\n");

        }

    } else {

        if (!inflateBuffer(fd, ptr, uncompLen, compLen))

            goto bail;

            goto unmap;

    }



    result = true;



unmap:

    file->release();

bail:

    return result;

}