Ticket #9885: 0013-Use-delete-instead-of-delete.patch

File 0013-Use-delete-instead-of-delete.patch, 5.7 KB (added by mt, 6 years ago)

Patch

  • src/apps/packageinstaller/PackageInfo.cpp

    From 4c47459a01ed1469383bd02e3f06e0fa38aff6b0 Mon Sep 17 00:00:00 2001
    From: Murai Takashi <tmurai01@gmail.com>
    Date: Sat, 20 Jul 2013 07:21:58 +0900
    Subject: [PATCH] Use delete[] instead of delete
    
    ---
     src/apps/packageinstaller/PackageInfo.cpp |   40 ++++++++++++++---------------
     1 file changed, 20 insertions(+), 20 deletions(-)
    
    diff --git a/src/apps/packageinstaller/PackageInfo.cpp b/src/apps/packageinstaller/PackageInfo.cpp
    index 9354934..c94d49f 100644
    a b PackageInfo::Parse()  
    255255            uint8 *compressed = new uint8[length];
    256256            if (fPackageFile->Read(compressed, length)
    257257                    != static_cast<int64>(length)) {
    258                 delete compressed;
     258                delete[] compressed;
    259259                RETURN_AND_SET_STATUS(B_ERROR);
    260260            }
    261261
    PackageInfo::Parse()  
    263263            status_t ret = inflate_data(compressed, length, disclaimer,
    264264                original);
    265265            disclaimer[original] = 0;
    266             delete compressed;
     266            delete[] compressed;
    267267            if (ret != B_OK) {
    268                 delete disclaimer;
     268                delete[] disclaimer;
    269269                RETURN_AND_SET_STATUS(B_ERROR);
    270270            }
    271271
    272272            fDisclaimer = (char *)disclaimer;
    273             delete disclaimer;
     273            delete[] disclaimer;
    274274
    275275            continue;
    276276        } else if (!memcmp(buffer, splashScreenMarker, 7)) {
    PackageInfo::Parse()  
    291291            uint8 *compressed = new uint8[length];
    292292            if (fPackageFile->Read(compressed, length)
    293293                    != static_cast<int64>(length)) {
    294                 delete compressed;
     294                delete[] compressed;
    295295                RETURN_AND_SET_STATUS(B_ERROR);
    296296            }
    297297
    PackageInfo::Parse()  
    299299            status_t ret = inflate_data(compressed, length,
    300300                static_cast<uint8 *>(const_cast<void *>(fImage.Buffer())),
    301301                original);
    302             delete compressed;
     302            delete[] compressed;
    303303            if (ret != B_OK) {
    304304                RETURN_AND_SET_STATUS(B_ERROR);
    305305            }
    PackageInfo::Parse()  
    370370                    fPackageFile->Read(name, length);
    371371                    name[length] = 0;
    372372                    group.name = name;
    373                     delete name;
     373                    delete[] name;
    374374                } else if (!memcmp(buffer, "GrpD", 5)) {
    375375                    if (!groupStarted) {
    376376                        RETURN_AND_SET_STATUS(B_ERROR);
    PackageInfo::Parse()  
    385385                    fPackageFile->Read(desc, length);
    386386                    desc[length] = 0;
    387387                    group.description = desc;
    388                     delete desc;
     388                    delete[] desc;
    389389                } else if (!memcmp(buffer, "GrHt", 5)) {
    390390                    if (!groupStarted) {
    391391                        RETURN_AND_SET_STATUS(B_ERROR);
    PackageInfo::Parse()  
    465465                    }
    466466                    fDescription.Remove(0, i);
    467467
    468                     delete description;
     468                    delete[] description;
    469469                    parser_debug("Description text reached\n");
    470470
    471471                    // After this, there's a known size sequence of bytes, which
    PackageInfo::Parse()  
    494494                    fPackageFile->Read(name, length);
    495495                    name[length] = 0;
    496496                    fName = name;
    497                     delete name;
     497                    delete[] name;
    498498                } else if (!memcmp(buffer, versionMarker, 7)) {
    499499                    parser_debug("Package version reached\n");
    500500                    fPackageFile->Read(&length, 4);
    PackageInfo::Parse()  
    505505                    fPackageFile->Read(version, length);
    506506                    version[length] = 0;
    507507                    fVersion = version;
    508                     delete version;
     508                    delete[] version;
    509509                } else if (!memcmp(buffer, devMarker, 7)) {
    510510                    parser_debug("Package developer reached\n");
    511511                    fPackageFile->Read(&length, 4);
    PackageInfo::Parse()  
    516516                    fPackageFile->Read(dev, length);
    517517                    dev[length] = 0;
    518518                    fDeveloper = dev;
    519                     delete dev;
     519                    delete[] dev;
    520520                } else if (!memcmp(buffer, shortDescMarker, 7)) {
    521521                    parser_debug("Package short description reached\n");
    522522                    fPackageFile->Read(&length, 4);
    PackageInfo::Parse()  
    527527                    fPackageFile->Read(desc, length);
    528528                    desc[length] = 0;
    529529                    fShortDesc = desc;
    530                     delete desc;
     530                    delete[] desc;
    531531                } else if (!memcmp(buffer, helpMarker, 7)) {
    532532                    // The help text is a stored in deflated state, preceded by a 64 bit
    533533                    // compressed size, 64 bit inflated size and a 32 bit integer
    PackageInfo::Parse()  
    564564                    fPackageFile->Read(ti, length);
    565565                    ti[length] = 0;
    566566                    parser_debug("DQTi - %s\n", ti);
    567                     delete ti;
     567                    delete[] ti;
    568568                } else if (!memcmp(buffer, "DQSz", 5)) {
    569569                    parser_debug("DQSz\n");
    570570                    uint64 size;
    PackageInfo::Parse()  
    588588                    fPackageFile->Read(signature, length);
    589589                    signature[length] = 0;
    590590                    parser_debug("DQMi - %s\n", signature);
    591                     delete signature;
     591                    delete[] signature;
    592592                } else if (!memcmp(buffer, "PaNa", 5)) {
    593593                    parser_debug("PaNa\n");
    594594                    fPackageFile->Read(&length, 4);
    PackageInfo::Parse()  
    602602                    if (length > 0 && pathname[length - 1] == '/')
    603603                        path->Remove(length - 1, 1);
    604604                    userPaths.AddItem(path);
    605                     delete pathname;
     605                    delete[] pathname;
    606606                } else if (!memcmp(buffer, padding, 7)) {
    607607                    parser_debug("Padding!\n");
    608608                    continue;
    PackageInfo::Parse()  
    675675            name[length] = 0;
    676676
    677677            nameString = name;
    678             delete name;
     678            delete[] name;
    679679        } else if (!memcmp(buffer, "Grps", 5)) {
    680680            if (element == P_NONE) {
    681681                RETURN_AND_SET_STATUS(B_ERROR);
    PackageInfo::Parse()  
    763763            parser_debug("Mime: %s\n", mime);
    764764
    765765            mimeString = mime;
    766             delete mime;
     766            delete[] mime;
    767767        } else if (!memcmp(buffer, "CmpS", 5)) {
    768768            if (element == P_NONE) {
    769769                RETURN_AND_SET_STATUS(B_ERROR);
    PackageInfo::Parse()  
    821821            parser_debug("Signature: %s\n", signature);
    822822
    823823            signatureString = signature;
    824             delete signature;
     824            delete[] signature;
    825825        } else if (!memcmp(buffer, "Link", 5)) {
    826826            if (element != P_LINK) {
    827827                RETURN_AND_SET_STATUS(B_ERROR);
    PackageInfo::Parse()  
    837837            parser_debug("Link: %s\n", link);
    838838
    839839            linkString = link;
    840             delete link;
     840            delete[] link;
    841841        } else if (!memcmp(buffer, padding, 7)) {
    842842            PackageItem *item = NULL;
    843843