From f4af1f713a8025ca59d00967d4302290d1b1c299 Mon Sep 17 00:00:00 2001 From: Jack Yu Date: Thu, 22 Dec 2022 03:23:32 -0800 Subject: [PATCH] Added permission check for notifying subscription changed Added permission check for notifying subscription changed event. This is same as the reverted ag/20813431. Bug: 239607619 Test: Manual Change-Id: I0103d62a5adcfb4f5bdcd05bb236029dd04acdc8 --- .../core/java/com/android/server/TelephonyRegistry.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/services/core/java/com/android/server/TelephonyRegistry.java b/services/core/java/com/android/server/TelephonyRegistry.java index 9bedbd0ec5845..7b8ca912ecf2b 100644 --- a/services/core/java/com/android/server/TelephonyRegistry.java +++ b/services/core/java/com/android/server/TelephonyRegistry.java @@ -1000,6 +1000,10 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub { @Override public void notifySubscriptionInfoChanged() { if (VDBG) log("notifySubscriptionInfoChanged:"); + if (!checkNotifyPermission("notifySubscriptionInfoChanged()")) { + return; + } + synchronized (mRecords) { if (!mHasNotifySubscriptionInfoChangedOccurred) { log("notifySubscriptionInfoChanged: first invocation mRecords.size=" @@ -1026,6 +1030,10 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub { @Override public void notifyOpportunisticSubscriptionInfoChanged() { if (VDBG) log("notifyOpptSubscriptionInfoChanged:"); + if (!checkNotifyPermission("notifyOpportunisticSubscriptionInfoChanged()")) { + return; + } + synchronized (mRecords) { if (!mHasNotifyOpportunisticSubscriptionInfoChangedOccurred) { log("notifyOpptSubscriptionInfoChanged: first invocation mRecords.size="