[automerger] Fix DynamicRefTable::load security bug am: 8cf0f988b0 am: d65dbf91ce am: aeb2fc64d9 am: 0938689606 am: 6f76fbfc69
Change-Id: I43c9c68ca69d26f6d4d2dee3cd2a66d5e4a73761
This commit is contained in:
@@ -6415,8 +6415,16 @@ status_t ResTable::parsePackage(const ResTable_package* const pkg,
|
||||
}
|
||||
|
||||
} else if (ctype == RES_TABLE_LIBRARY_TYPE) {
|
||||
|
||||
if (group->dynamicRefTable.entries().size() == 0) {
|
||||
status_t err = group->dynamicRefTable.load((const ResTable_lib_header*) chunk);
|
||||
const ResTable_lib_header* lib = (const ResTable_lib_header*) chunk;
|
||||
status_t err = validate_chunk(&lib->header, sizeof(*lib),
|
||||
endPos, "ResTable_lib_header");
|
||||
if (err != NO_ERROR) {
|
||||
return (mError=err);
|
||||
}
|
||||
|
||||
err = group->dynamicRefTable.load(lib);
|
||||
if (err != NO_ERROR) {
|
||||
return (mError=err);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user