Modify Dangerous permission state pullers to collect all permissions.

Test: statsd_testdrive
Bug: 154373877
Change-Id: If9c2ab0d3b0b90e325e4423f0647b8f3e9a0868f
This commit is contained in:
Stanislav Zholnin
2020-12-04 16:52:15 +00:00
parent 7db76b6723
commit 4e66025888

View File

@@ -605,6 +605,7 @@ public class StatsPullAtomService extends SystemService {
return pullRoleHolderLocked(atomTag, data);
}
case FrameworkStatsLog.DANGEROUS_PERMISSION_STATE:
// fall-through - same call covers two cases
case FrameworkStatsLog.DANGEROUS_PERMISSION_STATE_SAMPLED:
synchronized (mDangerousPermissionStateLock) {
return pullDangerousPermissionStateLocked(atomTag, data);
@@ -2913,7 +2914,7 @@ public class StatsPullAtomService extends SystemService {
}
int numPerms = pkg.requestedPermissions.length;
for (int permNum = 0; permNum < numPerms; permNum++) {
for (int permNum = 0; permNum < numPerms; permNum++) {
String permName = pkg.requestedPermissions[permNum];
PermissionInfo permissionInfo;
@@ -2926,10 +2927,6 @@ public class StatsPullAtomService extends SystemService {
continue;
}
if (permissionInfo.getProtection() != PROTECTION_DANGEROUS) {
continue;
}
if (permName.startsWith(COMMON_PERMISSION_PREFIX)) {
permName = permName.substring(COMMON_PERMISSION_PREFIX.length());
}
@@ -2941,15 +2938,17 @@ public class StatsPullAtomService extends SystemService {
(pkg.requestedPermissionsFlags[permNum]
& REQUESTED_PERMISSION_GRANTED)
!= 0,
permissionFlags);
permissionFlags, permissionInfo.getProtection()
| permissionInfo.getProtectionFlags());
} else {
// DangerousPermissionStateSampled atom.
// DangeorusPermissionStateSampled atom.
e = FrameworkStatsLog.buildStatsEvent(atomTag, permName,
pkg.applicationInfo.uid,
(pkg.requestedPermissionsFlags[permNum]
& REQUESTED_PERMISSION_GRANTED)
!= 0,
permissionFlags);
permissionFlags, permissionInfo.getProtection()
| permissionInfo.getProtectionFlags());
}
pulledData.add(e);
}