| commit | 2faf3adc2580829ef44ffd44f2f19cf94f10668f | [log] [tgz] |
|---|---|---|
| author | Yifan Hong <elsk@google.com> | Fri Oct 29 21:26:06 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Fri Oct 29 21:26:06 2021 +0000 |
| tree | f912c8e927904e6428196b58559645321055a1f5 | |
| parent | 75f172861783d7fa5e96724ba4b6dac292c25fc5 [diff] | |
| parent | e1bc376ed44f07ef5d3889a8c75eecefb4ba6369 [diff] |
Merge "health impl: compare interfaces with their binders" am: 1f5c57c965 am: e1bc376ed4 Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1875175 Change-Id: Ia245dfb5e5a8511b3a2b98b08745225b03b63044
diff --git a/health/aidl/default/Health.cpp b/health/aidl/default/Health.cpp index 2d91ce0..812e64a 100644 --- a/health/aidl/default/Health.cpp +++ b/health/aidl/default/Health.cpp
@@ -261,7 +261,7 @@ std::lock_guard<decltype(callbacks_lock_)> lock(callbacks_lock_); auto matches = [callback](const auto& linked) { - return linked->callback() == callback; // compares shared_ptr + return linked->callback()->asBinder() == callback->asBinder(); // compares binder object }; auto it = std::remove_if(callbacks_.begin(), callbacks_.end(), matches); bool removed = (it != callbacks_.end());