drgreen/android_device_google_gs-common
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

powerstats: fix buffer underflow issue in CPUPM data provider

Browse Source 
Bug: 267827563
Test: dumpsys android.hardware.power.stats.IPowerStats/default
Change-Id: I569a20f250c7ca3586f71084918022f04d6693d4
Signed-off-by: Darren Hsu <darrenhsu@google.com>
This commit is contained in:
Darren Hsu 2023-02-07 10:45:26 +08:00
parent d865bd3bf9
commit d094ba620f
1 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
powerstats/CpupmStateResidencyDataProvider.cpp
View File

@ -100,19 +100,21 @@ bool CpupmStateResidencyDataProvider::getStateResidencies(
stateId = temp;
}
if (stateId >= 0) {
entityIndex = matchEntity(line);
it = residencies->find(mConfig.entities[entityIndex].first);
if (stateId < 0) continue;
if (it != residencies->end()) {
if (parseState(line, &duration, &count)) {
it->second[stateId].totalTimeInStateMs = duration / US_TO_MS;
it->second[stateId].totalStateEntryCount = count;
} else {
LOG(ERROR) << "Failed to parse duration and count from [" << std::string(line)
<< "]";
return false;
}
entityIndex = matchEntity(line);
if (entityIndex < 0) continue;
it = residencies->find(mConfig.entities[entityIndex].first);
if (it != residencies->end()) {
if (parseState(line, &duration, &count)) {
it->second[stateId].totalTimeInStateMs = duration / US_TO_MS;
it->second[stateId].totalStateEntryCount = count;
} else {
LOG(ERROR) << "Failed to parse duration and count from [" << std::string(line)
<< "]";
return false;
}
}
}