Merge "libc fortify: make stdio.h use diagnose_if" am: 6e0de9da49
am: ced73cda27

Change-Id: I144dd0b3ad3a27394592e3ee391fbc792450a9f6

Merge "libc fortify: make stdio.h use diagnose_if"
am: 6e0de9da49

Change-Id: Ic847d12c7822f7f8e25ec5a40e6cf7143417dad9

Merge "libc fortify: make stdio.h use diagnose_if"

Merge "libc fortify: make poll.h and stat.h use diagnose_if" am: cff14d9e3e
am: f5b6f13394

Change-Id: I55e54e5aee89e9deb7c01f0f43fdfefe33217c45

Merge "libc fortify: make poll.h and stat.h use diagnose_if"
am: cff14d9e3e

Change-Id: I4cd2863d8f9d859e8441da7cc54f2fc15294b42e

Merge "libc fortify: make poll.h and stat.h use diagnose_if"

libc fortify: make stdio.h use diagnose_if

This also throws in some reformatting to match the GCC FORTIFY style in
this file (e.g. 4 space standard indent).

Per b/36984245, some function protos are unwrapped beyond 100 chars, as
well.

Bug: 12231437
Test: m checkbuild on bionic internal master; CtsBionicTestCases show
no new failures.

Change-Id: I6dce60ec14d5cd703d9e724699634a197ca11023

libc fortify: make poll.h and stat.h use diagnose_if

Bug: 12231437
Test: m checkbuild on bionic internal master; CtsBionicTestCases show
no new failures.

Change-Id: I7dbdeca12c46d1ee3804f13546b11c3f3b3a7596

Merge "Mark new/delete as weak in the NDK stubs." am: aa8f766bf0
am: f781e14912

Change-Id: Ic5a36db8cc08af92d7bcc766e5d41dbaf2a22a42

Merge "libc fortify: make unistd.h use diagnose_if" am: 0ec52ce15b
am: a28402b431

Change-Id: I2732c95e39ebb3236d30cc777b2de16a0bbb46b0

Merge "Mark new/delete as weak in the NDK stubs."
am: aa8f766bf0

Change-Id: I1033bc0a963dd03476a77c0a54915b9c4353a5aa

Merge "libc fortify: make unistd.h use diagnose_if"
am: 0ec52ce15b

Change-Id: I1361011fc3f466f681d651f85e8e828228c77b41

Merge "Remove nullability specifications." am: fe3d58f49f
am: 0fa654aef5

Change-Id: I8a05d8dcb9f5c48eab215fa634f2076be370d9a5

Merge "Mark new/delete as weak in the NDK stubs."

Merge "libc fortify: make unistd.h use diagnose_if"

Merge "Remove nullability specifications."
am: fe3d58f49f

Change-Id: Id4fff7a1ba3f1ed76ae77ec42fdd0ca664dceeff

Merge "Remove nullability specifications."

Merge "Clean up linker fatals slightly." am: 1bac61f09f
am: f54872e5bf

Change-Id: Icc88209333d75289dbcd42cfa40539b0ff5ed7d9

Merge "Clean up linker fatals slightly."
am: 1bac61f09f

Change-Id: Icf7d5b5b0def92ddb60c5c8a5c15e8867e76b2ad

libc fortify: make unistd.h use diagnose_if

Also:
- remove a NULL check. The bug that's been working around has been
  fixed since clang r298431, and our compiler is built from r300080, and
- unwrap some function prototypes, per b/36984245.

If you're interested, here's what a new error versus an old one looks
like: https://drive.google.com/file/d/0B-8OBPboNJ_4MVdlYXdJbUFBYUk/view

Bug: 12231437
Test: m checkbuild on bullhead internal master; CtsBionicTestCases show
no new failures.

Change-Id: Ibafe61198988b4c8bee8391bcdf01bcbc04233de

Merge "Clean up linker fatals slightly."

Remove nullability specifications.

Bug: http://b/64251432
Test: builds
Change-Id: I5b1613484783f7478d30b5e694007f77fa626659

Merge "Consistently say va_arg rather than __va_arg." am: 3e523a6562
am: d02c95a2ca

Change-Id: Ia835f3c96867817a05789ae9f68bd1e37d461bfe

Merge "Consistently say va_arg rather than __va_arg."
am: 3e523a6562

Change-Id: I3a01c16fdf736a5df9b9be4a6ba58706aadb9f6d

Merge "Consistently say va_arg rather than __va_arg."

Merge "Clean up some <stdio.h> constants." am: 7f86f1dd15
am: 84136f4be7

Change-Id: I9b12b7be23eac57adccfedec7d4490d1c9134156

Merge "Clean up some <stdio.h> constants."
am: 7f86f1dd15

Change-Id: I917e76ed6786ce08fac63f887e84c04b20cd02b8

