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

Commit 7e1d3956 authored by Iliyan Malchev's avatar Iliyan Malchev
Browse files

libs/utils: replace malloc() + memset() to zero with calloc() 



Change-Id: I8bdf4360147e51e35c162856c9a859aed6acac34
Signed-off-by: default avatarIliyan Malchev <malchev@google.com>
parent 8d25cb8a

libs/utils/BackupHelpers.cpp

+1 −4
Original line number Diff line number Diff line
@@ -546,7 +546,7 @@ int write_tarfile(const String8& packageName, const String8& domain, 


    // read/write up to this much at a time.

    const size_t BUFSIZE = 32 * 1024;

    char* buf = new char[BUFSIZE];

    char* buf = (char *)calloc(1,BUFSIZE);

    char* paxHeader = buf + 512;    // use a different chunk of it as separate scratch

    char* paxData = buf + 1024;
@@ -556,9 +556,6 @@ int write_tarfile(const String8& packageName, const String8& domain, 
        goto cleanup;

    }



    // Good to go -- first construct the standard tar header at the start of the buffer

    memset(buf, 0, BUFSIZE);



    // Magic fields for the ustar file format

    strcat(buf + 257, "ustar");

    strcat(buf + 263, "00");

libs/utils/ResourceTypes.cpp

+3 −6
Original line number Diff line number Diff line
@@ -379,8 +379,7 @@ status_t ResStringPool::setTo(const void* data, size_t size, bool copyData) 
        size_t charSize;

        if (mHeader->flags&ResStringPool_header::UTF8_FLAG) {

            charSize = sizeof(uint8_t);

            mCache = (char16_t**)malloc(sizeof(char16_t**)*mHeader->stringCount);

            memset(mCache, 0, sizeof(char16_t**)*mHeader->stringCount);

            mCache = (char16_t**)calloc(mHeader->stringCount, sizeof(char16_t**));

        } else {

            charSize = sizeof(char16_t);

        }
@@ -3252,16 +3251,14 @@ ssize_t ResTable::getBagLocked(uint32_t resID, const bag_entry** outBag, 


    // Bag not found, we need to compute it!

    if (!grp->bags) {

        grp->bags = (bag_set***)malloc(sizeof(bag_set*)*grp->typeCount);

        grp->bags = (bag_set***)calloc(grp->typeCount, sizeof(bag_set*));

        if (!grp->bags) return NO_MEMORY;

        memset(grp->bags, 0, sizeof(bag_set*)*grp->typeCount);

    }



    bag_set** typeSet = grp->bags[t];

    if (!typeSet) {

        typeSet = (bag_set**)malloc(sizeof(bag_set*)*NENTRY);

        typeSet = (bag_set**)calloc(NENTRY, sizeof(bag_set*));

        if (!typeSet) return NO_MEMORY;

        memset(typeSet, 0, sizeof(bag_set*)*NENTRY);

        grp->bags[t] = typeSet;

    }