Merge "Clear binder identity in callback of IMS IPC interfaces" (67e10842) · Commits · e / os / android_frameworks_base

telephony/java/android/telephony/TelephonyManager.java

+6 −1

Original line number	Diff line number	Diff line
		@@ -13425,7 +13425,12 @@ public class TelephonyManager {
		IBooleanConsumer aidlConsumer = callback == null ? null : new IBooleanConsumer.Stub() {
		@Override
		public void accept(boolean result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> callback.accept(result));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		};

telephony/java/android/telephony/ims/ImsMmTelManager.java

+24 −4

Original line number	Diff line number	Diff line
		@@ -444,7 +444,12 @@ public class ImsMmTelManager implements RegistrationManager {
		iTelephony.getImsMmTelRegistrationState(mSubId, new IIntegerConsumer.Stub() {
		@Override
		public void accept(int result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> stateCallback.accept(result));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		});
		} catch (RemoteException e) {
		@@ -485,7 +490,12 @@ public class ImsMmTelManager implements RegistrationManager {
		new IIntegerConsumer.Stub() {
		@Override
		public void accept(int result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> transportTypeCallback.accept(result));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		});
		} catch (RemoteException e) {
		@@ -802,7 +812,12 @@ public class ImsMmTelManager implements RegistrationManager {
		iTelephony.isMmTelCapabilitySupported(mSubId, new IIntegerConsumer.Stub() {
		@Override
		public void accept(int result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> callback.accept(result == 1));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		}, capability, transportType);
		} catch (ServiceSpecificException sse) {
		@@ -1348,7 +1363,12 @@ public class ImsMmTelManager implements RegistrationManager {
		iTelephony.getImsMmTelFeatureState(mSubId, new IIntegerConsumer.Stub() {
		@Override
		public void accept(int result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> callback.accept(result));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		});
		} catch (ServiceSpecificException sse) {

telephony/java/android/telephony/ims/ImsRcsManager.java

+12 −2

Original line number	Diff line number	Diff line
		@@ -299,7 +299,12 @@ public class ImsRcsManager {
		imsRcsController.getImsRcsRegistrationState(mSubId, new IIntegerConsumer.Stub() {
		@Override
		public void accept(int result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> stateCallback.accept(result));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		});
		} catch (RemoteException e) {
		@@ -343,7 +348,12 @@ public class ImsRcsManager {
		new IIntegerConsumer.Stub() {
		@Override
		public void accept(int result) {
		final long identity = Binder.clearCallingIdentity();
		try {
		executor.execute(() -> transportTypeCallback.accept(result));
		} finally {
		Binder.restoreCallingIdentity(identity);
		}
		}
		});
		} catch (RemoteException e) {