diff options
| author | 2022-10-04 18:20:00 +0000 | |
|---|---|---|
| committer | 2022-10-04 18:20:00 +0000 | |
| commit | c2bb53032f64d11ce4f859a2c8ec65445804d07b (patch) | |
| tree | 7754fefac03769e754237972ef4a5ac579c30e57 | |
| parent | e22a33b566d516a8858669cf9867538f01718ef4 (diff) | |
| parent | 9df841c82c8a36ba245f8264a8e492830d4820f2 (diff) | |
Merge "binderRpcTest: actually include strong count in leaked ref" am: 9df841c82c
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2237852
Change-Id: Ie5129b3d99210163fcd321456ac70c3a350ff59b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | libs/binder/tests/binderRpcTest.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp index 7a550f7e1a..652da991a4 100644 --- a/libs/binder/tests/binderRpcTest.cpp +++ b/libs/binder/tests/binderRpcTest.cpp @@ -181,9 +181,10 @@ struct ProcessSession { wp<RpcSession> weakSession = session; session = nullptr; - EXPECT_EQ(nullptr, weakSession.promote()) + sp<RpcSession> strongSession = weakSession.promote(); + EXPECT_EQ(nullptr, strongSession) << (debugBacktrace(host.getPid()), debugBacktrace(getpid()), "Leaked sess: ") - << session->getStrongCount(); + << strongSession->getStrongCount(); } } }; |