| e9e9fe40fb702ab1390a32c61981bc927177258a |
|
01-Mar-2017 |
Mathias Agopian <mathias@google.com> |
fallout of cleaning-up libutils includes
Test: run & compile
Bug: n/a
Change-Id: I99e462d2db6560742fa3f065d5de50cd60111130
/frameworks/native/libs/binder/IMemory.cpp
|
| e0ea99cc4ba89452b9b6b1baa33bddcb11f3351a |
|
12-Jan-2017 |
Mark Salyzyn <salyzyn@google.com> |
resolve merge conflicts of a2c2d11f9 to master
Test: compile
Bug: 26552300
Bug: 31289077
Change-Id: I84d0ffa430df3beca0dcc6ed205ce220d7be1f87
|
| 7823e124e00576e20e47ec717cbe8bc89f0f2bf2 |
|
29-Sep-2016 |
Mark Salyzyn <salyzyn@google.com> |
Replace cutils/log.h and rationalize log/log.h and android/log.h
- cutils/log.h treat as deprecated and lead by example
- android/log.h to be used instead of log/log.h if possible
- add system includes that are assumed as side effects
- define LOG_TAG first
Test: compile
Bug: 31289077
Bug: 30465923
Change-Id: I8d99b24c333578c9b5aa9f2a01324bd0bba268dd
/frameworks/native/libs/binder/IMemory.cpp
|
| c03d283e8b3f830d76dd94822b2a13872c05c730 |
|
28-Dec-2016 |
Mark Salyzyn <salyzyn@google.com> |
resolve merge conflicts of 525a5f2 to master
Test: build
Change-Id: Id1c6d3d9dd158fb086b0d8ab74f7838ef0a51af1
|
| a5e161b1207ef447a51e99856097d69d4a6111e1 |
|
29-Sep-2016 |
Mark Salyzyn <salyzyn@google.com> |
Replace cutils/log.h and rationalize log/log.h and android/log.h
- cutils/log.h treat as deprecated and lead by example
- android/log.h to be used instead of log/log.h if required
- add system includes that are assumed as side effects of log.h
- modules that use SLOG should use SLOG for all logging (installd)
- define LOG_TAG first
- remove logging infrastructure if not used
Test: build
Bug: 31289077
Change-Id: Iea147a0104c7ab7f12451304131d6500f42141e7
/frameworks/native/libs/binder/IMemory.cpp
|
| ec9ec7d55c63f791ab3ed9221e68d6215f7b928a |
|
18-Dec-2016 |
Nick Kralevich <nnk@google.com> |
libbinder: replace dup() with fcntl(F_DUPFD_CLOEXEC)
Replace calls to dup() with fcntl(F_DUPFD_CLOEXEC). The only difference
between the two is that O_CLOEXEC is set on the newly duped file
descriptor. This helps address file descriptor leaks crossing an exec()
boundary in multi-threaded processes, and potentially fixes the following
non-reproducible SELinux denials which may be occurring because of FD
leakage from netd to clatd/dnsmasq.
avc: denied { use } for comm="clatd" path="socket:[860297]" dev="sockfs"
ino=860297 scontext=u:r:clatd:s0 tcontext=u:r:untrusted_app:s0:c512,c768
tclass=fd permissive=0
avc: denied { read write } for comm="clatd" path="socket:[1414454]"
dev="sockfs" ino=1414454 scontext=u:r:clatd:s0
tcontext=u:r:system_server:s0 tclass=tcp_socket permissive=0
avc: denied { use } for comm="clatd" path="socket:[681600]" dev="sockfs"
ino=681600 scontext=u:r:clatd:s0 tcontext=u:r:priv_app:s0:c512,c768
tclass=fd permissive=0
Test: Device boots and no obvious problems
Change-Id: I9dcd9911a093f329c6f12e39d2c49ef3df651ae5
/frameworks/native/libs/binder/IMemory.cpp
|
| 4ce4db768d64d2aa7a958bb25c439e23b8b59fd3 |
|
13-Jul-2016 |
Hans Boehm <hboehm@google.com> |
Fix memory ordering issues; document IMemory peculiarities
Convert to standard atomics.
Correct mHeapId memory ordering. Required acquire ordering on loads
was missing in several places.
Remove atomic updates to count, since it is only updated with lock
held. (And would be missing fences if this were not true.)
Document the peculiar use of copy-in-write vectors in a context in
which copy-on-write is unsafe.
While we're here, consistently check dup() for errors.
Bug: 28816986
Merged-in: I05b9f96e3867fa2e0abe6f319be8c56b89624c41
Change-Id: I05b9f96e3867fa2e0abe6f319be8c56b89624c41
/frameworks/native/libs/binder/IMemory.cpp
|
| 7e20297c52edc7aae41cf7453b2aad1d0a318d8f |
|
13-Jul-2016 |
Hans Boehm <hboehm@google.com> |
Fix memory ordering issues; document IMemory peculiarities
Convert to standard atomics.
Correct mHeapId memory ordering. Required acquire ordering on loads
was missing in several places.
Remove atomic updates to count, since it is only updated with lock
held. (And would be missing fences if this were not true.)
Document the peculiar use of copy-in-write vectors in a context in
which copy-on-write is unsafe.
While we're here, consistently check dup() for errors.
Bug: 28816986
Change-Id: I05b9f96e3867fa2e0abe6f319be8c56b89624c41
/frameworks/native/libs/binder/IMemory.cpp
|
| a2db19c4cb0146713dca1aa6f1b3caf424809aba |
|
13-Jun-2016 |
John Eckerdal <john.eckerdal@sonymobile.com> |
Merge \"Fixed file descriptor leak in IMemory\" into nyc-dev
am: a13a22649b
Change-Id: I8ab67bafd9c9031924cfb7cf3b2c07a7dce45795
|
| 6b0b06348d4cc8eb0087d95826dbeba9a72565d8 |
|
21-Apr-2016 |
John Eckerdal <john.eckerdal@sonymobile.com> |
Fixed file descriptor leak in IMemory
Include dup in critical section to avoid multiple dup calls from
different threads.
The leak happens more frequently since patch in frameworks/av -
6773d4777f4ccbbe6377e4ae1b42c117066ae6ba when recording video since it
will access IMemory from CameraUtils::isNativeHandleMetadata
Bug: 28924660
Author: Henrik Antonsson <henrik.antonsson@sonymobile.com>
Change-Id: I769a914c3adc505b522659a0c43719fb2922e6fd
/frameworks/native/libs/binder/IMemory.cpp
|
| 68a593e7fa07c947fef2c5de53c42e14ed630a8a |
|
28-Apr-2016 |
Chih-Hung Hsieh <chh@google.com> |
resolve merge conflicts of 168dc86 to nyc-dev-plus-aosp
Change-Id: I1554cec6ea880a87921f723bc96972cb8e1d3c5c
|
| e2347b7953f70232541bdc1b392230d6231aaeb9 |
|
26-Apr-2016 |
Chih-Hung Hsieh <chh@google.com> |
Fix google-explicit-constructor warnings.
Bug: 28341362
Change-Id: I0ce850de85e0de6a5472dd142f738856142db9ec
/frameworks/native/libs/binder/IMemory.cpp
|
| ebdad60d6b0560493e50092c30888d3a7b5cc8c9 |
|
17-Feb-2016 |
Christopher Tate <ctate@google.com> |
Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de
am: 84f488f354
* commit '84f488f354c33a7ece30cb569c85cd91f0259066':
Sanity check IMemory access versus underlying mmap
|
| 97f49e50ded651f1ddcf8a9981a7d97fe2816ad7 |
|
17-Feb-2016 |
Christopher Tate <ctate@google.com> |
Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab
am: ef6908e2b3
* commit 'ef6908e2b3e6ee6514620acc338b458ade7c3640':
Sanity check IMemory access versus underlying mmap
|
| 94b0d4e3ab023cfa03a7a4e85f3e09d3743da715 |
|
06-Feb-2016 |
Christopher Tate <ctate@google.com> |
Sanity check IMemory access versus underlying mmap
Bug 26877992
Change-Id: Ibbf4b1061e4675e4e96bc944a865b53eaf6984fe
/frameworks/native/libs/binder/IMemory.cpp
|
| 097ca275f4717a2c47a5d49f302ed2b72c8a1370 |
|
14-Nov-2014 |
Marco Nelissen <marcone@google.com> |
Replace IInterface::asBinder() with a static
so we can do NULL checks again, and update calls to IInterface::asBinder()
to use the new static version.
Change-Id: Ia7b10eb38ca55b72278bfd33d3bf647f338b4e6a
/frameworks/native/libs/binder/IMemory.cpp
|
| 2ea926bda2fa30f3ba8d0ed2d2395a8ada952e6e |
|
14-Nov-2014 |
Marco Nelissen <marcone@google.com> |
Replace IInterface::asBinder() with a static
so we can do NULL checks again, and update calls to IInterface::asBinder()
to use the new static version.
Change-Id: Ia7b10eb38ca55b72278bfd33d3bf647f338b4e6a
/frameworks/native/libs/binder/IMemory.cpp
|
| 6f4f3ab36c5ed1df84eb3a9f7475f0ac42952f58 |
|
06-Feb-2014 |
Colin Cross <ccross@android.com> |
binder: fix all warnings
Fix warnings related to casting pointers to ints, using %d or %ld to
print size_t/ssize_t, and unused parameters.
Change-Id: I7a13ba83d402952989c1f795cd9e880a95b98d9e
/frameworks/native/libs/binder/IMemory.cpp
|
| cab25d680e644d962041d05a319e485b96136a5d |
|
22-Mar-2013 |
Mathias Agopian <mathias@google.com> |
improved CallStack a bit
- added a ctor that updates and dumps the stack immediately
- added a "logtag" parameter to dump()
Change-Id: Ie51c256071d282591752243bdb4f68cf9ff8829d
/frameworks/native/libs/binder/IMemory.cpp
|
| c47f0988a36c3032a3c8850c4df3a704dd59e468 |
|
14-Mar-2013 |
Elliott Hughes <enh@google.com> |
Revert "Add a compatibility symbol for the MemoryBase constructor."
Cherry pick from AOSP to fix the build.
See https://android-review.googlesource.com/#/c/53690/
This reverts commit 7f974ff2f317ffbf9256579109f000ec876c3037
Change-Id: I1b123f446d0ff41e11ff81d4ef5f1472b59cac93
/frameworks/native/libs/binder/IMemory.cpp
|
| 7f974ff2f317ffbf9256579109f000ec876c3037 |
|
13-Mar-2013 |
Elliott Hughes <enh@google.com> |
Add a compatibility symbol for the MemoryBase constructor.
Third-party libraries are currently trying to use the
MemoryBase constructor but failing because we fixed the
definition of ssize_t. This is a stop-gap for users of
this private API until we can get them fixed.
Bug: 8253769
Change-Id: I8a19770f3252d88ee87023fde625cc6289924b0d
/frameworks/native/libs/binder/IMemory.cpp
|
| e6f43ddce78d6846af12550ff9193c5c6fe5844b |
|
06-Jan-2012 |
Steve Block <steveblock@google.com> |
Rename (IF_)LOGE(_IF) to (IF_)ALOGE(_IF) DO NOT MERGE
See https://android-git.corp.google.com/g/#/c/157220
Bug: 5449033
Change-Id: Ic9c19d30693bd56755f55906127cd6bd7126096c
/frameworks/native/libs/binder/IMemory.cpp
|
| 9d4536835248525f32f1504a3d28d5bbfa0a2910 |
|
20-Dec-2011 |
Steve Block <steveblock@google.com> |
Rename (IF_)LOGD(_IF) to (IF_)ALOGD(_IF) DO NOT MERGE
See https://android-git.corp.google.com/g/156016
Bug: 5449033
Change-Id: I4c4e33bb9df3e39e11cd985e193e6fbab4635298
/frameworks/native/libs/binder/IMemory.cpp
|
| 5728a92e29c1c9729017a82c5d0bc18fc1069923 |
|
22-Jun-2011 |
Anu Sundararajan <sanuradha@ti.com> |
MemoryHeapBase: Save and binderize the offset
The offset that is used in the creation of the MemoryHeapBase must be saved, so
that it can be used to recreate the Heap when an IMemory object is passed
across process boundary through the binder.
Change-Id: Ie618fb5c0718e6711f55ed9235616fd801e648dc
Signed-off-by: Anu Sundararajan <sanuradha@ti.com>
Signed-off-by: Iliyan Malchev <malchev@google.com>
/frameworks/native/libs/binder/IMemory.cpp
|
| 3c772b706dd8e53fe17b284bb2e60c146867f787 |
|
20-Apr-2010 |
Mathias Agopian <mathias@google.com> |
fix [2599939] "cannot play video" after open/close a video player a dozen of times
get rid off the MAP_ONCE flag is MemoryHeapBase (as well as it's functionality),
this feature should not be used anymore.
the software renderer was incorrectly using the default ctor which set MAP_ONCE,
causing the leak. the software renderer itself is incorrectly used while coming
back from sleep.
Change-Id: I123621f8d140550b864f352bbcd8a5729db12b57
/frameworks/native/libs/binder/IMemory.cpp
|
| 83c0446f27b9542d6c2e724817b2b2d8d1f55085 |
|
23-May-2009 |
Mathias Agopian <mathias@google.com> |
some work to try to reduce the code size of some native libraries
- make sure that all binder Bn classes define a ctor and dtor in their respective library.
This avoids duplication of the ctor/dtor in libraries where these objects are instantiated.
This is also cleaner, should we want these ctor/dtor to do something one day.
- same change as above for some Bp classes and various other non-binder classes
- moved the definition of CHECK_INTERFACE() in IInterface.h instead of having it everywhere.
- improved the CHECK_INTERFACE() macro so it calls a single method in Parcel, instead of inlining its code everywhere
- IBinder::getInterfaceDescriptor() now returns a "const String16&" instead of String16, which saves calls to String16 and ~String16
- implemented a cache for BpBinder::getInterfaceDescriptor(), since this does an IPC. HOWEVER, this method never seems to be called.
The cache makes BpBinder bigger, so we need to figure out if we need this method at all.
/frameworks/native/libs/binder/IMemory.cpp
|
| c5b2c0bf8007562536b822eb060fc54a01f8e08b |
|
20-May-2009 |
Mathias Agopian <mathias@google.com> |
move libbinder's header files under includes/binder
/frameworks/native/libs/binder/IMemory.cpp
|
| 208059f67ed2dd9fa025e07fcb6954d3cb61c79e |
|
19-May-2009 |
Mathias Agopian <mathias@google.com> |
checkpoint: split libutils into libutils + libbinder
/frameworks/native/libs/binder/IMemory.cpp
|