Merge "Revert "Make libc.so global""
am: e7ea7612f9

Change-Id: I975b9aa0aaa8307592593163fef1bbfb2940f8e5

Merge "Revert "Make libc.so global""

Merge "Be more strict about using invalid `pthread_t`s."
am: 0370905502

Change-Id: I476dfe579855d06ea7a1ab34888642f43c58725b

Revert "Make libc.so global"

This reverts commit 879177c9e12df4efe9f17ff3536005c258f625e8.

Bug: http://b/26833548
Change-Id: I23e0b34777d18e369063185da14c57994807da29

Merge "Be more strict about using invalid `pthread_t`s."

Merge "Add __system_property_wait and return the serial in __system_property_read_callback."
am: ade3be476f

Change-Id: I33c5acd3ebba551723639aa2b2540603e2434b4c

Merge "Add __system_property_wait and return the serial in __system_property_read_callback."

Be more strict about using invalid `pthread_t`s.

Another release, another attempt to remove the global thread list.

But this time, let's admit that it's not going away. We can switch to using
a read/write lock for the global thread list, and to aborting rather than
quietly returning ESRCH if we're given an invalid pthread_t.

This change affects pthread_detach, pthread_getcpuclockid,
pthread_getschedparam/pthread_setschedparam, pthread_join, and pthread_kill:
instead of returning ESRCH when passed an invalid pthread_t, if you're
targeting O or above, they'll abort with the message "attempt to use
invalid pthread_t".

Note that this doesn't change behavior as much as you might think: the old
lookup only held the global thread list lock for the duration of the lookup,
so there was still a race between that and the dereference in the caller,
given that callers actually need the tid to pass to some syscall or other,
and sometimes update fields in the pthread_internal_t struct too.

(This patch replaces such users with calls to pthread_gettid_np, which
at least makes the TOCTOU window smaller.)

We can't check thread->tid against 0 to see whether a pthread_t is still
valid because a dead thread gets its thread struct unmapped along with its
stack, so the dereference isn't safe.

Taking the affected functions one by one:

    * pthread_getcpuclockid and pthread_getschedparam/pthread_setschedparam
      should be fine. Unsafe calls to those seem highly unlikely.

    * Unsafe pthread_detach callers probably want to switch to
      pthread_attr_setdetachstate instead, or using
      pthread_detach(pthread_self()) from the new thread's start routine
      rather than doing the detach in the parent.

    * pthread_join calls should be safe anyway, because a joinable thread
      won't actually exit and unmap until it's joined. If you're joining an
      unjoinable thread, the fix is to stop marking it detached. If you're
      joining an already-joined thread, you need to rethink your design.

    * Unsafe pthread_kill calls aren't portably fixable. (And are obviously
      inherently non-portable as-is.) The best alternative on Android is to
      use pthread_gettid_np at some point that you know the thread to be
      alive, and then call kill/tgkill directly.

      That's still not completely safe because if you're too late, the tid
      may have been reused, but then your code is inherently unsafe anyway.

Bug: http://b/19636317
Test: ran tests
Change-Id: I0372c4428e8a7f1c3af5c9334f5d9c25f2c73f21

Merge "Do not use std::vector in android_namespace_t::is_accessible"
am: 9fc52deab1

Change-Id: I0449e82bb130e9eed59ccf932a97ce69c16b7a4d

Merge "Do not use std::vector in android_namespace_t::is_accessible"

Add __system_property_wait and return the serial in __system_property_read_callback.

In order to implement android::base::WaitForProperty well, we need a way to
wait not for *any* property to change (__system_property_wait_any), but to
specifically wait for the property represented by a given `prop_info` to
change.

The android::base::WaitForProperty implementation, like attempts to cache
system properties in the past, also needs a way to keep serials and values
in sync, but the existing functions don't provide a cheap way to get a
consistent snapshot. Change the __system_property_read_callback callback's
type to include the serial corresponding to the given value.

