Made broadcast EXTERNAL_PROVIDER_CHANGE protected and with FLAG_RECEIVER_REPLACE_PENDING.
android.provider.action.EXTERNAL_PROVIDER_CHANGE should be protected
since it's being sent by system app (TelephonyProvider).
android.provider.action.EXTERNAL_PROVIDER_CHANGE broadcast in notifyIfNotDefaultSmsApp()
should be using the FLAG_RECEIVER_REPLACE_PENDING flag on its
Intent, so that it's never be stacked up multiple copies of it in the queue.
Bug: 27341575
Bug: 27429156
Change-Id: I980d38bd2dbaa3888cfe0d6a6cfe0e4d991f2431
diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index 06a331c..88acca2 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -24,6 +24,7 @@
<uses-permission android:name="android.permission.INTERACT_ACROSS_USERS" />
<uses-permission android:name="android.permission.INTERACT_ACROSS_USERS_FULL" />
+ <protected-broadcast android:name="android.provider.action.EXTERNAL_PROVIDER_CHANGE" />
<!-- This permission is only used to send the ACTION_EXTERNAL_PROVIDER_CHANGE intent. -->
<uses-permission android:name="android.permission.MODIFY_PHONE_STATE" />
diff --git a/src/com/android/providers/telephony/ProviderUtil.java b/src/com/android/providers/telephony/ProviderUtil.java
index 7f5a2bd..7053e9a 100644
--- a/src/com/android/providers/telephony/ProviderUtil.java
+++ b/src/com/android/providers/telephony/ProviderUtil.java
@@ -101,6 +101,7 @@
final Intent intent =
new Intent(Telephony.Sms.Intents.ACTION_EXTERNAL_PROVIDER_CHANGE);
+ intent.setFlags(Intent.FLAG_RECEIVER_REPLACE_PENDING);
intent.setComponent(componentName);
if (uri != null) {
intent.setData(uri);