commit c360a109193590d2e46c4c9b6ab18ca6851f0172
author Steven Moreland <smoreland@google.com> Wed Mar 06 15:02:23 2019 -0800
committer Steven Moreland <smoreland@google.com> Wed Mar 06 15:19:34 2019 -0800
tree c57984a0c6273b29b268344854536706e2a29c78
parent 304f9d37b1baa40024090d0de47a078a01768fb2

libbinder_ndk: Fix 'isRemote' value for Bn*.

Bn* is not remote (it's local).

This wasn't noticed before since the AIDL generated method 'associate'
always wrapped a proxy, and since a wrapped Bn object will always have
a null remoteBinder, it would properly show that it was local even
though it was wrapped.

Bug: 127336441
Test: atest android.binder.cts
Change-Id: Ic4d96822a355e88bc8f3dbbc8fa602835653139e

libs/binder/ndk/include_ndk/android/binder_interface_utils.h

diff --git a/libs/binder/ndk/include_ndk/android/binder_interface_utils.h b/libs/binder/ndk/include_ndk/android/binder_interface_utils.h
index fe92984..83a1048 100644
--- a/libs/binder/ndk/include_ndk/android/binder_interface_utils.h
+++ b/libs/binder/ndk/include_ndk/android/binder_interface_utils.h
@@ -150,7 +150,7 @@
 
     SpAIBinder asBinder() override;
 
-    bool isRemote() override { return true; }
+    bool isRemote() override { return false; }
 
    protected:
     /**