Add a test, slightly clean up some of the existing tests (and name them to
include the names of the functions they're testing, in our usual style).

Bug: http://b/35201172
Test: ran tests
Change-Id: Ibc8ebe2e88eef1e333a1bd3dd7f68135f1ba7fb5

Merge "Make libc.so global"
am: 5b74e9b2f5

Change-Id: Ie5dc459b78b4b7fad148af3196aeec6df1faba59

Merge "Make libc.so global"

Merge "Match __bos0 to __pass_object_size0 in FORTIFY"
am: 572b1dde03

Change-Id: Ic2993a4692105f18d24573297479ce7f57964dea

Merge "Match __bos0 to __pass_object_size0 in FORTIFY"

Do not use std::vector in android_namespace_t::is_accessible

Avoid constructing vector and walking all the parents of a soinfo
to check if it is accessible. The most likely scenario that the
very first check returns true.

Bug: http://b/35313368
Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Change-Id: I06c65cf61ed1c30e5e454a169de4c41038863587

Make libc.so global

This is a way to avoid loading multiple libc.so
when non-default namespace search path includes
/system/lib. This is used by some art tests.

Bug: http://b/26833548
Test: m -j32 test-art-target-run-test-004-JniTest
Change-Id: I919d3a0560bd3c9ac19df21a235641a667f0f017

Merge "Replace public library list with shared lib sonames (part 2/2)"
am: 2cb4dfbb8c

Change-Id: Ie62e6810486fce1c2e84df7991ba11c8d69a6fef

Merge "Replace public library list with shared lib sonames (part 2/2)"

Merge "loader: fix d-tor call order"
am: c9fb1774c5

Change-Id: I99b0284fcb0b5d0b3ec011b969d174a1ad8db32e

Merge "loader: fix d-tor call order"

Match __bos0 to __pass_object_size0 in FORTIFY

pass_object_size(N) forwards the result of __builtin_object_size(param,
N) to a function. So, a function that looks like:

  size_t foo(void *const p __pass_object_size) { return __bos0(p); }
  int bar = foo(baz);

would effectively be turned into

  size_t foo(void *const p, size_t sz) { return sz; }
  int bar = foo(baz, __bos(baz)); // note that this is not __bos0

This is bad, since if we're using __bos0, we want more relaxed
objectsize checks.

__bos0 should be more permissive than __bos in all cases, so this
change Should Be Fine™.

This change also makes GCC and clang share another function's
implementation (recv). I just realized we need to add special
diagnostic-related overloads bits for clang to it, but I can do that in
another patch.

Bug: None
Test: Bullhead builds and boots; CtsBionicTestCases passes.
Change-Id: I6818d0041328ab5fd0946a1e57321a977c1e1250

loader: fix d-tor call order

In the case when there are multiple dependencies on
the same library in the local_group the unload may
in some situations (covered now by tests) result
calling d-tors for some libraries prematurely.

In order to have correct call order loader checks if this
is last dependency in local group before adding it to BFS
queue.

Bug: http://b/35201832
Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Test: bionic-unit-tests-glibc --gtest_filter=dl*
Change-Id: I4c6955b9032acc7147a51d9f09b61d9e0818700c

Merge "Replace public library list with shared lib sonames (part 1/2)"
am: 132768084e

Change-Id: If0454fdc0c632d1560662cbd0f660687bbb35fad

Merge "Replace public library list with shared lib sonames (part 1/2)"

Replace public library list with shared lib sonames (part 2/2)

This commit updates interface of libdl.c.

1. android_init_namespaces is replaces with android_init_anonymous_namespace
2. added 2 arguments to android_create_namespace to specify linked namespace
   and the list of shared libraries sonames.
3. symbol lookup does not get past boundary libraries (added check and test for it).

Bug: http://b/26833548
Bug: http://b/21879602
Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Change-Id: I32921da487a02e5bd0d2fc528904d1228394bfb9

Replace public library list with shared lib sonames (part 1/2)

Replace public library list with shared lib sonames
which are property of a link between namespaces

This change does not touch any external interfaces
so from outside it behaves almost as it was before

One significant difference is that there is no longer
need to preload public libraries.

Bug: http://b/26833548
Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Change-Id: I57e44e18a9b4f07dcd6556436346be52f52b79d7

Merge "Removing the kuser_helper elf note from building"
am: e86a8d605b

Change-Id: Ica80892b0973f2370bc10d23aa4279fd76c6ef7e

Merge "Removing the kuser_helper elf note from building"

Merge "Remove unnecessary memsets in linker allocators"
am: 43801a50b0

Change-Id: I9916105c744178672b03803234b3c22e64435f8b

Removing the kuser_helper elf note from building

This change removes the kuser_helper note from building automatically
with any binary but leaves the note.
Also fixes a typo in the note.

Original patch adding the note:
Ib8366e2a0810092b71381d57dee4bdaa56369a24

Bug: 34815073
Test: Manual - Note is no longer added to binaries
Change-Id: Ieb81f9d9127d1f8a522434a31c696d743238e2a5

Merge "Remove unnecessary memsets in linker allocators"

Merge "libc: add clang FORTIFY support"
am: 5ac650936f

Change-Id: Id9b7bc08a9c60f1ce34768063cdc8d61098ffc0e

Merge "libc: add clang FORTIFY support"

Merge "Add test checking init/fini call order"
am: 4e7ab9292d

Change-Id: If03d6740dbd8e9202ca39a502854a98c95cb4500

Merge "Add test checking init/fini call order"

Remove unnecessary memsets in linker allocators

Test: linker-unit-tests
Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Change-Id: I96f983dba77c9cb5698a9eb2e51b33bc128c6dd7

libc: add clang FORTIFY support

This patch adds clang-style FORTIFY to Bionic. For more information on
FORTIFY, please see https://goo.gl/8HS2dW . This implementation works
for versions of clang that don't support diagnose_if, so please see the
"without diagnose_if" sections. We plan to swap to a diagnose_if-based
FORTIFY later this year (since it doesn't really add any features; it
just simplifies the implementation a lot, and it gives us much prettier
diagnostics)

Bug: 32073964
Test: Builds on angler, bullhead, marlin, sailfish. Bionic CTS tests
pass on Angler and Bullhead.

Change-Id: I607aecbeee81529709b1eee7bef5b0836151eb2b

Add test checking init/fini call order

This test is checking that loader complies with the order described in
http://www.sco.com/developers/gabi/latest/ch5.dynamic.html#init_fini

Bug: http://b/35201832
Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Test: bionic-unit-tests-glibc --gtest_filter=dl*
Change-Id: I4cdf878c043112442c191e82aa9f5d5077e4d8c4

Merge "Add "benchmark" to time atomic operations"
am: 2bfb8c9180

Change-Id: I2111c967865e706aec9ff2ab2519e0eaee2e4f66

Merge "Add "benchmark" to time atomic operations"

Merge "Revert "Stop checking the global thread list in several trivial cases.""
am: f201c704d6

Change-Id: Ib18af2245909eb8b495c40deb1ca7cea04e73b8f

Merge "Revert "Stop checking the global thread list in several trivial cases.""

Revert "Stop checking the global thread list in several trivial cases."

This reverts commit f5a4992b7186c01b98cb9400b2b5b490805a87bb.

Breaks OMX_ImgEnc in cameraserver (http://b/35088254).

Change-Id: I6dcf12706a184b0b8b72451584567a42dfa1bb4f

Merge "Improve a comment."
am: a1fa522927

Change-Id: Ia3c2609527c9631aa92f3a030978215f97811c9e

Merge "Improve a comment."

Merge "Stop checking the global thread list in several trivial cases."
am: 4708f435f5

Change-Id: I1b3ba92036b68a94f342579ea34c385e3496a84b

Merge "Stop checking the global thread list in several trivial cases."

Stop checking the global thread list in several trivial cases.

Since removing the global thread is hard, let's take the different
groups of functions individually.

The existing code was racy anyway, because the thread might still be
on the list but have exited (leaving tid == 0).

Bug: http://b/19636317
Test: ran tests
Change-Id: Icc0986ff124d5f9b8a653edf718c549d1563973b

Improve a comment.

Bug: http://b/23675822
Test: builds
Change-Id: Ia58ffabf38012035076e4c682faa4c93f3690e24

Merge "Create seccomp policy without TRAP for further processing"
am: 4a47d2c436

Change-Id: Ib16e18901aac6dbe91ffb326bb8bb31b9d9116e2

Merge "Create seccomp policy without TRAP for further processing"

Create seccomp policy without TRAP for further processing

Bug: 34946764
Test: Make sure boots, seccomp still blocks, and is faster
Change-Id: Ib4abf4307ae545ee69a3fb9328f62c760a1b40f7

Merge "Fix CFI initialization crash on x86."
am: 9886ea3e2d

Change-Id: I115c36331b2e531d3e33848e5251847e47f1a8b1

Merge "Fix CFI initialization crash on x86."

Merge "isnan and isinf aren't functions."
am: 33ae5a97f2

Change-Id: Ibdfdec89d9e7bd04f0b42b74b20d3a2b7521e63c

Merge "isnan and isinf aren't functions."

Fix CFI initialization crash on x86.

Third try.

Bug: 34752378
Test: bionic tests
Change-Id: I247c127489a8ee38404e104f28d916a704e35f36

isnan and isinf aren't functions.

isnan was a function in POSIX in 1997, but changed to a macro only in 2004
to align with the C standard. isinf wasn't in POSIX until 2004, where it
appeared as a macro only (but other C libraries already had it as a
function).

Now the C++ standard has added ::isnan and ::isinf functions with different
signatures from the historical ones, so we need to move our historical cruft
out of the way.

We'll keep the implementations for backwards compatibility.

Bug: http://b/34724220
Test: ran tests
Change-Id: Id665f0344af6fe6ed245106e60231f4ef2027f41

Merge "Revert "Remove the global thread list.""
am: a187a6f12e

Change-Id: I3df28298769f7f83aadad09a80166ce9a150e540

Merge "Revert "Remove the global thread list.""

Revert "Remove the global thread list."

This reverts commit b0e8c565a622b5519e03d4416b0b5b1a5f20d7f5.

Breaks swiftshader (http:/b/34883464).

Change-Id: I7b21193ba8a78f07d7ac65e41d0fe8516940a83b

Merge "Exit failure if trying to load non-PIE."
am: 1fedfedda8

Change-Id: I55eaba81f182ff854e81d7de2b882b4dc26a7c23

Merge "Exit failure if trying to load non-PIE."

Merge "Check overflows in c-tor instead of doing it in d-tor"
am: 118b198af6

Change-Id: Ia6d3e43724799103b287bab09d7e7bfd2ba7b5ab

Merge "Check overflows in c-tor instead of doing it in d-tor"

Check overflows in c-tor instead of doing it in d-tor

Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Change-Id: Id3b17a79732155b4e469c43c764afef1dfc59057

Merge "Do not load second copy of a library into same namespace"
am: 3c838bbb4a

Change-Id: If1effbf7f7591bc0759b55d8dc0fcaafbf651e17

Do not load second copy of a library into same namespace
am: 3bd906146b

Change-Id: I11498855ff95af2af4b0edcc3a5712d957470b55

Merge "Do not load second copy of a library into same namespace"

Do not load second copy of a library into same namespace

This workaround was introduced in M in order to make sure
that linker loads libraries provided in apk in case a library
with the same name was loaded by the system.

This is no longer a problem starting with Android N because
app is using different namespace and therefore does not see
libraries loaded by the system.

Test: bionic-unit-tests --gtest_filter=dl*:Dl*
Change-Id: I9995258c0f361e8df35420682b84e85a0cb924b4

Merge "Remove the global thread list."
am: 9b9d4ebc8b

Change-Id: I590ffebd5bfa287fdc4c7018e487c7c086f8ae7e

Merge "Remove the global thread list."

Merge "bionic: move ADF uapi header out of bionic"
am: b88112dcfc

Change-Id: I9e8d80152b5b43fbf6a0544de5f4d957785be891

Merge "bionic: move ADF uapi header out of bionic"

Merge "Revert "Fix CFI initialization crash on x86.""
am: 9586fe2744

Change-Id: Iae5548dea289cd77b406d37ecb13e9123d9f490f

Merge "Revert "Fix CFI initialization crash on x86.""

Revert "Fix CFI initialization crash on x86."

Breaks aosp-master/sdk.

This reverts commit 4ccd4317f9301ec9b32aa0d3317a0439b994d7bf.

Merge "Fix CFI initialization crash on x86."
am: dd76f744e7

Change-Id: I66175dd9bc705a64a5ad59edf26d471b4f304ea3

Merge "Fix CFI initialization crash on x86."

Exit failure if trying to load non-PIE.

Amazingly this was actually breaking the NDK's ability to build
libstdc++ for x86.

Test: mma
Bug: None
Change-Id: Iafa55c31fdeb35caca7d7d7a39a3e7afa0713557

Fix CFI initialization crash on x86.

Second try.

Bug: 34752378
Test: bionic tests
Change-Id: I0a7453154671918b2e6e72f0bb9074c29ceb0f34

Merge "s/NULL/nullptr/ in system properties code."
am: eb35144e58

Change-Id: Ibfa128262eacf89b20f58c6de88b744978e6087d

Add "benchmark" to time atomic operations

The intent here is to confirm that the compiler behaves as expected,
and to guide optimization efforts, particularly where there is a
choice between primitives.

Test: Built and ran benchmark on Angler repeatedly. Manually confirmed
that the compiler behaves roughly as expected.

Change-Id: I059b245d1ba8296e9b28602559b53eafafe0a30f

Merge "s/NULL/nullptr/ in system properties code."

s/NULL/nullptr/ in system properties code.

gerrit still hasn't sent out the mail asking for this in an earlier code
review...

Bug: http://b/33926793
Test: builds and boots.
Change-Id: I080de633e50e47f9a052211d47da96b971b16424

Merge "clang-format system_properties.cpp."
am: 44bea80d31

Change-Id: Ib3365fbb65588d6e7f65212a9e764714cc64616b

Merge "clang-format system_properties.cpp."

Merge "Revert "Fix CFI initialization crash on x86.""
am: 9750d6b691

Change-Id: I40dde15cb6a5a6748284a0aebce7caa8865c41f1

Merge "Revert "Fix CFI initialization crash on x86.""

Revert "Fix CFI initialization crash on x86."

This reverts commit be46d3871c91902504e5ec4c7f575c86f647aafc.

Reverting due to broken build:

https://android-build.googleplex.com/builds/branch-dashboard/aosp-master?build_id=3687823

Change-Id: I30968398edd9a094fce878839ba0a002ed487a77

clang-format system_properties.cpp.

Asked for on a previous code review.

Bug: http://b/33926793
Test: builds and boots.
Change-Id: Idf34804896149db5edbddf4ed88396a5df7f708a

Merge "Fix CFI initialization crash on x86."
am: a5066640cc

Change-Id: Ie0ecb7225f7c392f5229cbfc49b6eb396789e969

Merge "Fix CFI initialization crash on x86."

Merge "glibc ipc64_perm __key compatibility."
am: 5dc75c6c4d

Change-Id: I781475ecad513babc647115f606e316ad4599863

Merge "glibc ipc64_perm __key compatibility."

Fix CFI initialization crash on x86.

Bug: 34752378
Test: bionic tests
Change-Id: If8e33f76a1a2d83356d818fed506ea624f579860

Merge "property_contexts: split into platform and non-platform components"
am: 93e272d00a

Change-Id: Iad3eada49f676597bc3210e1843d6bdd21c8aa8b

property_contexts: split into platform and non-platform components
am: 34f0cfa3a8

Change-Id: Ia1bcc896d3901217323a3d1299aada8223eceff5

Merge "property_contexts: split into platform and non-platform components"