Merge "Add __attribute__((sentinel)) tags to execl and friends."

Merge "Check alignment along with range of mapped file fragments"

Merge "Move brillo closer to Android."

Merge "Remove __static_cast macro from <sys/cdefs.h>."

Add __attribute__((sentinel)) tags to execl and friends.

Give a compile time diagnostic when the nullptr sentinel is missing
from an execl family call, instead of just pulling arbitrary values.

Bug: http://b/30793878
Change-Id: I1c49005c58b7ad4a5b04f0435b722d4c744e8f0e
Test: Built bullhead

Remove __static_cast macro from <sys/cdefs.h>.

We don't use it, and we added the more general __BIONIC_CAST anyway.

Change-Id: I2cb8a108b58bb2cb24ed2b1890d9efed671196dc

Merge "Fortify vsnprintf in more cases."

Move brillo closer to Android.

Hiding our legacy cruft seemed like a good idea, but in practice it will only
mean worse interoperability.

Plus we got it wrong, as the recent `putw` example showed.

Change-Id: I167c7168eff133889028089c22a7a0dfb8d6d0cf

Fortify vsnprintf in more cases.

Bug: http://b/30445072
Change-Id: I1893890f0e3b56533eef053eda1bd96a0b9a5119

Check alignment along with range of mapped file fragments

Improve sanity-checks of elf-file by adding alignment check
of mapped sections and section headers.

Bug: http://b/30687964
Change-Id: I7f06ddaa56a13989ce7be847b3a73c352e32c008
(cherry picked from commit 7e2d49ae3ebbae7ebf1ca033f86e87002c0dd83c)

Merge "Remove more stdio copy/paste."

Remove more stdio copy/paste.

Change-Id: Ia92629b75d2c153ecf1cec711e2f9575eef604ab

Merge "linker: fix mips build"

linker: fix mips build

Change-Id: I88c5ba7f33825ae9b36992b32d30579ec4a66915
Test: lunch aosp_mips-eng && mm

Merge "Don't use the same declaration to declare multiple symbol names."

Merge changes I868417f4,I57ac3667

* changes:
  Extract soinfo and globals to separate files.
  Move android_namespace_t to a separate file.

Don't use the same declaration to declare multiple symbol names.

This doesn't play well with preprocessing.

Bug: http://b/30170081
Change-Id: Id7fc648113336dcccdfa0a6b108f3d452aaf1ed2

Extract soinfo and globals to separate files.

Move soinfo and globals out of linker.cpp to
separate files.

Breaking up huge linker.cpp into smaller peaces
in order to make it easier to extract part of the
code that belongs to libdl.so and remove parts of
the code that do not belong to linker

(refactoring part 2 of many)

Change-Id: I868417f4b8d2b84d0e8265e354bc7977161497e2

Move android_namespace_t to a separate file.

Breaking up huge linker.cpp into smaller peaces
in order to make it easier to extract part of the
code that belongs to libdl.so and remove parts of
the code that do not belong to linker

(refactoring part 1 of many)

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

Merge "Update NOTICE files."

Update NOTICE files.

Change-Id: I591dc91b54804aebc05ba6f9974ef9add660ecfe

Merge "Guard fortify inlines with __ANDROID_API__."

Merge "Reimplement remove(3) without the lstat(2)."

Guard fortify inlines with __ANDROID_API__.

Test: make checkbuild with libc ndk_library patches applied
Change-Id: Ic5e248994c4f2702b0f4d6dfeda787187ea86017

Reimplement remove(3) without the lstat(2).

This assumes that it's more likely we're unlinking a file than a directory,
though even if that's not true, as long as a failed unlink(2) is cheaper
than a successful lstat(2) -- which seems likely since there's no data to
copy -- we still win.

Change-Id: I0210e9cd3d31b8cf1813c55c810262ef327382ed

Merge changes I01c4d9a8,Ic34734b4

* changes:
  Run genversion-scripts.py.
  Annotate libdl.map.txt.

Run genversion-scripts.py.

Change-Id: I01c4d9a85d3397b02b00b4d33fafe6d9ae692ed7

Annotate libdl.map.txt.

Test: readelf diff between soong generated stubs and prebuilts/ndk
Bug: http://b/30465923
Change-Id: Ic34734b40870c8b628449458798c0343648e4e97