Merge "Clean up some <stdio.h> constants."

Merge "Remove restrict/__restrict." am: 6616056f0f
am: b3f1343098

Change-Id: Ieefde6091e565fda3f48605683912395cc0d0f3f

Merge "Remove restrict/__restrict."
am: 6616056f0f

Change-Id: I5343c191c3f424343d1c7617e054b2347d0dc6b5

Consistently say va_arg rather than __va_arg.

Bug: http://b/64250392
Test: builds
Change-Id: I435242f620245d825b8a68365c71c99a60aba625

Merge "Remove restrict/__restrict."

Clean up linker fatals slightly.

Use __linker_cannot_link more consistently, and fix a comment.

Bug: http://b/22798163
Test: ran tests
Change-Id: Id6d868f459997eaa67b8cbbf85b0b0f2749bf43d

Remove restrict/__restrict.

We've never really used __restrict: only <string.h> and <stdio.h> (which
are still very similar to upstream BSD headers) consistently have these
annotations. Neither clang nor GCC warns for trivial cases, and there's
little obvious documentation benefit.

Bug: http://b/30833514
Test: builds
Change-Id: I3e4384281865475d0c55d764b546d8166419ee31

Merge "libc: Add diagnose_if macros to cdefs." am: c2a10f7f1b
am: 8bfe9f81ba

Change-Id: I41df96251e2d85666b61429e44ff6dbd89109272

Merge "libc: Add diagnose_if macros to cdefs."
am: c2a10f7f1b

Change-Id: If9203f290902beb68cdc8d4986f74ba30dabd088

Merge "libc: Add diagnose_if macros to cdefs."

Merge "libc: Split FORTIFY into its own headers" am: 1db06faa38
am: f51724ab6b

Change-Id: I952061459221251b35bfd421cf120b2e335aac05

Merge "libc: Split FORTIFY into its own headers"
am: 1db06faa38

Change-Id: I387fb3d28c9d3f827f0e0011718e72efb0949db7

Merge "libc: Split FORTIFY into its own headers"

Clean up some <stdio.h> constants.

And actually test an assertion rather than just state it in a comment.

Bug: N/A
Test: ran tests
Change-Id: I07699483aca4aac4e089d8b99123cb5bde9b3c63

libc: Add diagnose_if macros to cdefs.

I plan on having one review per file for the enable_if->diagnose_if
FORTIFY migration. Having this in means that no one review is dependent
on another.

Bug: 12231437
Test: m.
Change-Id: Ic0b07d7f7f6782e371c8792eb1e40cdfa32d3e35

Merge "Allow calling v?dprintf for any target API level." am: 4dd33f92d8
am: b4bf27218b

Change-Id: I8028014fc369511b6c35736c70f9014c9cce6cf4

Merge "Allow calling v?dprintf for any target API level."
am: 4dd33f92d8

Change-Id: I740b3424c4e440350d63c3b7056c6e21acd77e60

Merge "Allow calling v?dprintf for any target API level."

Merge "Move back to the previous version of ion.h." am: f94ddd7a0e
am: c0bafec3d8

Change-Id: I6fe966d1a91863ee3deadd4ad2a1f5c789ed3149

Merge "Move back to the previous version of ion.h."
am: f94ddd7a0e

Change-Id: I7a782bf91587547dfd4e2137dbdb529d1cd80040

Merge "Move back to the previous version of ion.h."

Move back to the previous version of ion.h.

The new kernel headers changed this and lots of parts of the tree
want the old definition, so restore the previous version.

Test: Builds.
Change-Id: I725e56bb7c51c9fcd23ebc4b67eac33d66e196e7

Merge "Remove workaround in <sys/epoll.h>." am: 6c26e9ec6f
am: 578763f75b

Change-Id: I1abcc88a894d88efdedcfc464bce0c296f7b50cf

Merge "Update to kernel headers v4.12.3." am: c478c32d2f
am: ff5803059e

Change-Id: I1dcd64e016569623a14665cf633a71db8017118f

Merge "Remove workaround in <sys/epoll.h>."
am: 6c26e9ec6f

Change-Id: Ib77bb0a43f800061638e71bd572445820f4369b1

Merge "Update to kernel headers v4.12.3."
am: c478c32d2f

Change-Id: If521c86d7895959f49e6933be0a8e9959105ad0c

Merge "Remove workaround in <sys/epoll.h>."

Merge "Update to kernel headers v4.12.3."

Merge "versioner: compile with `clang -include foo.h -`." am: b1bbf2bb24
am: 336f2b105e

Change-Id: I73bc160783674b7eefdd7b518245824b47ab23f4

Merge "versioner: compile with `clang -include foo.h -`."
am: b1bbf2bb24

Change-Id: Ia5cd7d937435a147b152b82433e483ab0bea3b57

