summaryrefslogtreecommitdiffstats
path: root/libs/binder
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Replace all instances of intptr_t with uintptr_t.Riley Andrews2015-01-131-8/+8
| | | | | | | | | | | | | | | | | | | | | When compiling 32 bit binaries against a 64 bit binder interface, implicit promotions of intptr_t types to uint64_t for fields in the binder ioctl structures can result in invalid pointers because of sign extension.
* | | am 41a160c4: Merge "Fix signed/unsigned comparison warnings in driver ↵Riley Andrews2015-01-131-7/+7
|\ \ \ | |/ / | | | | | | | | | | | | | | | interface test." * commit '41a160c4cda1ef2c916028ec2d0801da738b5249': Fix signed/unsigned comparison warnings in driver interface test.
| * | Fix signed/unsigned comparison warnings in driver interface test.Riley Andrews2015-01-121-7/+7
| | |
* | | am 53186a4d: Merge "Add binder tests"Riley Andrews2014-12-192-1/+960
|\ \ \ | |/ / | | | | | | | | | * commit '53186a4d78479ba21213b6e055c7a85faeb318e2': Add binder tests
| * | Add binder testsRiley Andrews2014-12-192-1/+960
| | | | | | | | | | | | Change-Id: Ib0b38708dd6387fc4d5a4857ca74a217f3edad1a
* | | am dbd252ea: Merge "Add binder driver interface test"Riley Andrews2014-12-192-0/+382
|\ \ \ | |/ / | | | | | | | | | * commit 'dbd252eab09d669a7d4b917890d75408193a85d3': Add binder driver interface test
| * | Add binder driver interface testRiley Andrews2014-12-192-0/+382
| | | | | | | | | | | | | | | | | | Test ioctls and some binder commands without using libbinder. Change-Id: I86dde56ff1380bca17d086021ac5324dc5210354
* | | am 80e81505: Merge "binder: Add {read,write}Uint32 methods to Parcel"Dan Stoza2014-12-011-0/+15
|\ \ \ | |/ / | | | | | | | | | * commit '80e8150549c9d8380dadf942336d41ab3d92bf78': binder: Add {read,write}Uint32 methods to Parcel
| * | binder: Add {read,write}Uint32 methods to ParcelDan Stoza2014-12-011-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | Adds readUint32 and writeUint32 methods to the Parcel class. This saves a lot of static_casting in anything implementing a Binder interface on the native side. Change-Id: Iafc73b0633654a3a4c49767f41806b56906c924f
* | | am 104e1259: Merge "binder: Return pid_t/uid_t instead of int"Dan Stoza2014-12-011-2/+2
|\ \ \ | |/ / | | | | | | | | | * commit '104e12594e085bc8dd32eedb96643b774b1508a0': binder: Return pid_t/uid_t instead of int
| * | binder: Return pid_t/uid_t instead of intDan Stoza2014-12-011-2/+2
| | | | | | | | | | | | | | | | | | | | | Makes IPCThreadState::getCalling{Pid,Uid} return a more appropriate type. Change-Id: I97f8a83c1c0722bc1afbf8a6df1a288f8f1f0d2c
* | | am 2d7ec9d7: Merge "Fix build with clang in C++11 mode"Chih-hung Hsieh2014-11-251-6/+6
|\ \ \ | |/ / | | | | | | | | | * commit '2d7ec9d7b5dfdb93930b94c134fa2dc10fd27313': Fix build with clang in C++11 mode
| * | Fix build with clang in C++11 modeBernhard Rosenkränzer2014-11-251-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we're in C++11 mode by default, clang complains about switch statements with case values not matching the type -- since some binder_driver_return_protocol values are > 0x7fffffff, we need to make the switch statements operate on uint32_t rather than int32_t. BUG: 18466763 Change-Id: Iedbfd5c7a3d3d9f087d2eab4ff21343ad7a2a448 Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
* | | am 2e42a906: Merge "Binder: Remove unused functions & variables"Andreas Gampe2014-11-252-10/+0
|\ \ \ | |/ / | | | | | | | | | * commit '2e42a90653abd0fc8a76909b5f101782d187136e': Binder: Remove unused functions & variables
| * | Binder: Remove unused functions & variablesAndreas Gampe2014-11-242-10/+0
| | | | | | | | | | | | | | | | | | | | | For build-system CFLAGS clean-up, remove unused functions and variables. Change-Id: I88abc2bc99e6b91c7aa56470ff9671bf8bf29f8d
* | | am e0f91c37: Merge "We HAVE_LITTLE_ENDIAN."Elliott Hughes2014-11-221-5/+0
|\ \ \ | |/ / | | | | | | | | | * commit 'e0f91c3731b16b010eb31117e941e5468bdfeb2e': We HAVE_LITTLE_ENDIAN.
| * | We HAVE_LITTLE_ENDIAN.Elliott Hughes2014-11-211-5/+0
| | | | | | | | | | | | Change-Id: I68551ad663f2649a06b1958e9bef152e6d026e72
* | | am c894c204: Merge "C++11 compatibility."Dan Albert2014-11-202-2/+2
|\ \ \ | |/ / | | | | | | | | | * commit 'c894c2045f589571c80af0648e7a9dcd7e97b6a8': C++11 compatibility.
| * | C++11 compatibility.Dan Albert2014-11-202-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | * Explicit conversion for atomic_uintptr_t initialization. * Fix string literal concatenation to not be a UD literal. * Use __typeof__ instead of typeof (should become decltype once this actually moves to C++11). Bug: 18466763 Change-Id: I4eedddfb945a2a703ed27317cb6e2b3041b1ebfc
* | | am 55f71bc0: Merge "Replace IInterface::asBinder() with a static"Marco Nelissen2014-11-175-14/+19
|\ \ \ | |/ / | | / | |/ |/| * commit '55f71bc0800b9ec7a779c5d30d5a90e074c2526b': Replace IInterface::asBinder() with a static
| * Replace IInterface::asBinder() with a staticMarco Nelissen2014-11-175-14/+19
| | | | | | | | | | | | | | so we can do NULL checks again, and update calls to IInterface::asBinder() to use the new static version. Change-Id: Ia7b10eb38ca55b72278bfd33d3bf647f338b4e6a
* | am a4cff88d: Fix issue #18356768: some app process may hang at Runtime#exit...Dianne Hackborn2014-11-142-20/+22
|\ \ | | | | | | | | | | | | * commit 'a4cff88d80bf1acbdbd063af88ecad83a8b9b7b4': Fix issue #18356768: some app process may hang at Runtime#exit...
| * | Fix issue #18356768: some app process may hang at Runtime#exit...Dianne Hackborn2014-11-132-20/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | ...in Parcel destructor Don't use a Mutux object, just use simple posix mutex primitives, to avoid static init/destroy order problems. Change-Id: Ic012d94297564c0a55d58869f8276d7d10545fbc
* | | am ca69e8f2: Merge "Add tracking of parcel memory allocations." into lmp-mr1-devDianne Hackborn2014-11-122-2/+56
|\ \ \ | |/ / | | | | | | | | | * commit 'ca69e8f2d0a1fc963bd170f9b2bf164028761323': Add tracking of parcel memory allocations.
| * | Add tracking of parcel memory allocations.Dianne Hackborn2014-11-112-2/+56
| | | | | | | | | | | | Change-Id: I76ffed0e69ccfe70e87d98646f11d4de7c3fd980
* | | am fee99040: Revert "Add more logging for dup(fd) failure"Jesse Hall2014-11-041-15/+3
|\ \ \ | |/ / | | | | | | | | | * commit 'fee99040de000a64f6036a7db172fb7c68d9764a': Revert "Add more logging for dup(fd) failure"
| * | Revert "Add more logging for dup(fd) failure"Jesse Hall2014-11-041-15/+3
| | | | | | | | | | | | | | | | | | | | | | | | Underlying bug has been fixed, extra debug code no longer needed This reverts commit 36273c991b189c5516819f4d5e3f83061c320593. Change-Id: Ia47f1ae9ee7338329bfb8d710aff57e15e944f20
* | | am 8e5cf83a: Revert "Parcel: extra validation/debug code for ↵Jesse Hall2014-11-041-25/+0
|\ \ \ | |/ / | | | | | | | | | | | | | | | writeDupFileDescriptor" * commit '8e5cf83afbe2b671f96d04786cf0d0f967e342f9': Revert "Parcel: extra validation/debug code for writeDupFileDescriptor"
| * | Revert "Parcel: extra validation/debug code for writeDupFileDescriptor"Jesse Hall2014-11-041-25/+0
| | | | | | | | | | | | | | | | | | | | | | | | Underlying bug has been fixed, extra debug code no longer needed. This reverts commit ccf851f108c008d086b7f3c3923d60d385985e71. Change-Id: I19ad9239efdf1d8f70cd74af9b7c2ae97713208f
* | | am ac5cd96f: am dbab409c: Merge "Avoid two warnings from clang."Chih-Hung Hsieh2014-11-041-13/+11
|\ \ \ | | |/ | |/| | | | | | | * commit 'ac5cd96fe24690d9e0ae823c77334b8579139aa2': Avoid two warnings from clang.
| * | am dbab409c: Merge "Avoid two warnings from clang."Chih-Hung Hsieh2014-11-041-13/+11
| |\ \ | | | | | | | | | | | | | | | | * commit 'dbab409c49ca5dd6686912d0b3eeb19ef1c37a3e': Avoid two warnings from clang.
| | * | Avoid two warnings from clang.Chih-Hung Hsieh2014-10-311-13/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clang compiler warns about undefined static functions like getReturnString before all references are removed inside IF_LOG_COMMANDS. To remove such warnings, we can leave those static functions defined and the compiler will remove them at the end when they are not used. Add a cast to long before converting from int to void*, to avoid type cast warnings on 64 bit targets. BUG: 17043248 Change-Id: Ica31101522279d2c8eacc90e2939767002fb827b
| * | | am 9cc84950: Merge "Don\'t check if this == NULL."Dan Albert2014-10-151-2/+2
| |\ \ \ | | |/ / | | | | | | | | | | | | * commit '9cc849500df0bba42d3ed19b90c38ff1dffd2262': Don't check if this == NULL.
| * | | am 9ea09757: Fix broken error check in Parcel::readBlobNarayan Kamath2014-10-091-1/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | * commit '9ea09757fa63ad2a9bea9c846b51c1210d69c79f': Fix broken error check in Parcel::readBlob
| * \ \ \ am 78639c22: Merge "Parcel: extra validation/debug code for ↵Jesse Hall2014-10-061-0/+25
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | writeDupFileDescriptor" into lmp-dev * commit '78639c22acfa5e56d87fd6fe8340d083536b29fc': Parcel: extra validation/debug code for writeDupFileDescriptor
| * \ \ \ \ am 36462d9d: Merge "Add more logging for dup(fd) failure" into lmp-devMichael Lentine2014-10-021-3/+15
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit '36462d9de00105ae46ee3dbb0f644fa6af599f97': Add more logging for dup(fd) failure
| * \ \ \ \ \ am 5ee2c9d2: Work on issue #17656716: Unhandled exception in Window ManagerDianne Hackborn2014-09-301-8/+13
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit '5ee2c9d20c0fbecf6b4a482eb8d8bfdf85d0d424': Work on issue #17656716: Unhandled exception in Window Manager
* | \ \ \ \ \ \ am 98536859: Merge "Fix output of pointers for 64bit devices." into lmp-mr1-devMichael Lentine2014-10-311-2/+2
|\ \ \ \ \ \ \ \ | | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | * commit '9853685931298ba02066ebb50a8377ad4aa80f00': Fix output of pointers for 64bit devices.
| * | | | | | | Fix output of pointers for 64bit devices.Michael Lentine2014-10-311-2/+2
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | Change-Id: I279c1cf8d4c126b98f4a92ca807ade3749d01ff0
* | | | | | | am 9cc84950: Merge "Don\'t check if this == NULL."Dan Albert2014-10-151-2/+2
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | * commit '9cc849500df0bba42d3ed19b90c38ff1dffd2262': Don't check if this == NULL.
| * | | | | | Merge "Don't check if this == NULL."Dan Albert2014-10-151-2/+2
| |\ \ \ \ \ \
| | * | | | | | Don't check if this == NULL.Dan Albert2014-09-161-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Entering a method with this == NULL is undefined behavior. Clang whines about this. Change-Id: Ibde628395ca10dfef0d2f59e81280576f104b83c
* | | | | | | | am 16410d45: am 9ea09757: Fix broken error check in Parcel::readBlobNarayan Kamath2014-10-101-1/+1
|\ \ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | * commit '16410d458fdf9915dc8cb82b1bae3949a8100124': Fix broken error check in Parcel::readBlob
| * | | | | | | Fix broken error check in Parcel::readBlobNarayan Kamath2014-10-081-1/+1
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mmap returns MAP_FAILED (which is -1) and not NULL on failure. Diagnosed by cferris. bug: 17909809 Change-Id: I609788ebf94742ef88af002d2d3f3bc9b9e520ac
* | | | | | | am 366998d8: am 78639c22: Merge "Parcel: extra validation/debug code for ↵Jesse Hall2014-10-071-0/+25
|\ \ \ \ \ \ \ | |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | writeDupFileDescriptor" into lmp-dev * commit '366998d87109578eb8c33d2e94d61fe7f189beb8': Parcel: extra validation/debug code for writeDupFileDescriptor
| * | | | | | Parcel: extra validation/debug code for writeDupFileDescriptorJesse Hall2014-10-061-0/+25
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Temporary extra debug validation for b/17477219: a Parcel recipient is getting a positive but invalid fd unexpectedly. Trying to track down where it's coming from. Debug code for bug: 17477219 Change-Id: Idb1e71621025a3928c7adc88fd44790e1abd2a01
* | | | | | am 36462d9d: Merge "Add more logging for dup(fd) failure" into lmp-devMichael Lentine2014-10-021-3/+15
|\ \ \ \ \ \ | |/ / / / / | | | | | | | | | | | | | | | | | | * commit '36462d9de00105ae46ee3dbb0f644fa6af599f97': Add more logging for dup(fd) failure
| * | | | | Add more logging for dup(fd) failureMichael Lentine2014-10-021-3/+15
| | |_|_|/ | |/| | | | | | | | | | | | | Bug: 17477219 Change-Id: Ide0ae16d777c9af783023c705c18a93c00999147
* | | | | am 5ee2c9d2: Work on issue #17656716: Unhandled exception in Window ManagerDianne Hackborn2014-10-021-8/+13
|\ \ \ \ \ | |/ / / / | | | | / | |_|_|/ |/| | | * commit '5ee2c9d20c0fbecf6b4a482eb8d8bfdf85d0d424': Work on issue #17656716: Unhandled exception in Window Manager
| * | | Work on issue #17656716: Unhandled exception in Window ManagerDianne Hackborn2014-09-301-8/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Try to clean up the code paths coming in and out of binder IPCs to plug any places where we could disrupt the gather flag of a thread, causing it to keep gathering stack crawls (which is the thing that is causing our strict mode data to become so large). We now take care of saving and restoring this state in the core IPC code path, not at the Java layer. Change-Id: I73d564778da127bdce00f304225930e7f2318293