diff options
| author | 2023-03-02 05:23:55 +0000 | |
|---|---|---|
| committer | 2023-03-02 05:23:55 +0000 | |
| commit | bc50ae3f16d91f639a49fe8cc2de9cb5380af9dd (patch) | |
| tree | 915dcc6260d2aef601da912e28f8667636309706 | |
| parent | 707cbc9d4375581e9c9dad749d6748a1eced36d6 (diff) | |
| parent | 7a1eccb8a07f891083f101ef52bed006128b006f (diff) | |
Merge "binderLibTest: calling freeze multiple times starts to succeed" am: bc2c1e87a4 am: c6abed315f am: 7a1eccb8a0
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2433913
Change-Id: Ifc611e88ccbc352ce951b11d24c963206a82c517
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | libs/binder/tests/binderLibTest.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/libs/binder/tests/binderLibTest.cpp b/libs/binder/tests/binderLibTest.cpp index 955c650205..09bff74231 100644 --- a/libs/binder/tests/binderLibTest.cpp +++ b/libs/binder/tests/binderLibTest.cpp @@ -507,7 +507,13 @@ TEST_F(BinderLibTest, Freeze) { } EXPECT_EQ(-EAGAIN, IPCThreadState::self()->freeze(pid, true, 0)); - EXPECT_EQ(-EAGAIN, IPCThreadState::self()->freeze(pid, true, 0)); + + // b/268232063 - succeeds ~0.08% of the time + { + auto ret = IPCThreadState::self()->freeze(pid, true, 0); + EXPECT_TRUE(ret == -EAGAIN || ret == OK); + } + EXPECT_EQ(NO_ERROR, IPCThreadState::self()->freeze(pid, true, 1000)); EXPECT_EQ(FAILED_TRANSACTION, m_server->transact(BINDER_LIB_TEST_NOP_TRANSACTION, data, &reply)); |