Merge "versioner: compile with `clang -include foo.h -`."

Merge "POSIX getlogin_r/LOGIN_NAME_MAX/TTY_NAME_MAX." am: 8aa4bc7c55
am: b82a2d0bd5

Change-Id: I7ed5cd170a4b3d937ab035a08c693dc117e104c6

Merge "POSIX getlogin_r/LOGIN_NAME_MAX/TTY_NAME_MAX."
am: 8aa4bc7c55

Change-Id: I578d3e5c096401b0447f9ed7223d3e20ffb5b19e

Merge "POSIX getlogin_r/LOGIN_NAME_MAX/TTY_NAME_MAX."

Merge "Fix the sysconf _SC_2_VERSION test." am: 3e183a5d30
am: 105e768638

Change-Id: I815b4c7c6d72cd577f3f5f69ed6e36108e50dd3f

Merge "Fix the sysconf _SC_2_VERSION test."
am: 3e183a5d30

Change-Id: I384fc20676715a1385a182428e5bbd17d4e3d8e8

Merge "Fix the sysconf _SC_2_VERSION test."

Fix the sysconf _SC_2_VERSION test.

Not sure how this ended up broken. Too many branches!

Bug: N/A
Test: ran tests
Change-Id: Ieef1e8170c9aaf139d9e0664475ccb08b8fcdeba

POSIX getlogin_r/LOGIN_NAME_MAX/TTY_NAME_MAX.

Bug: N/A
Test: ran tests
Change-Id: I97c8ecc82fb09c3fa4bb5e6f000d46f04f3d1702

Merge "Implement wcwidth(3) in terms of icu4c." am: bafa1985ec
am: a97079d94e

Change-Id: I173f75c4080a61bce178ded9b2a13a1a1ba63dfc

Merge "Implement wcwidth(3) in terms of icu4c."
am: bafa1985ec

Change-Id: Ia7e74b6124206fe5f2c5bf372c03ee5246045462

Merge "Implement wcwidth(3) in terms of icu4c."

Merge "Actually turn on -Werror for crtbegin." am: 07d2d327bf
am: 249792a561

Change-Id: I314f9d32c69c2afb66c1edd94afc196bc4f8373c

Merge "Actually turn on -Werror for crtbegin."
am: 07d2d327bf

Change-Id: Ia9be400c31c9815e2753bdbd30ef3f224dfda58f

Merge "Actually turn on -Werror for crtbegin."

versioner: compile with `clang -include foo.h -`.

At least one warning only triggers in files that are included, instead
of being passed directly. Switch to compiling with -include, and fix
the resulting warnings.

Bug: https://github.com/android-ndk/ndk/issues/474
Test: mma -j && versioner
Test: python tools/versioner/run_tests.py
Change-Id: I784698c18540c9cc30f372f279a1cec1d75721ea

Mark new/delete as weak in the NDK stubs.

Test: make ndk
Bug: None
Change-Id: I76eef4f0265e87ddbad84ac8c5d56225aa240239

Implement wcwidth(3) in terms of icu4c.

Based on com.google.i18n.CharWidth by the icu-team folks.

Bug: N/A
Test: ran tests
Change-Id: Ia9d818ec4ae60f2f3978533195330d00699397c5

libc: Split FORTIFY into its own headers

This patch cleans up our standard headers by moving most of the FORTIFY
cruft out in to its own sandbox. In order to include the *_chk and
*_real declarations, you can either enable FORTIFY, or `#define
__BIONIC_DECLARE_FORTIFY_HELPERS`.

Both sys/select.h and strings.h are explicitly ignored by this patch.
Both of these files have very small __BIONIC_FORTIFY blocks, and don't
define any actual FORTIFY'ed functions (just macros, and 3 *_chk
functions).

This patch also makes the versioner ignore the FORTIFY implementation
headers, since we're guaranteed to pick the FORTIFY'ed headers up when
looking at the regular headers. (...Not to mention that making the
FORTIFY'ed headers freestanding would be annoying to do and maintain for
~no benefit).

We bake the knowledge of where FORTIFY headers live directly into the
versioner. We could go with a more general approach (e.g. adding an -X
IGNORED_FILE flag that tells the versioner to ignore
$HEADER_PATH/$IGNORED_FILE), but we'd then have to repeat that for every
test, every manual invocation of the versioner, etc. for no benefit
that's obvious to me.

Bug: 12231437
Test: m checkbuild on bullhead internal master + CtsBionicTestCases. no
new errors.

Change-Id: Iffc0cc609009b33d989cdaddde0a809282131a5b

Actually turn on -Werror for crtbegin.