Merge "Guard include of uchar.h in stdatomic.h"

Guard include of uchar.h in stdatomic.h

Bionic stdatomic.h is also used for the host (via a copy in
prebuilts/clang).  Revert to guarding the include of uchar.h based on
__STDC_VERSION__, so it is included only when needed.

Change-Id: I5b45c7f5d16da223478512104702a3e4e5a975ae
Test: bionic tests on host and Angler.  Confirmed failures I am
      seeing are KIs.

Merge changes Iecc1b127,Ib67f07db,I7672d34c

* changes:
  Run genversion-scripts.py.
  Only check arch tag if we have *any* arch tags.
  Add pylintrc.

Merge changes Ib5962ea3,I8c1a7464,I06ce7c31

* changes:
  Add version information for legacy inlines.
  versioner: remove unnecessary kernel/common symlink.
  versioner: add symlink for kernel/android/uapi.

Run genversion-scripts.py.

Change-Id: Iecc1b127c67a96aecc4e86582c1cc3291a4acf00

Only check arch tag if we have *any* arch tags.

With the introduction of new tags for ndk_library, we'll have a lot
of tags that aren't architecture tags. If we have something tagged
`introduced=21`, it should be in all architectures.

Change-Id: Ib67f07db14625f6903919c181050316eb183bed5

Add pylintrc.

Disable all the style differences present in genversion-scripts.py.

Change-Id: I7672d34c4f7698ba445968dd5d9f5ef1fdc60056

Merge changes Iedab3259,I333fe8ae

* changes:
  Put bsd_signal back.
  Unversion some symbols for the NDK.

Add version information for legacy inlines.

Bug: http://b/30170081
Change-Id: Ib5962ea3037dd692a98b6691d263871549aac8af

versioner: remove unnecessary kernel/common symlink.

The target was moved by commit 5956b4e, but none of the headers
contained were actually necessary to compile any bionic headers.

Bug: http://b/30170081
Change-Id: I8c1a7464de0992c20af194340c0e36d131a836e7

versioner: add symlink for kernel/android/uapi.

The android-only kernel headers were moved to their own directory by
commit d6e8b8c. Add a new symlink to match.

Change-Id: I06ce7c3167fbb1e058e4a137fae9375bb5792a46

Merge "Correct libstdc++ version"

Put bsd_signal back.

With this change (and all the others below it in the stack), the only
difference between the arm android-9 libc.so in r13-beta1 and the one
we are now generating is the addition of LIBC_N and LIBC_O versions.

Test: make ndk, readelf stub libc.so to check symbol exists
Bug: https://github.com/android-ndk/ndk/issues/160
Change-Id: Iedab32592b2d979f3cc922ffd4ed406427de3dda

Unversion some symbols for the NDK.

These functions were erroneously released in LIBC_PRIVATE for M, but
in fact need to be public. Since we need to be able to load them on M
even if they were built for a lower platform (and M needs to load on
newer platforms), we need to unversion them.

Change-Id: I333fe8ae7380cc2a5dbd699414399ec52f602383

Merge "More stdio one-liners."

More stdio one-liners.

This actually turns up a bug in fmemopen, so I guess that's what I'll
look at next...

Change-Id: I2971ecd1b5a3a3c7f43c22d985f88e389af89e97

Merge changes I2d334db2,Iab9fbc73

* changes:
  Clean up 32-bit x86 memset assembler.
  Fix x86 __memset_chk.

Correct libstdc++ version

The version script is introduced in O

Bug: http://b/28987948
Change-Id: Icd0a27d1b67e8227e44aa6e771816f0e6ed489b8

Merge "Update testing docs."

Merge "Sync linker with internal branch"

Clean up 32-bit x86 memset assembler.

Change-Id: I2d334db29226f9c60cf20220a569f1b772bf7c66

Update testing docs.

Change-Id: Ic0e1c3105d1cf8e73c0fbfd51052895beed76227

Merge "More stdio cleanup."

Sync linker with internal branch

This change includes dlwarning implementation and
the compatibility greylist for apps targeting pre-N.

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

Merge "Restore attribute alloc_size for GCC."

Fix x86 __memset_chk.

When __memset_chk_fail moved to C++, we didn't take into account
that the stack wasn't set up correctly for the jump from assembler.
Presumably no one had run the 32-bit x86 tests until now!

