commit 90a78649f2ec991baf4f29ecabe7b772eb464850
author Les Lee <lesl@google.com> Tue Jun 29 23:46:02 2021 +0800
committer Les Lee <lesl@google.com> Wed Jun 30 23:56:05 2021 +0800
tree d7c25d19543370b38a9aa713ec3d5879c03b0206
parent 5ded797ef4b81a39e0ed2b66a4974c066a402390

wifi: Fix the Security Report for pending intent

Bug: 170644642
Test: Manual Test, it work normally.
Change-Id: If24e7232f0740c8b43cedd56be7d54fcb3cba5a3

packages/SettingsProvider/src/com/android/providers/settings/WifiSoftApConfigChangedNotifier.java

diff --git a/packages/SettingsProvider/src/com/android/providers/settings/WifiSoftApConfigChangedNotifier.java b/packages/SettingsProvider/src/com/android/providers/settings/WifiSoftApConfigChangedNotifier.java
index c33f02df..dc51c40 100644
--- a/packages/SettingsProvider/src/com/android/providers/settings/WifiSoftApConfigChangedNotifier.java
+++ b/packages/SettingsProvider/src/com/android/providers/settings/WifiSoftApConfigChangedNotifier.java
@@ -15,17 +15,24 @@
  */
 package com.android.providers.settings;
 
+import android.app.ActivityManager;
 import android.app.Notification;
 import android.app.NotificationChannel;
 import android.app.NotificationManager;
 import android.app.PendingIntent;
 import android.content.Context;
 import android.content.Intent;
+import android.content.pm.PackageManager;
+import android.content.pm.ResolveInfo;
 import android.content.res.Resources;
+import android.os.UserHandle;
+import android.provider.Settings;
 
 import com.android.internal.messages.nano.SystemMessageProto;
 import com.android.internal.notification.SystemNotificationChannels;
 
+import java.util.List;
+
 /**
  * Helper class for sending notifications when the user's Soft AP config was changed upon restore.
  */
@@ -81,8 +88,25 @@
 
     private static PendingIntent getPendingActivity(Context context) {
         Intent intent = new Intent("com.android.settings.WIFI_TETHER_SETTINGS")
-                .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
+                .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
+                .setPackage(getSettingsPackageName(context));
         return PendingIntent.getActivity(context, 0, intent,
                 PendingIntent.FLAG_UPDATE_CURRENT | PendingIntent.FLAG_IMMUTABLE);
     }
+
+    /**
+     * @return Get settings package name.
+     */
+    private static String getSettingsPackageName(Context context) {
+        if (context == null) return null;
+
+        Intent intent = new Intent(Settings.ACTION_WIFI_SETTINGS);
+        List<ResolveInfo> resolveInfos = context.getPackageManager().queryIntentActivitiesAsUser(
+                intent, PackageManager.MATCH_SYSTEM_ONLY | PackageManager.MATCH_DEFAULT_ONLY,
+                UserHandle.of(ActivityManager.getCurrentUser()));
+        if (resolveInfos == null || resolveInfos.isEmpty()) {
+            return "com.android.settings";
+        }
+        return resolveInfos.get(0).activityInfo.packageName;
+    }
 }