Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

am 71dfc7e9: Merge "Memory leak in parsePackage"

Browse Source 
* commit '71dfc7e9accc1ca3cda52bdf475b03fa2a334e78':
  Memory leak in parsePackage
This commit is contained in:
Narayan Kamath
2014-06-24 14:21:14 +00:00
committed by Android Git Automerger
parent 22a32d8837 71dfc7e9ac
commit b20b0629c2
1 changed files with 16 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
libs/androidfw/ResourceTypes.cpp
View File

@@ -5405,23 +5405,33 @@ status_t ResTable::parsePackage(const ResTable_package* const pkg,
return (mError=err);
}
//printf("Adding new package id %d at index %d\n", id, idx);
err = mPackageGroups.add(group);
err = group->packages.add(package);
if (err < NO_ERROR) {
delete group;
delete package;
return (mError=err);
}
group->basePackage = package;
//printf("Adding new package id %d at index %d\n", id, idx);
err = mPackageGroups.add(group);
if (err < NO_ERROR) {
delete group;
return (mError=err);
}
mPackageMap[id] = (uint8_t)idx;
} else {
group = mPackageGroups.itemAt(idx-1);
if (group == NULL) {
delete package;
return (mError=UNKNOWN_ERROR);
}
}
err = group->packages.add(package);
if (err < NO_ERROR) {
return (mError=err);
err = group->packages.add(package);
if (err < NO_ERROR) {
delete package;
return (mError=err);
}
}
} else {
LOG_ALWAYS_FATAL("Package id out of range");