Merge "Add check for pthread_self() when looking up a thread" am: b90837c3d3
am: 7e694a20fd

* commit '7e694a20fd4136f8aca9afa294dae663a49efacc':
  Add check for pthread_self() when looking up a thread

Merge "Add check for pthread_self() when looking up a thread"
am: b90837c3d3

* commit 'b90837c3d316c6ea8d6f0b0b6a3643d0d3245a0e':
  Add check for pthread_self() when looking up a thread

Merge "Add check for pthread_self() when looking up a thread"

Add check for pthread_self() when looking up a thread

Check if thread_id is in fact pthread_self before
locking on g_thread_list_lock in __pthread_internal_find.

The main reason for doing this is not performance but to allow
the linker use raise() which was not working because pthread_kill()
couldn't find pthread_self() thread because the global thread
list is initialized in libc.so and the linker's version of this
list is empty.

Bug: http://b/25867917
Change-Id: I18fe620e8cd465b30f0e1ff45fff32958f3c5c00

Merge "Add in6addr_any and in6addr_loopback symbols." am: d2be380258
am: 316eb69ad4

* commit '316eb69ad4c5e89c3a0f057e79d7e9342609a010':
  Add in6addr_any and in6addr_loopback symbols.

Merge "Add in6addr_any and in6addr_loopback symbols."
am: d2be380258

* commit 'd2be3802582e69bfaf1a8ab02849bc44c4fa2ce6':
  Add in6addr_any and in6addr_loopback symbols.

Merge "Add in6addr_any and in6addr_loopback symbols."

Add in6addr_any and in6addr_loopback symbols.

Change-Id: I3a90fa448b6bd43321672ba74b84a4e9e8a67738

Merge "Really add adjtimex(2), and add clock_adjtime(2) too." am: 5da33dd673
am: d1cfd67c6b

* commit 'd1cfd67c6b92767c959a4ed9a00126e1c7ecbc19':
  Really add adjtimex(2), and add clock_adjtime(2) too.

Merge "Really add adjtimex(2), and add clock_adjtime(2) too."
am: 5da33dd673

* commit '5da33dd67350fcb7c12a57027b1af5f4e72a9c21':
  Really add adjtimex(2), and add clock_adjtime(2) too.

Merge "linker: Change the library search order" am: c92ffa95fc
am: 10e4fd56b5

* commit '10e4fd56b5e86ca1a9a6bbcb4b62f6f4592bf508':
  linker: Change the library search order

Merge "Really add adjtimex(2), and add clock_adjtime(2) too."

Merge "linker: Change the library search order"
am: c92ffa95fc

* commit 'c92ffa95fc2f9ef4d381033d313797cffe14709e':
  linker: Change the library search order

Merge "linker: Change the library search order"

Really add adjtimex(2), and add clock_adjtime(2) too.

Change-Id: I81fde2ec9fdf787bb19a784ad13df92d33a4f852

Merge "Add fopen64/freopen64/tmpfile64 aliases." am: 36ff699570
am: e8e8873d9f

* commit 'e8e8873d9f1d277ede6c9156f5bd78ef9eee3f1e':
  Add fopen64/freopen64/tmpfile64 aliases.

Merge "Add fopen64/freopen64/tmpfile64 aliases."
am: 36ff699570

* commit '36ff6995707112e66dd0dec78be7f8ee9b95d321':
  Add fopen64/freopen64/tmpfile64 aliases.

Merge "Add fopen64/freopen64/tmpfile64 aliases."

Add fopen64/freopen64/tmpfile64 aliases.

Our fopen/freopen/tmpfile are already always O_LARGEFILE, but let's add
the aliases for _LARGEFILE_SOURCE compatibility.

Bug: http://b/24807045
Change-Id: I5d99b3ef3c9f27ce70f13313f6a92e96c7f21f80

Merge "Fix ifaddrs#getifaddrs_INET flakiness." am: a279324094
am: 02075d2100

* commit '02075d2100686edd059efe0ac6dc8f3eb7f439bd':
  Fix ifaddrs#getifaddrs_INET flakiness.

