diff options
author | Carlos Llamas <cmllamas@google.com> | 2024-02-03 00:49:00 +0000 |
---|---|---|
committer | Carlos Llamas <cmllamas@google.com> | 2024-02-03 15:40:38 +0000 |
commit | ecdacd1e9a5aa894c522e7143567c3700eca6080 (patch) | |
tree | 61a792f29b337df30160939e727e3a95bb4c7809 | |
parent | 7aa48af991da5e3264d10c93df46bd4a6b9527f6 (diff) | |
download | libhwbinder-ecdacd1e9a5aa894c522e7143567c3700eca6080.tar.gz |
Revert "Log a warning when libhwbinder gets BR_TRANSACTION_PENDING_FROZEN"
This reverts commit 034e8fe57c33c5f7f6d5b228e96641f2733b06ae.
Handling BR_TRANSACTION_PENDING_FROZEN in executeCommand() can lead to
oneway calls getting stuck. A temporary solution to handle this response
in waitForResponse() is implemented in a subsequent commit.
Bug: 319168480
Change-Id: I35d2f45f2dbd71c0f6ea9584cce866f1061264dd
Signed-off-by: Carlos Llamas <cmllamas@google.com>
-rw-r--r-- | IPCThreadState.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/IPCThreadState.cpp b/IPCThreadState.cpp index 6ae379f..2084a81 100644 --- a/IPCThreadState.cpp +++ b/IPCThreadState.cpp @@ -90,7 +90,6 @@ static const char *kReturnStrings[] = { "BR_FAILED_REPLY", "BR_FROZEN_REPLY", "BR_ONEWAY_SPAM_SUSPECT", - "BR_TRANSACTION_PENDING_FROZEN", }; static const char *kCommandStrings[] = { @@ -1270,10 +1269,6 @@ status_t IPCThreadState::executeCommand(int32_t cmd) mProcess->spawnPooledThread(false); break; - case BR_TRANSACTION_PENDING_FROZEN: - ALOGW("Sending oneway calls to frozen process."); - break; - default: ALOGE("*** BAD COMMAND %d received from Binder driver\n", cmd); result = UNKNOWN_ERROR; |