diff options
author | Roshan Pius <rpius@google.com> | 2015-07-10 16:54:42 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2015-07-10 16:54:43 +0000 |
commit | 1cb0de17a9a4de137247f7bf69ec44a83e30efa2 (patch) | |
tree | 865d94315d03a1d5d70197f92c8dc6d5678bcf37 /telecomm | |
parent | 3b9ce37b51dcc7729489701d4e797fe91f2ef559 (diff) | |
parent | 75c36b681e3331f4e60fa62abe4615c1ffc5b401 (diff) | |
download | frameworks_base-1cb0de17a9a4de137247f7bf69ec44a83e30efa2.zip frameworks_base-1cb0de17a9a4de137247f7bf69ec44a83e30efa2.tar.gz frameworks_base-1cb0de17a9a4de137247f7bf69ec44a83e30efa2.tar.bz2 |
Merge "Correct the comparison done in removeAdapter." into mnc-dev
Diffstat (limited to 'telecomm')
-rw-r--r-- | telecomm/java/android/telecom/ConnectionServiceAdapter.java | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/telecomm/java/android/telecom/ConnectionServiceAdapter.java b/telecomm/java/android/telecom/ConnectionServiceAdapter.java index 4ab9ee5..4562514 100644 --- a/telecomm/java/android/telecom/ConnectionServiceAdapter.java +++ b/telecomm/java/android/telecom/ConnectionServiceAdapter.java @@ -48,6 +48,12 @@ final class ConnectionServiceAdapter implements DeathRecipient { } void addAdapter(IConnectionServiceAdapter adapter) { + for (IConnectionServiceAdapter it : mAdapters) { + if (it.asBinder() == adapter.asBinder()) { + Log.w(this, "Ignoring duplicate adapter addition."); + return; + } + } if (mAdapters.add(adapter)) { try { adapter.asBinder().linkToDeath(this, 0); @@ -58,8 +64,13 @@ final class ConnectionServiceAdapter implements DeathRecipient { } void removeAdapter(IConnectionServiceAdapter adapter) { - if (adapter != null && mAdapters.remove(adapter)) { - adapter.asBinder().unlinkToDeath(this, 0); + if (adapter != null) { + for (IConnectionServiceAdapter it : mAdapters) { + if (it.asBinder() == adapter.asBinder() && mAdapters.remove(it)) { + adapter.asBinder().unlinkToDeath(this, 0); + break; + } + } } } |