Merge "Fix ifaddrs#getifaddrs_INET flakiness."
am: a279324094

* commit 'a279324094d748b97201412b2457d2f6e26287dd':
  Fix ifaddrs#getifaddrs_INET flakiness.

Merge "Fix ifaddrs#getifaddrs_INET flakiness."

Merge "Expose ftello64." am: b3cfafd2bd
am: e74d1d6507

* commit 'e74d1d6507a28c56941d95889219e1ea24b0ac5c':
  Expose ftello64.

Merge "Expose ftello64."
am: b3cfafd2bd

* commit 'b3cfafd2bd3aa307457fe89573d6ac232a5de1c6':
  Expose ftello64.

Merge "Expose ftello64."

Fix ifaddrs#getifaddrs_INET flakiness.

The interface name wasn't being nul-terminated for the ioctl. Also clean up
the code a bit to give more useful diagnostics on failure.

Bug: http://b/26887941
Change-Id: I30c6bdc1a32733971a27ed1fb7db9d8239b6262b

Expose ftello64.

Also add the missing test that would have caught this.

Bug: http://b/24807045
Change-Id: I756a4d825595b52396b15898e3f717156fd1ba2f

Merge "Fix our fortify tests for global -Werror." am: 3e75110bcb
am: 9fbe4a0482

* commit '9fbe4a048221b7823104265ba410e130263867a9':
  Fix our fortify tests for global -Werror.

Merge "Fix our fortify tests for global -Werror."
am: 3e75110bcb

* commit '3e75110bcb1d7b941ab0633e7e5c36c604512ded':
  Fix our fortify tests for global -Werror.

Merge "Fix our fortify tests for global -Werror."

Merge "use GNU C89 extern inline semantics for __size_mul_overflow()" am: 153e9928b8
am: 8e8f990fc4

* commit '8e8f990fc4fde79c2d10a5c40341d2ce90eb4e95':
  use GNU C89 extern inline semantics for __size_mul_overflow()

Merge "use GNU C89 extern inline semantics for __size_mul_overflow()"
am: 153e9928b8

* commit '153e9928b8bab5e11d4bf4679bf307b81a88108e':
  use GNU C89 extern inline semantics for __size_mul_overflow()

Merge "use GNU C89 extern inline semantics for __size_mul_overflow()"

use GNU C89 extern inline semantics for __size_mul_overflow()

__size_mul_overflow generates warning under following compilation envrionment:

-OX -D_FORTIFY_SOURCE=2 (X=1, 2, 3)

For example:

echo '#include <stdio.h>' | \
prebuilts/gcc/linux-x86/arm/arm-linux-androideabi-4.9/bin/arm-linux-androideabi-gcc \
    -I bionic/libc/arch-arm/include \
    -I bionic/libc/include \
    -I bionic/libc/kernel/uapi \
    -I bionic/libc/kernel/common \
    -I bionic/libc/kernel/uapi/asm-arm \
    -I bionic/libm/include \
    -I bionic/libm/include/arm \
    -I bionic/libc/include \
    -Werror \
    -O1 \
    -D_FORTIFY_SOURCE=2 \
    -c \
    -x c \
    -

