diff --git a/libs/androidfw/ApkAssets.cpp b/libs/androidfw/ApkAssets.cpp old mode 100644 new mode 100755 index 05f4d6b63a4cc..e15b42d46f536 --- a/libs/androidfw/ApkAssets.cpp +++ b/libs/androidfw/ApkAssets.cpp @@ -496,6 +496,11 @@ std::unique_ptr ApkAssets::LoadImpl( const StringPiece data( reinterpret_cast(loaded_apk->resources_asset_->getBuffer(true /*wordAligned*/)), loaded_apk->resources_asset_->getLength()); + if (data.data() == nullptr || data.empty()) { + LOG(ERROR) << "Failed to read '" << kResourcesArsc << "' data in APK '" << path << "'."; + return {}; + } + loaded_apk->loaded_arsc_ = LoadedArsc::Load(data, loaded_apk->loaded_idmap_.get(), property_flags); if (!loaded_apk->loaded_arsc_) { @@ -523,9 +528,14 @@ std::unique_ptr ApkAssets::LoadTableImpl( const StringPiece data( reinterpret_cast(loaded_apk->resources_asset_->getBuffer(true /*wordAligned*/)), loaded_apk->resources_asset_->getLength()); + if (data.data() == nullptr || data.empty()) { + LOG(ERROR) << "Failed to read resources table data in '" << path << "'."; + return {}; + } + loaded_apk->loaded_arsc_ = LoadedArsc::Load(data, nullptr, property_flags); if (loaded_apk->loaded_arsc_ == nullptr) { - LOG(ERROR) << "Failed to load '" << kResourcesArsc << path; + LOG(ERROR) << "Failed to read resources table in '" << path << "'."; return {}; }