And then -Wno-gcc-compat in the right place this time. (But leaving the
other one because I don't care in the non-crtbegin code either.)

Bug: N/A
Test: builds without warnings now
Change-Id: Id2cfa3392ee80c045644ebfdf92c6d341db90964

Allow calling v?dprintf for any target API level.

Old versions of Android called these fdprintf and vfdprintf out of
fears that the glibc names would collide with user debug printfs.

Allow users to just use dprintf and vfdprintf on any version by
renaming those calls to their legacy equivalents if needed.

Test: built trivial NDK module targeting android-14 and using dprintf
Test: make checkbuild
Bug: https://github.com/android-ndk/ndk/issues/72
Change-Id: I90de149278f931380418536abaef47c5cee5c195

Update to kernel headers v4.12.3.

Test: Built angler.
Change-Id: Icbcf4fac2334de8409b049ed7a3b4c24b4e98ce9

Remove workaround in <sys/epoll.h>.

Linux 4.12 has a better fix: https://lkml.org/lkml/2017/2/23/416 and
https://lkml.org/lkml/2017/2/23/417.

Bug: N/A
Test: builds
Change-Id: I9cbf0c60817332142c48297899062f1a25f2b082

Merge "Use O_APPEND for stdio append mode." am: 38bcf2b704
am: 72f06ce7fe

Change-Id: I66d66f210091334c6579ecdecea6944d8dab4593

Merge "Use O_APPEND for stdio append mode."
am: 38bcf2b704

Change-Id: I4e1274d93adafffc6c4795cbcb5755735533c4de

Merge "Use O_APPEND for stdio append mode."

Merge "Fix error condition on strtof inline." am: 9faa95c9eb
am: e3b1101d62

Change-Id: I8b4d9b01e641ae0e8c9dceafd835bcadd610b506

Merge "Fix error condition on strtof inline."
am: 9faa95c9eb

Change-Id: Ic09c01e387ad5cf701368972aee6673e78822603

Merge "Fix error condition on strtof inline."

Merge "Add __ANDROID_NDK__ to identify using the NDK." am: d3e656abeb
am: c441ad73fe

Change-Id: Ie762fb6c9c0d59acdbaebdfd0d6f36f7294f1153

Fix error condition on strtof inline.

Test: make checkbuild
Test: pull into ndk, ./run_tests.py
Bug: https://github.com/android-ndk/ndk/issues/415
Change-Id: I1575f799c9e849d66456c785208a36709a7b8122

Merge "Add __ANDROID_NDK__ to identify using the NDK."
am: d3e656abeb

Change-Id: I8c590998e93937c6702bcb91b9dcadb509f341ca

Merge "Add __ANDROID_NDK__ to identify using the NDK."

Add __ANDROID_NDK__ to identify using the NDK.

I've added some things like __ANDROID_MAJOR__ to an ndk-version.h, but
that is only in the NDK itself and so doesn't help the platform. Add
__ANDROID_NDK__ to identify that you're building for the NDK and not
the platform.

Test: make checkbuild
Bug: https://github.com/android-ndk/ndk/issues/407
Change-Id: I2d1f1c28e3764e4e658cf675b290b7a17253ee33

Merge "libc: Move FORTIFY into one file; make style fixups" am: 4f36cabd8d
am: 78d878c8df

Change-Id: I0b0c74d9d3862a5e3267f3485e34bbcb0d6d9a74

Merge "libc: Move FORTIFY into one file; make style fixups"
am: 4f36cabd8d

Change-Id: I17f125e6100e0af30c083df7a6a7a19a90696dc3

Merge "libc: Move FORTIFY into one file; make style fixups"

Use O_APPEND for stdio append mode.

"Although not explicitly required by this volume of POSIX.1-2008, a good
implementation of append (a) mode would cause the O_APPEND flag to be set."

Yeah, about that...

Bug: N/A
Test: ran tests
Change-Id: I23c4bc5c1ebc92e0cb44025d2d313f321f9ffa68

libc: Move FORTIFY into one file; make style fixups

This addresses post-commit feedback from
I88c39ca166bacde0b692aa3063e743bb046a5d2f. With this, our FORTIFY impl
now sits in one file.

Bug: 12231437
Test: mma; no new CtsBionicTestCases failures on bullhead internal
master.
Change-Id: I6f9ff81c3e86cf9d6a0efa650eb5765f1e2fa09c

Merge "Build crtbegin with clang too." am: 9306889509
am: da32d3349b

Change-Id: I7523c4b638f07361038ebb0fcd22b7fc0ee8d950

Merge "Build crtbegin with clang too."
am: 9306889509

Change-Id: Ic36b2f3a9c25fe2567917cc6f9bbca9c104c189a

Merge "Build crtbegin with clang too."

Merge "Create global seccomp policy." am: 33f9043f48
am: 383f77b01a

Change-Id: I8b4dea5dd1941eab2fd1c504cf79bf8856522228