Bug: http://b/30513094
Change-Id: Iab9fbc731b6bfecd7ca4d3f3844d0568a5d30d86

Restore attribute alloc_size for GCC.

clang doesn't support attribute alloc_size, but GCC uses it to give
diagnostics where clang can't, and we had a test for one such case.

Bug: http://b/30513330
Change-Id: I4c14771258992d6bea684db7f91d60312642662f

Merge changes I946cde38,I4b8e0ea1,I2e8141a8

* changes:
  Mark LIBC_O as being a future version.
  Mark LIBC_N as being introduced in android-24.
  Annotate the libc version map with NDK info.

Mark LIBC_O as being a future version.

None of these symbols are in the NDK yet. Skip them.

Change-Id: I946cde38e767190c396593d562babc90457973fb

Mark LIBC_N as being introduced in android-24.

Change-Id: I4b8e0ea1fc9b42c1f98f36ae7aaaae2c6322e16b

Annotate the libc version map with NDK info.

Generated by running
https://android-review.googlesource.com/#/c/251477/ for libc.

Test: make checkbuild with
https://android-review.googlesource.com/#/q/branch:master+topic:ndk_library-libc-libm
Change-Id: I2e8141a83c1d79fd89b5e5c4432400b8301ba347

More stdio cleanup.

Time to get back to cleaning up stdio, so start with a bunch of easy
one-liners...

Change-Id: I8df5fdc72500a89b977bfaa6c64c3639198d4e3e

Merge "Improvements to initialization of linker_allocator"

Merge "Upgrade to tzcode-2016f."

Improvements to initialization of linker_allocator

Make linker_allocator independent of the order of global constructors.

Bug: http://b/30483811
Change-Id: I18a323175661b8e1c9e398f2d6112f7a08d2cceb
Test: boot the device with I3ac91758a1a043146c65f2ae0f36fcfbe372c30f

Upgrade to tzcode-2016f.

No significant changes other than a default implementation of strftime_l.

Change-Id: I6edd2c03e5e7559f012c0c87d43f7109d641d3ca

Merge "Add symbol versions to libstdc++.so"

Add symbol versions to libstdc++.so

Bug: http://b/28987948
Change-Id: I3853528bd432ceab37e4867f2c4112d9ee0b40dc

Merge "Add IOV_MAX to <limits.h>."

Merge "Admit that we don't actually support thread priority inheritance."

Add IOV_MAX to <limits.h>.

strace 4.13 assumes we have IOV_MAX. POSIX says it lives here. 1024 is
a Linux kernel limitation.

Change-Id: Icb428172670e4936a459db38a6a190b7178fe9f3

Admit that we don't actually support thread priority inheritance.

Bug: https://code.google.com/p/android/issues/detail?id=218323
Change-Id: I6c69b0fbb8414e7cc2deeb8be50b6af4830ed55e

Merge "Clean up <uchar.h> and <wctype.h> transitive includes."

Merge "More kernel header cleanup."

Clean up <uchar.h> and <wctype.h> transitive includes.

Bug: http://b/30367366
Change-Id: I63212a34c54ca07223a15aa5be4da3bdc6aa38a8

Merge changes I0262eed9,Iba9ee03a

* changes:
  Add a temporary hack to work around __NDK_FPABI__.
  Remove _GNU_SOURCE guard on tmpnam.

Add a temporary hack to work around __NDK_FPABI__.

Test: make checkbuild
Bug: http://b/27533932
Change-Id: I0262eed91b10b0dd5fe27a61fb8f35b3c062280e

Remove _GNU_SOURCE guard on tmpnam.

This shouldn't be guarded by _GNU_SOURCE. external/opencv3 expects
P_tmpdir to be available.

http://pubs.opengroup.org/onlinepubs/009695399/functions/tmpnam.html

Test: make checkbuild
Change-Id: Iba9ee03aec6e712356b571f7165ed1809e6227fe

Merge "Remove #pragma GCC visibility."

Remove #pragma GCC visibility.

We have better control over visibility now.

Bug: http://b/24767418
Change-Id: Ifa9a1e5af4bd4217db08422050908770247b3b69

Merge "Stop #define'ing __func__ and __restrict."

Stop #define'ing __func__ and __restrict.

