commit 6e69d65a3c575969e0461558d2496c3b2d877871
author Steven Moreland <smoreland@google.com> Wed Jul 10 14:17:55 2019 -0700
committer Steven Moreland <smoreland@google.com> Fri Jul 12 20:40:41 2019 +0000
tree d17961c1f394c709a1d8e733bfd904f3533fe0b4
parent add824adfe02d6c2e710d5f94ea5597f165ee1fa

libbinder: empty ping transaction

It came up recently that the libbinder ping transaction uses 4-bytes in
the reply header, but it really only needs to use the binder return
code (how servicemanager did it before when written in C).

Fixes: 137203409
Test: TH
Change-Id: I4555fdba0f80263caa3f3727ef5c5458d160a585

libs/binder/Binder.cpp

diff --git a/libs/binder/Binder.cpp b/libs/binder/Binder.cpp
index 96ee295..1729d6a 100644
--- a/libs/binder/Binder.cpp
+++ b/libs/binder/Binder.cpp
@@ -128,7 +128,7 @@
     status_t err = NO_ERROR;
     switch (code) {
         case PING_TRANSACTION:
-            reply->writeInt32(pingBinder());
+            err = pingBinder();
             break;
         default:
             err = onTransact(code, data, reply, flags);