diff options
| -rw-r--r-- | telephony/java/android/telephony/ims/feature/MmTelFeature.java | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/telephony/java/android/telephony/ims/feature/MmTelFeature.java b/telephony/java/android/telephony/ims/feature/MmTelFeature.java index c073d1ab03d6..aaf1a1cf8f20 100644 --- a/telephony/java/android/telephony/ims/feature/MmTelFeature.java +++ b/telephony/java/android/telephony/ims/feature/MmTelFeature.java @@ -139,34 +139,44 @@ public class MmTelFeature extends ImsFeature { @Override public int queryCapabilityStatus() throws RemoteException { - return MmTelFeature.this.queryCapabilityStatus().mCapabilities; + synchronized (mLock) { + return MmTelFeature.this.queryCapabilityStatus().mCapabilities; + } } @Override public void addCapabilityCallback(IImsCapabilityCallback c) { + // no need to lock, structure already handles multithreading. MmTelFeature.this.addCapabilityCallback(c); } @Override public void removeCapabilityCallback(IImsCapabilityCallback c) { + // no need to lock, structure already handles multithreading. MmTelFeature.this.removeCapabilityCallback(c); } @Override public void changeCapabilitiesConfiguration(CapabilityChangeRequest request, IImsCapabilityCallback c) throws RemoteException { - MmTelFeature.this.requestChangeEnabledCapabilities(request, c); + synchronized (mLock) { + MmTelFeature.this.requestChangeEnabledCapabilities(request, c); + } } @Override public void queryCapabilityConfiguration(int capability, int radioTech, IImsCapabilityCallback c) { - queryCapabilityConfigurationInternal(capability, radioTech, c); + synchronized (mLock) { + queryCapabilityConfigurationInternal(capability, radioTech, c); + } } @Override public void setSmsListener(IImsSmsListener l) throws RemoteException { - MmTelFeature.this.setSmsListener(l); + synchronized (mLock) { + MmTelFeature.this.setSmsListener(l); + } } @Override |