bionic/libc/include/stdio.h:360:13: error: '__size_mul_overflow' is
static but used in inline function 'fread' which is not static [-Werror]
         if (__size_mul_overflow(size, count, &total)) {
                          ^

C99 - 6.7.4
"An inline definition of a function with external linkage shall not contain
a definition of a modifiable object with static storage duration, and shall
not contain a reference to an identifier with internal linkage."

As standard does not require compiler to determine when it is safe to
reference an internal function in an external inline function, but instead
delegalizes such reference as a whole, gcc throws a warning for such code
under C99 compilation.  Warning produced by libc header is inhereted widely
and strips the option of using -Werror to track code sanity.

Replace static inline specifier with gnu89 extern inline. Latter "is used
only for inlining. In no case is the function compiled on its own", which
is slightly different from former semantically, but should produce the same
result here.

Change-Id: I6a3374498e5499d110e54468cf9d0d67d2debbe2

Fix our fortify tests for global -Werror.

If anyone has a better way of doing this it would be welcome...

Bug: http://b/26936282
Change-Id: I796896866327b4b5b5ee4ec36994fb320993f85d

Merge "Remove the unused TrackHeader structure." am: fe60bae4b7
am: 15c35a02da

* commit '15c35a02daf0a1f8b2c47ac66fde0fd267c33fc1':
  Remove the unused TrackHeader structure.

Merge "Remove the unused TrackHeader structure."
am: fe60bae4b7

* commit 'fe60bae4b71f9b4d50912f06acac601762e55b73':
  Remove the unused TrackHeader structure.

Merge "Remove the unused TrackHeader structure."

Remove the unused TrackHeader structure.

Change-Id: I07ae5de717e1fcfbe5dfe552a531db9f630da622

Merge "Add better free tracking." am: 4821db1b3e
am: e98d704731

* commit 'e98d704731dd685d495025ff7cfd751c6e672668':
  Add better free tracking.

Merge "Add better free tracking."
am: 4821db1b3e

* commit '4821db1b3ecc14828d4acf4abed26bb7d8b8cf6a':
  Add better free tracking.

Merge "Add better free tracking."

Merge changes I8ca219fa,Ia92d35b1 am: 5992e5fb6d
am: 33c8164b63

* commit '33c8164b6321d80550b5ec7fbb5f7b49c4ddd954':
  socket.h: update AF_* constants
  Add adjtimex

Merge changes I8ca219fa,Ia92d35b1
am: 5992e5fb6d

* commit '5992e5fb6da2722fb4c3850048f71ef077735381':
  socket.h: update AF_* constants
  Add adjtimex

Merge changes I8ca219fa,Ia92d35b1

* changes:
  socket.h: update AF_* constants
  Add adjtimex

linker: Change the library search order

Look into "/system/lib" before "/vendor/lib", this way
we can ensure that in case of name conflict /system libraries
take precedence.

Change-Id: Ifacdc9dc7a4e175853f499ed09782b2257f09e5a

socket.h: update AF_* constants

Change-Id: I8ca219fa429ca8d8f872ed1db98275cd1a06f00b
Signed-off-by: Greg Hackmann <ghackmann@google.com>

linker: check if needed_by is nullptr

Check if needed_by is nullptr in the grey-list warning
also print the name of the library making call to dlopen()

Bug: http://b/26749878
Change-Id: I4e16223a3e3d8d1539f855a0b3f199c5529027e0

Add better free tracking.

Included in this change:

- Change the tag when a pointer is freed so it's easy to detect if
  an already freed pointer is being used.
- Move the free backtrace out of the header. This backtrace is only
  used under only some circumstances, so no need to allocate space
  in all headers for it.
- Add new option free_track_backtrace_num_frames to specify how many
  frames to record when the free occurs. This removes the dependency
  on the backtrace option to get backtraces.

Bug: 26739265
Change-Id: I76f5209507dcf46af67ada162a7cb2bf282116f2

Merge "Tell kati about generated dependencies around crt" am: 37591462fa
am: c91cd8a47f

* commit 'c91cd8a47f14a96e37d9a5054720886bdacf0bd5':
  Tell kati about generated dependencies around crt

Merge "Tell kati about generated dependencies around crt"
am: 37591462fa

* commit '37591462fa76791c2372d789c3532c7cd0ca7a84':
  Tell kati about generated dependencies around crt

Merge "Tell kati about generated dependencies around crt"

Merge "Do not allow System.load() of external libraries"

Merge "Add min/max values for config values." am: 50647711eb
am: bfc044628a

* commit 'bfc044628accf3af2ba696f44b2452ce5c8a17dc':
  Add min/max values for config values.

Merge "Add min/max values for config values."
am: 50647711eb

* commit '50647711ebaf2360aca05cc94a2fdf431c1a153e':
  Add min/max values for config values.

Merge "Add min/max values for config values."

Do not allow System.load() of external libraries

The greylisting mechanism had a bug that allowed
loading any library using absolute path from java code.

It mistakenly assumed they are needed by greylisted
system library when in fact it is dlopen() call from
libnativeloader.

This patch fixes this problem by checking that the library is in
fact dt_needed and it is not a dlopen call.

Bug: http://b/26394120
Bug: http://b/22548808
Change-Id: Ia3d4c19e77b04d09a6c3047d684d0c04bd5c785e

Add adjtimex

Change-Id: Ia92d35b1851e73c9f157a749dba1e98f68309a8d
Signed-off-by: Greg Hackmann <ghackmann@google.com>

Tell kati about generated dependencies around crt

This was a regression since kati has been introduced. Without
this, touching libc/arch-common/bionic/__dso_handle.h doesn't
let ninja rebuild object files.

Bug: 26839129
Change-Id: Ie9ef4559c0d1ff0910a4d250ea36938b291d15db

Merge "linker: mark library .bss pages with anon:.bss" am: 0e915b294b
am: 2c1589f568

* commit '2c1589f568ae56c85a8ae4d3d92e248d659eeee4':
  linker: mark library .bss pages with anon:.bss

Merge "linker: mark library .bss pages with anon:.bss"
am: 0e915b294b

* commit '0e915b294b2b1f0d20cd1b9a4e1721d59ccbfa7a':
  linker: mark library .bss pages with anon:.bss

Merge "linker: mark library .bss pages with anon:.bss"

Add min/max values for config values.

This is to only allow realistic values for some of the options.

Bug: 26739265

Change-Id: I172d5401c27e5d638d80f1c93c678c26c4dae0fc

linker: mark library .bss pages with anon:.bss

Bug: http://b/26778774
Change-Id: Ie2cdd841b7238de50ef389a897d79f915229616b

Merge changes I7dc213e9,I2928d682 am: fd799baf81
am: c22e0ea41d

* commit 'c22e0ea41d5e70130ecc0eb52ceb64c00cb3d3c2':
  Hide '__moddi3' symbol for x86 and mips
  Hide '__isthreaded' for lp64 libc.so

Merge changes I7dc213e9,I2928d682
am: fd799baf81

* commit 'fd799baf81e7642f2d91968ba01c39f097c0f357':
  Hide '__moddi3' symbol for x86 and mips
  Hide '__isthreaded' for lp64 libc.so

Merge changes I7dc213e9,I2928d682

* changes:
  Hide '__moddi3' symbol for x86 and mips
  Hide '__isthreaded' for lp64 libc.so

Hide '__moddi3' symbol for x86 and mips

Bug: http://b/26805673
Change-Id: I7dc213e9f25510a2a74a30e8c43fd7a64cb3dbdb

Hide '__isthreaded' for lp64 libc.so

Bug: http://b/26805673
Change-Id: I2928d68287894be9f9ae7ed1858af512d8c80a3b

Merge "Remove unused file." am: e4daf100b3
am: f788d6d8c9

* commit 'f788d6d8c9f445bca3140998bd394c58b35340b7':
  Remove unused file.

Merge "Remove unused file."
am: e4daf100b3

* commit 'e4daf100b33fb17936b4d61e0ddfe679fcd04123':
  Remove unused file.

Merge "Remove unused file."

Remove unused file.

This file should have been deleted with the malloc debug rewrite, but
popped back into existence due to a merge conflict.

Change-Id: I74e53daaf3febf650b20e3da5329558ac84c5bcd

Merge "Fix a sign extension bug in stdio." am: 172420e6a2
am: 0fb1e6bdac

* commit '0fb1e6bdac04c9cb113232ca95d57f8e96361df0':
  Fix a sign extension bug in stdio.

Merge "Fix a sign extension bug in stdio."
am: 172420e6a2

* commit '172420e6a2c5af527984a3a3a51f60a48b06ae9a':
  Fix a sign extension bug in stdio.

Merge "Fix a sign extension bug in stdio."

Merge "Unify names for pages allocated for small objects" am: eecae0a359
am: cdf8d292f3

* commit 'cdf8d292f3cc12516f808f1a341df1a9194adcbd':
  Unify names for pages allocated for small objects

Merge "Unify names for pages allocated for small objects"
am: eecae0a359

* commit 'eecae0a3598b9f9946b452107c54678da77c7226':
  Unify names for pages allocated for small objects

Merge "Unify names for pages allocated for small objects"

Fix a sign extension bug in stdio.

This also lets us test the EOVERFLOW behavior, which pointed out that the
fgetpos/fsetpos return on failure has always been wrong...

Bug: http://b/24807045
Change-Id: I35273eb07c8c9155af858adb27569983397580b6

Unify names for pages allocated for small objects

Bug: http://b/26776399
Change-Id: Id5b6f143a526cecca56de18ddad8857dfa1dd165

Merge "Replace snprintf() with __libc_format_buffer()" am: 194860a9e6
am: 7a62c10096

* commit '7a62c10096735f8cd0257dc5069a7af69a7f279e':
  Replace snprintf() with __libc_format_buffer()

Merge "Replace snprintf() with __libc_format_buffer()"
am: 194860a9e6

* commit '194860a9e638b1d093857f08b8fad37c9ec0e113':
  Replace snprintf() with __libc_format_buffer()

Merge "Replace snprintf() with __libc_format_buffer()"

Replace snprintf() with __libc_format_buffer()

If snprintf() is called from the linker, it may erroneously return a
null string.  The libc internal __libc_format_buffer() does not have
this problem, so it is now used instead.

Bug: 26756577

Change-Id: I37a97e27f59b3c0a087f54a6603cc3aff7f07522

Merge "Implement funopen64." am: 896224aa7f
am: 61af93ae54

* commit '61af93ae5471b1bf723d51b1a5ab62cc7c7376db':
  Implement funopen64.

Merge "Implement funopen64."
am: 896224aa7f

* commit '896224aa7f307b6c53d8c4dfa0b0e3556b3b9fda':
  Implement funopen64.

Merge "Implement funopen64."

Merge "Fix lockf for LP32." am: a449fafc89
am: 44c4b858ec

* commit '44c4b858ec3892243cc085faf8a96b44bb8a5a9c':
  Fix lockf for LP32.

Merge "Move android_ network functions to LIBC_PRIVATE" am: 3a33b52981
am: d580a8b150

* commit 'd580a8b1505be000c748a380cae9ef4a510a3823':
  Move android_ network functions to LIBC_PRIVATE

Merge "Fix lockf for LP32."
am: a449fafc89

* commit 'a449fafc894c47305b49087d995e821e878e59c1':
  Fix lockf for LP32.

Merge "Move android_ network functions to LIBC_PRIVATE"
am: 3a33b52981

* commit '3a33b5298101e959c83ea60003618d3052d0adfe':
  Move android_ network functions to LIBC_PRIVATE

Merge "Fix lockf for LP32."

Merge "Move android_ network functions to LIBC_PRIVATE"

Implement funopen64.

Bug: http://b/24807045
Change-Id: I161920978161389be34b707cc6ce8e05f760d552

Move android_ network functions to LIBC_PRIVATE

Make following functions platform-private:
 android_getaddrinfofornet
 android_getaddrinfofornetcontext
 android_gethostbyaddrfornet
 android_gethostbynamefornet

Bug: http://b/26794226
Change-Id: I0855b0b82721e4d24e918508931b4d5587fcae7d

Fix lockf for LP32.

Bug: http://b/13077650
Change-Id: Ibbe2d907d23791c908475e0875a5c745ac948813

Merge "Factor out the waiting for children in bionic tests." am: ced73ee45e
am: 644f9d1ea9

* commit '644f9d1ea9af956082046393b0e5fc502d2c2dd3':
  Factor out the waiting for children in bionic tests.

Merge "Support _FILE_OFFSET_BITS=64 for most of <stdio.h>." am: 94bb0fab93
am: 1500a65adb

* commit '1500a65adbd7cd18c7bb6d0282947cd44dc76ef6':
  Support _FILE_OFFSET_BITS=64 for most of <stdio.h>.

Merge "Implement POSIX lockf." am: d9bb708575
am: 7397cc04f0

* commit '7397cc04f0f2291ff893c69c9cf6debcfdb466ea':
  Implement POSIX lockf.