commit a124bb85f23586b29928f120e0c07838c5cd2dcb
author Steven Moreland <smoreland@google.com> Fri Sep 07 04:34:00 2018 -0700
committer android-build-merger <android-build-merger@google.com> Fri Sep 07 04:34:00 2018 -0700
tree 9e91c9d7702e454fa4b3837accc35803301da769
parent 981ce23106171c6464ad8cae8fb9de1ca4b085a7
parent 0f20c9f948fd337b361866be6884204eb4dee5a6

Merge "libbinder_ndk: warning for transact on local objs" am: 7ee24369cd am: 923360795d
am: 0f20c9f948

Change-Id: I907394a4cf9cfd73bb8aae0a0464815a49e46863

libs/binder/ndk/AIBinder.cpp

diff --git a/libs/binder/ndk/AIBinder.cpp b/libs/binder/ndk/AIBinder.cpp
index 83972f7..b7f5636 100644
--- a/libs/binder/ndk/AIBinder.cpp
+++ b/libs/binder/ndk/AIBinder.cpp
@@ -285,6 +285,10 @@
         return EX_ILLEGAL_STATE;
     }
 
+    if (!binder->isRemote()) {
+        LOG(WARNING) << "A binder object at " << binder << " is being transacted on, however, this object is in the same process as its proxy. Transacting with this binder is expensive compared to just calling the corresponding functionality in the same process.";
+    }
+
     *in = new AParcel(binder);
     binder_status_t status = (**in)->writeInterfaceToken(clazz->getInterfaceDescriptor());
     if (status != EX_NONE) {