libbinder: dump dropped oneway transaction data
oneway transaction errors or parcel contents will be dropped. Whenever
there is an error returned from one of these or data is written to the
parcel, we are now logging it here.
Fixes: 148640588
Test: boot, check logs
Change-Id: I96ad47dc39d6deb03805608ee1b21b090332411a
diff --git a/libs/binder/IPCThreadState.cpp b/libs/binder/IPCThreadState.cpp
index 4dcd07a..9e89c57 100644
--- a/libs/binder/IPCThreadState.cpp
+++ b/libs/binder/IPCThreadState.cpp
@@ -1230,6 +1230,11 @@
if (error < NO_ERROR) reply.setError(error);
sendReply(reply, 0);
} else {
+ if (error != OK || reply.dataSize() != 0) {
+ alog << "oneway function results will be dropped but finished with status "
+ << statusToString(error)
+ << " and parcel size " << reply.dataSize() << endl;
+ }
LOG_ONEWAY("NOT sending reply to %d!", mCallingPid);
}