am: 8c5384e50b
* commit '8c5384e50bfbc97e4b4ea74e52391879a09b68fa':
Report errno more often in fastboot.
Change-Id: I19a33597f29c1974063d4308eeccea60bbb350f5
am: e674d32d2d
* commit 'e674d32d2d2816255c836cb7eed51eee6c9e8147':
Add a fortify test to crasher.
Change-Id: I7d4c1342c514c73d195ad970703bb20338633db8
Passing empty permitted path should result in no-op.
This addresses review comment on CL with commit
f334cbf0e1
Bug: http://b/28639227
Change-Id: I4a4540e522e90a145a374939921932c86f35e88d
(cherry picked from commit d0b1531929)
am: 0756fae4f1
* commit '0756fae4f1aea5072ed820087b2487b408194c38':
Check if public namespace has already been initialized
Change-Id: Ia338baa984c3ca310f4a30db1d094704607e5cc9
Apps on sdcard live under /mnt/expand and therefore
this directory should be under linker namespaces
permitted_path.
Bug: http://b/28639227
Change-Id: I462f9f23656c95d9c2a48bb3f513abcd9d08f340
(cherry picked from commit f334cbf0e1)
am: c5016de66b
* commit 'c5016de66b6d778dd7c292e215aad6b5c079e92c':
<cutils/atomic.h> has been inlined for two years now.
Change-Id: Icc28b866c2dd081d1047263c7cfaea05d20975f6
There is no point in reading/loading the list of public
libraries once public namespace is already initialized.
Change-Id: I5a53c46bb57b8de241ba521a6321c12e94a76476
They are intended for platform use only and we shouldn't have them
in the public header file. This CL moves them to libnativeloader
private header file.
Bug: http://b/28174921
Change-Id: I01cf446d2d8d7cf6f9e31130aa3d4e54fbbbb9fc
am: 179fea40bb
* commit '179fea40bb1ed2da183f3a2a6398d1c68b9f6f32':
nativeloader: Fix the case of search_path == null
Change-Id: Idb502568c161693877c30bb7cd085e974433aa8f
When user creates custom classloader which does not extend
BaseDexClassLoader the librarySearchPath gets set to null
by java.lang.Runtime.doLoad().
This patch makes nativeloader correctly handle it.
Bug: http://b/28659864
Change-Id: I1b61c6bc952984d7c49775a9178fc3270948e62a
am: c6160d2a0e
* commit 'c6160d2a0ef648ccb3d217c589c60b5c00b80387':
Add public libs from an environment variable
Change-Id: I2be47c2e9fc02860e7ddd79b97f666c9ea6e881b
This is enabled only for builds with ro.debuggable=1
It is intended for use only in tests using dalvikvm
and needing access to platform libraries.
Bug: http://b/28449304
Change-Id: I402457d0da542996ccf265aeaa305f09881e4333
am: f34304019d
* commit 'f34304019d464de0c03fe62ca7fd9e158afb598c':
Set groups before dropping privileges.
Change-Id: If5396c913cfed13c01bd06d6a5d781439e5f6876
The code for dumping tombstones tries to read the log data. This was
silently failing after the change to drop root privileges. Fix this
by explicitly setting the groups allowed before dropping privileges.
Bug: 28590884
(cherry picked from commit edc2380109)
Change-Id: I225365edccb11f8752027566fd42f3b10ce10260
In the function dump_thread_info, if /proc/<PID>/comm is unreadable, then
threadname will be used in a strncmp causing a crash. The fix is to
avoid the check if threadname is null.
Bug: 28615417
(cherry picked from commit 039976e553)
Change-Id: I1e61431b1549ecfdc49a72cbf193a364069cda79
The previous false positive fix (df9c4a01) is enough to pass tests, and
it doesn't appear that there are any remaining issues.
Change-Id: Ib9812f1201ff0cd2ae8c8371737754fc328765b5
When someone enables wait_for_gdb, activity manager will kill the
stopped process before a developer can attach to the process. To
allow debugging in this case, change the code to only contact the
activity manager right before continuing the process that is
crashing.
Also, modify the conditions under which to do a gdb attach. The previous
code did a partial attach if perform_dump failed. The new version simply
allows an attach regardless of whether perform_dump passes or fails.
Bug: 28409358
(cherry picked from commit 9818bd2bbe)
Change-Id: I42f464b69332748e16b07d9d00f44b3aa26ce8b7
This method is called by art on UnloadNativeLibraries()
Bug: http://b/28406866
Change-Id: Ibccff93c83d6ccd3d0a1c0294a64b09281b4e05e
(cherry picked from commit 09a516bf16)
