Skip to content

Commit acc353d

Browse files
Steven Morelandandroid-build-merge-worker-robot
authored andcommitted
Merge "libbinder: also avaid sWarningCallback lock" into main am: a26652b
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3308937 Change-Id: Ibb2f43891fe19983a64f2a02bd7b5907ce7295e6 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2 parents b9f3ceb + a26652b commit acc353d

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

libs/binder/BpBinder.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,9 @@ sp<BpBinder> BpBinder::create(int32_t handle, std::function<void()>* postTask) {
197197
&& currentValue < sBinderProxyCountHighWatermark
198198
&& ((trackedValue & WARNING_REACHED_MASK) == 0)) [[unlikely]] {
199199
sTrackingMap[trackedUid] |= WARNING_REACHED_MASK;
200-
if (sWarningCallback) sWarningCallback(trackedUid);
200+
if (sWarningCallback) {
201+
*postTask = [=]() { sWarningCallback(trackedUid); };
202+
}
201203
} else if (currentValue >= sBinderProxyCountHighWatermark) {
202204
ALOGE("Too many binder proxy objects sent to uid %d from uid %d (%d proxies held)",
203205
getuid(), trackedUid, trackedValue);

0 commit comments

Comments
 (0)