__STDC_VERSION__ isn't defined for __cplusplus, so we've been removing
such checks. Some got missed.

Stop defining __func__ and just use the __PRETTY_FUNCTION__ GCC extension
in <assert.h>. Also fix the #if there so that C++ gets __assert2 rather
than __assert, and rewrite the cast to work with -I rather than -isystem.

Also remove __restrict and just always use the __restrict GCC extension.

Add a trivial test for <assert.h>.

Bug: http://b/30353757
Change-Id: Ie49bb417976293d3a9692b516e28fe3c0ae0a6d9
Test: ran bionic unit tests.

Merge "Remove bionic's now-unused UniquePtr."

Merge "Remove unnecessary 'extern's."

Remove bionic's now-unused UniquePtr.

Change-Id: Id77dbdd6d4255ae8317449725ef093be9422407f

Remove unnecessary 'extern's.

Change-Id: Iba2b3fb6ff88e504f1657b915120ae43d58a1e03

Merge "Always include <sys/cdefs.h>."

Always include <sys/cdefs.h>.

I made a mistake caught by code review earlier, so let's try to be
safer by default.

This patch also moves all our "forwarding" headers to the guardless
just-include-the-other-thing style that we usually use. (Where we
have a comment explaining where the header comes from, I've kept
that.)

Change-Id: I37342cf5e2563c6a269b2ba61a697069b1c7913b

Merge "Remove __BIONIC_LEGACY_INLINE."

Remove __BIONIC_LEGACY_INLINE.

Don't use the same declaration to declare both static and non-static
functions, to make life easier for versioner.

(Also, remove __BIONIC_LEGACY_INLINE from two functions in termios.h
that weren't actually legacy inlines.)

Bug: http://b/30170081
Change-Id: Ibb73377d77a2b4cee016289b7c46a01452e45fae

Merge "Add sanity checks for e_shentsize and e_shstrndx"

Merge "Make it possible to build bionic with -I instead of -isystem."

Make it possible to build bionic with -I instead of -isystem.

Various things:

* work around -Wnullability-completeness.
* use C++ casts in C++ and C casts in C.
* stop using attributes clang doesn't support (such as `warning`).
* remove duplicate definitions of XATTR_CREATE and XATTR_REPLACE.

Change-Id: I07649e46275b28a23ca477deea119fe843999533

Add sanity checks for e_shentsize and e_shstrndx

This helps us avoid situations when malformed elf-files
are mistakenly loaded, - which might result in unexpected
behavior.

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

More kernel header cleanup.

Modify the generate script to add a new option and copy the types.h
arm header file since it's not currently being created properly. Also
manually generate the arm types.h uapi header since it's not being
properly generated right now.

Modify both generate scripts to delete the target directories before
adding the files to handle moved/deleted header files.

Move the common/scsi headers into android and delete the common
directory. Change the scripts to reflect this change.

Update the scsi headers since they've been modified in upstream kernels.

Bug: 30072483

Change-Id: Ia43d4b238b6a041350d60cc30184ecbd4829d7d5

Merge changes I7718b1ba,I648adc35

* changes:
  Only initialize the global stack protector once.
  Make getpid work before TLS has been initialized.

Merge "Only define sa_family_t once."

Only define sa_family_t once.

Change-Id: Icf700910c836d22c8a27ad026ec8af11697edd5a

Merge "Treat "persist.sys.timezone" as being a Java-format timezone."

Treat "persist.sys.timezone" as being a Java-format timezone.

POSIX and Java disagree about the sign in a timezone string like "GMT+3".
This means that if you set "persist.sys.timezone" to such a string, native
code and Java code disagree about what time it is. Resolve this by
translating to POSIX form for native code.

Bug: http://b/25463955
Change-Id: I7c08c459dd8514348a12ce419edcbfbfe8f6d327
Test: manually tested with setprop and date.

Merge "Pass argc/argv/envp to dso constructors"

Pass argc/argv/envp to dso constructors

The glibc implementation of the loader passes argc/argv/envp
to all elf constructors. This change makes bionic linker
behave in the same way.

Bug: http://b/30145768
Change-Id: I1c65c42aa5305a5b133c64b5748205bbde869e0e
Test: run bionic-unit-tests --gtest_filter=dl*:Dl*