Merge changes from topic "am-1ab9514f-fd12-4093-b1cb-a77e9f0419c9" into oc-dev
* changes: [automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41am:a3f0976937am:166aa6e149am:b7f7f7e14eam:c26b8635b7am:afac0f46d9am:5786491706[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41am:a3f0976937am:166aa6e149am:b7f7f7e14eam:c26b8635b7am:afac0f46d9[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41am:a3f0976937am:166aa6e149am:b7f7f7e14eam:c26b8635b7[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41am:a3f0976937am:166aa6e149am:b7f7f7e14e[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41am:a3f0976937am:166aa6e149[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41am:a3f0976937[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682am:0ab4540c41[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99am:e2417e6682[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909am:c3d7250b99[automerger] Check for null-terminator in ResStringPool::string8At am:5ec65ae909Check for null-terminator in ResStringPool::string8At
This commit is contained in:
committed by
Android (Google) Code Review
commit
46fc48f57e
@@ -813,7 +813,13 @@ const char* ResStringPool::string8At(size_t idx, size_t* outLen) const
|
||||
*outLen = encLen;
|
||||
|
||||
if ((uint32_t)(str+encLen-strings) < mStringPoolSize) {
|
||||
return (const char*)str;
|
||||
// Reject malformed (non null-terminated) strings
|
||||
if (str[encLen] != 0x00) {
|
||||
ALOGW("Bad string block: string #%d is not null-terminated",
|
||||
(int)idx);
|
||||
return NULL;
|
||||
}
|
||||
return (const char*)str;
|
||||
} else {
|
||||
ALOGW("Bad string block: string #%d extends to %d, past end at %d\n",
|
||||
(int)idx, (int)(str+encLen-strings), (int)mStringPoolSize);
|
||||
|
||||
Reference in New Issue
Block a user