Merge changes Ia706a503,I04f245ac

* changes:
  Fix potential null package name
  Add log when denying client authorization state
This commit is contained in:
TreeHugger Robot
2022-07-06 16:24:11 +00:00
committed by Android (Google) Code Review
2 changed files with 14 additions and 0 deletions

View File

@@ -322,6 +322,17 @@ public class ContextHubClientBroker extends IContextHubClient.Stub
} else {
mPendingIntentRequest = new PendingIntentRequest(pendingIntent, nanoAppId);
}
if (packageName == null) {
String[] packages = mContext.getPackageManager().getPackagesForUid(
Binder.getCallingUid());
if (packages != null && packages.length > 0) {
packageName = packages[0];
}
Log.e(TAG, "createClient: Provided package name null. Using first package name "
+ packageName);
}
mPackage = packageName;
mAttributionTag = attributionTag;
mTransactionManager = transactionManager;

View File

@@ -1042,6 +1042,9 @@ public class ContextHubService extends IContextHubService.Stub {
}
/* package */ void denyClientAuthState(int contextHubId, String packageName, long nanoAppId) {
Log.i(TAG, "Denying " + packageName + " access to " + Long.toHexString(nanoAppId)
+ " on context hub # " + contextHubId);
mClientManager.forEachClientOfHub(contextHubId, client -> {
if (client.getPackageName().equals(packageName)) {
client.updateNanoAppAuthState(