Merge "Remove the last .jam file."

Remove the last .jam file.

How did I miss this before?

Change-Id: I2e582133fc4800656fdd81104f8dfc7ee8f5b580

Merge "MIPS support to sys/ headers"

Merge "Fix x86 build."

Fix x86 build.

Change-Id: Ice50ca71a19f3bf26de7cfd4c02075f1e6e0823f

Merge "Clean up the linker a bit, remove prelinking support."

Merge "Clean up libc/Android.mk a bit after the crtbrand changes."

Clean up libc/Android.mk a bit after the crtbrand changes.

Change-Id: Icfb95fffe70967418cd93da8cbeae22dcd68f8c9

Merge "Add a .note.android.ident section to Android ELF binaries."

Clean up the linker a bit, remove prelinking support.

Also make the errors more readable, since none of us seemed to know
what they actually meant. The new style is still as verbose as the
old, but that's probably necessary in the absence of chained exceptions
in C. Here's what you'd see if you try to boot after removing
libsurfaceflinger.so:

  32267 32267 E AndroidRuntime: java.lang.UnsatisfiedLinkError: Cannot load library: (linker.c:1629, pid 32259) soinfo_link_image: could not load library "libsystem_server.so" needed by "libandroid_servers.so"; caused by (linker.c:1629, pid 32259) soinfo_link_image: could not load library "libsurfaceflinger.so" needed by "libsystem_server.so"; caused by (linker.c:709, pid 32259) load_library: library "libsurfaceflinger.so" not found

This patch also fixes almost all of the compiler warnings.

Change-Id: I64bb59aed6d4e039c15ea45be2367f319ef879f8

Merge "When compiling with clang, don't "fortify_source" the strlcpy and strlcat."

When compiling with clang, don't "fortify_source" the strlcpy and
strlcat.

Change-Id: I91f58322f28e425ab9d22b51c23fcd6b772ede97

Merge "Revert "Fix the clang build: this compiler doesn't support the gnu_inline function attribute""

Merge "When compiling with clang, don't "fortify_source" the strlen."

When compiling with clang, don't "fortify_source" the strlen.

At this point, FORTIFY_SOURCE and clang are just plain incompatible.
Need to solve the underlying incompatibility first.

Change-Id: I3366477d19461e1ec93b1c30e0c7e8145b391b9b

Merge "Remove an awkward #include that is no longer upstream."

Revert "Fix the clang build: this compiler doesn't support the gnu_inline function attribute"

This reverts commit 4f1f149c3757c594f9ff987e5ddd919c3e7dba10.

Merge "Remove yet more SH4 cruft."

Remove yet more SH4 cruft.

Surely this is the last piece?

Change-Id: I4bd43afb102334ec34d0e14a65c2b319826473d7

Merge "bionic: modify syscall to use 6 registers to pass parameter."

bionic: modify syscall to use 6 registers to pass parameter.

Kernel allows to use 6 registers(exclude eax) to pass parameter.
But in syscall's implementation, it only uses five registers.
It will lead to error when 6 parameters passed.

Change-Id: I92d663194e6334c3847f0c0c257ca3b9dee0edef
Author: Jin Wei <wei.a.jin@intel.com>
Signed-off-by: Xiaokang Qin <xiaokang.qin@intel.com>
Signed-off-by: Beare, Bruce J <bruce.j.beare@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
Author-tracking-BZ: 30838

Merge "Upgrade to tzdata2012e."

Upgrade to tzdata2012e.

Includes this change:

  * australasia (Pacific/Fakaofo): Tokelau is UTC+13, not UTC+14.
    (Thanks to Steffen Thorsen.)

Change-Id: I51bfab759f424b90daeb1960c6cfb2b55b78ff46

Merge "Fix the clang build: this compiler doesn't support the gnu_inline function attribute"

Fix the clang build: this compiler doesn't support the gnu_inline function attribute

Recent clang compiler doesn't have gnu_inline working. When frameworks/rs
is compiled with clang instead of gcc, it no longer works. Will be fixed
by this patch.

Change-Id: I8b281a1305151909e18958b73914ea27343cb4cd

Merge "[MIPS] Support for TLS register"

Remove an awkward #include that is no longer upstream.

This was causing trouble in https://android-review.googlesource.com/#/c/37590/.

Change-Id: Ib7cb4fbaef6e9c1470c59f67d24bd24c7ce90bcc

MIPS support to sys/ headers

Change-Id: I32207a1d918e4842da341f6b242ae39c69a83b03

[MIPS] Support for TLS register

Change-Id: I6b5194e5e379b573e3b1203d8015e20fb3edd647
Signed-off-by: Chris Dearman <chris@mips.com>

Merge "MIPS support for libc."

MIPS support for libc.

Change-Id: I2864dea04b3faf2d919165dcaa600af5b16c41c8
Signed-off-by: Chris Dearman <chris@mips.com>
Signed-off-by: Raghu Gandham <raghu@mips.com>

Merge "Fix a ENOTUP/ENOTSUP typo."

Fix a ENOTUP/ENOTSUP typo.

Change-Id: Ifa347a09a2aa1f505965ae7c1383fd0caf95793c

Merge "Error numbers are target specific."

Merge "MIPS support to the linker"

MIPS support to the linker

Change-Id: I37ec2d6c51d82bb9e9dbfef4b38c85366bead255
Signed-off-by: Chris Dearman <chris@mips.com>
Signed-off-by: Raghu Gandham <raghu@mips.com>
Signed-off-by: Bhanu Chetlapalli <bhanu@mips.com>

Error numbers are target specific.

Use the system supplied error numbers when mapping error numbers to messages.

Change-Id: I520556fa3e2ff668fdc4eda36ad31491fbb48ea8
Signed-off-by: Chris Dearman <chris@mips.com>
Signed-off-by: Raghu Gandham <raghu@mips.com>

Merge "Had include/arm/fenv.h, but missed include/mips/fenv.h."

Merge "Updated the cleaned header file mman.h for MIPS."

Updated the cleaned header file mman.h for MIPS.

Change-Id: I8b3734e535711ca07fbf4066e6d967d1ce0f30eb

Merge "bionic: Stack pointer/signal race condition."

Merge "fix __cxa_finalize() implementation to be thread safe."

fix __cxa_finalize() implementation to be thread safe.

__cxa_finalize() modifies the access permissions of __atexit
global variable without acquiring  _ATEXIT_LOCK(). Fix it prevent
any possible races.

Change-Id: I11939d0ebcbf6f360c14163222d40a449d96948e

Add a .note.android.ident section to Android ELF binaries.

This allows debugging tools to know they are working with Android
binaries and adapt accordingly.

Signed-off-by: Thiago Jung Bauermann <thiago.bauermann@linaro.org>
Signed-off-by: Michael Hope <michael.hope@linaro.org>
Change-Id: Ic906992fcad61c028bb765821637a3e1333bf52b

Had include/arm/fenv.h, but missed include/mips/fenv.h.

Needed for the device-build for LLVM's constant folding pass.

Change-Id: I412a98d52458231650801a8b59fe3159a789b8b9
Author: fu@mips.com

Merge "Install crt(begin|end)_so.o to target library path."

Install crt(begin|end)_so.o to target library path.

BUG=6886348.

Change-Id: I176b2c4963d9731b06d10d4e0b92c010a0006b59

Merge "Report errors to the log, not just stderr."

Report errors to the log, not just stderr.

In particular this affects assert(3) and __cxa_pure_virtual, both of
which have managed to confuse people this week by apparently aborting
without reason. (Because stderr goes nowhere, normally.)

Bug: 6852995
Bug: 6840813
Change-Id: I7f5d17d5ddda439e217b7932096702dc013b9142

Merge "Fix a TEMP_FAILURE_RETRY usage error in the linker."

Fix a TEMP_FAILURE_RETRY usage error in the linker.

Similar to the fix in c20d0f3993ebb0d3dec958a306a68ebb48bfeadd.

grep(1) says this was the only other instance in bionic.

Change-Id: I1729038762ee1c7c4743a6bd11d5558afd6f5749

Merge "Replace ALL_PREBUILTS with BUILD_PREBUILT"

Replace ALL_PREBUILTS with BUILD_PREBUILT

(cherry-picked from 186b36b65bdfbf2c89a661539bf530359364eac9.)

Bug: 6857263
Change-Id: If787d84415dd2ac07105fe0d0b7a27c9be75df64

Merge "Correction to use of TEMP_FAILURE_RETRY in send_prop_msg"

Correction to use of TEMP_FAILURE_RETRY in send_prop_msg

RETRY macro may retry command if result is -1. In this
case the command was "connect < 0" instead of just
connect. The comparison will not return -1 and thus
retry is never done. This is now corrected so that
interrupts will cause retry instead of fail.

(There was no other negative side effect of the bug.
The result code from RETRY was used in an if-statement
and it would be true for all negative connect results.
This was according to expectations.)

Change-Id: Ie206b39878e9befea4e3be9a4061ee39eb232d80

bionic: Stack pointer/signal race condition.

Move the stackpointer so a captured signal does not corrupt
stack variables needed for __thread_entry.

Change-Id: I3e1e7b94a6d7cd3a07081f849043262743aa8064

Merge "Remove the 'Factory' time zone by not processing the 'factory' file."

Remove the 'Factory' time zone by not processing the 'factory' file.

The factory file (and Factory time zone) was meant as a way to say
"not configured" that would give a clear error when running date(1).
For us it would just look like UTC, so it is of no value.

Bug: 2997381
Change-Id: I1a4b85dce97d1d9370b22ba79e8fe5dafff56541

Merge "vfprintf: fix spelling."

vfprintf: fix spelling.

Change-Id: I6dc611143fb1f3e9ccf0a77b40ee48a3c1d72ca9

Merge "[MIPS] Use an aligned address to provoke SIGSEGV in abort()"

[MIPS] Use an aligned address to provoke SIGSEGV in abort()

Change-Id: I269c9ccc07058773fb0f9d70673673157ab38f6a

Merge "Fix implementation of generic atomic operations"

Fix implementation of generic atomic operations

Change-Id: Ie1ea5aacc561e2d6d40125d2952ed0e9116b7b0d

Merge "[MIPS] Add atomic routines"

[MIPS] Add atomic routines

Change-Id: I2cb20ce44dd230d222b7fc1ede2e1e3dce6e692b

Merge "Upgrade to tzdata2012d."

Upgrade to tzdata2012d.

This upgrade involved rewriting the script; the data has moved to ftp.iana.org,
where it's slightly less convenient to access, so it's time to use something
that can talk FTP...

As for tzdata2012d, it's just updating Morocco for this weekend's changes, now
they've been decided at the last minute (as usual).

Change-Id: I772df57a6e09b3bf3d9541bfc08930d6f18633b4

Merge "Update time.h for CLOCK_BOOTTIME."

Update time.h for CLOCK_BOOTTIME.

(cherry-pick of 8958a383296ea0e1cc106ae245c37671809ffd49)

Change-Id: Ie8de6b32fa81566db53ad7e9fd4b197f4cede628

Merge "linker: Fix LD_PRELOADS for calling constructors"

Merge "bionic: Report linker relocation address to gdb"

Merge "linker: cleanup of undefined state handling, which is really dead code"

linker: cleanup of undefined state handling, which is really dead code

Given that _elf_lookup (and thus, _do_lookup) cannot possibly return an
undefined symbol (due to the check for SHN_UNDEF in _elf_lookup), there's
no need for spurious checks for SHN_UNDEF on its return value.

Conflicts:

linker/linker.c

Change-Id: Ic73cf439924b45f72d4d9ba3f64a888c96cbbd9b

Merge "linker: Use SHN_UNDEF instead of describing what it is trying to do."

linker: Use SHN_UNDEF instead of describing what it is trying to do.

Merge "Use std=gnu99 for the dynamic linker as well as libc."

Use std=gnu99 for the dynamic linker as well as libc.

Change-Id: I76dd78576c5af6eb6282555f069647b6260edc31

Merge "limits.h: Include page.h when relying on PAGE_SIZE"

Merge "FORTIFY_SOURCE: revert memcpy changes."

bionic: Report linker relocation address to gdb

GDB needs the runtime linker's base address in order to
locate the latter's ".text" and ".plt" sections, for the
purpose of detecting solib trampolines.  It also can
potentially use this to calculate the relocated address
of rtld_db_dlactivity.

Bug: http://code.google.com/p/android/issues/detail?id=34856
Change-Id: I63d3e7ae4e20a684ceb25967f2241e7d58dd685d
Signed-off-by: Ryan V. Bissell <ryan@bissell.org>

linker: Fix LD_PRELOADS for calling constructors

Change-Id: I1eae77a4c59e8a5acc009127d271455bb6fc01b6

FORTIFY_SOURCE: revert memcpy changes.

Performance regressions.  Hopefully this is a temporary
rollback.

Bug: 6821003
Change-Id: I84abbb89e1739d506b583f2f1668f31534127764

Merge "FORTIFY_SOURCE: introduce __BIONIC_FORTIFY_UNKNOWN_SIZE macro"

Merge "FORTIFY_SOURCE: strlen check."

Merge "FORTIFY_SOURCE: restore __memcpy_chk()"

FORTIFY_SOURCE: introduce __BIONIC_FORTIFY_UNKNOWN_SIZE macro

Replace all occurances of "(size_t) -1" with a
__BIONIC_FORTIFY_UNKNOWN_SIZE macro.

Change-Id: I0b188f6cf31417d2dbef0e1bd759de3f9782873a

FORTIFY_SOURCE: strlen check.

This test is designed to detect code such as:

int main() {
  char buf[10];
  memcpy(buf, "1234567890", sizeof(buf));
  size_t len = strlen(buf); // segfault here with _FORTIFY_SOURCE
  printf("%d\n", len);
  return 0;
}

or anytime strlen reads beyond an object boundary. This should
help address memory leakage vulnerabilities and make other
unrelated vulnerabilities harder to exploit.

Change-Id: I354b425be7bef4713c85f6bab0e9738445e00182

FORTIFY_SOURCE: restore __memcpy_chk()

In our previous FORTIFY_SOURCE change, we started using a custom
inline for memcpy(), rather than using GCC's __builtin_memcpy_chk().
This allowed us to delete our copy of __memcpy_chk(), and replace it
by __memcpy_chk2().

Apparently GCC uses __memcpy_chk() outside of __builtin_memcpy_chk().
Specifically, __memcpy_chk() is used by __builtin__memMOVE_chk() under
certain optimization levels.

Keep the old __memcpy_chk() function around, and have it call into
__memcpy_chk2().

Change-Id: I2453930b24b8a492a3b6ed860e18d92a6b762b80

Merge "FORTIFY_SOURCE: enhanced memcpy protections."

FORTIFY_SOURCE: enhanced memcpy protections.

Two changes:

1) Detect memory read overruns.

For example:

int main() {
  char buf[10];
  memcpy(buf, "abcde", sizeof(buf));
  sprintf("%s\n", buf);
}

because "abcde" is only 6 bytes, copying 10 bytes from it is a bug.
This particular bug will be detected at compile time.  Other similar
bugs may be detected at runtime.

2) Detect overlapping buffers on memcpy()

It is a bug to call memcpy() on buffers which overlap. For
example, the following code is buggy:

  char buf3[0x800];
  char *first_half  = &buf3[0x400];
  char *second_half = &buf3[1];
  memset(buf3, 0, sizeof(buf3));
  memcpy(first_half, second_half, 0x400);
  printf("1: %s\n", buf3);

We now detect this at compile and run time.

Change-Id: I092bd89f11f18e08e8a9dda0ca903aaea8e06d91

Merge "memmove: Don't call memcpy if regions overlap"

memmove: Don't call memcpy if regions overlap

memmove() unconditionally calls memcpy() if "dst" < "src". For
example, in the code below, memmove() would end up calling memcpy(),
even though the regions of memory overlap.

int main() {
  char buf3[0x800];
  char *dst  = &buf3[1];
  char *src = &buf3[0x400];
  memset(buf3, 0, sizeof(buf3));
  memmove(dst, src, 0x400);
  printf("1: %s\n", buf3);
  return 0;
}

Calling memcpy() on overlaping regions only works if you assume
that memcpy() copies from start to finish. On some architectures,
it's more efficient to call memcpy() from finish to start.

This is also triggering a failure in some of my code.

More reading:
* http://lwn.net/Articles/414467/
* https://bugzilla.redhat.com/show_bug.cgi?id=638477#c31 (comment 31)

Change-Id: I65a51ae3a52dd4af335fe5c278056b8c2cbd8948

Don't use -fstack-protector on ssp.c

libc's stack protector initialization routine (__guard_setup)
is in bionic/ssp.c. This code deliberately modifies the stack
canary.  This code should never be compiled with -fstack-protector-all
otherwise it will crash (mismatched canary value).

Force bionic/ssp.c to be compiled with -fno-stack-protector

Change-Id: Ib95a5736e4bafe1a460d6b4e522ca660b417d8d6

limits.h: Include page.h when relying on PAGE_SIZE

limits.h relies on PAGE_SIZE being defined without actually including
page.h. Make sure this is included to avoid compilation failures.

Signed-off-by: Arun Raghavan <arun.raghavan@collabora.co.uk>

Merge "FORTIFY_SOURCE: Add openat, fix bug"

FORTIFY_SOURCE: Add openat, fix bug

Add fortify_source support for openat(). This change requires that
an argument be supplied when using O_CREAT.

Fix unnecessary call to __open_2. If, at compile time, we know that
"flags" is constant and DOESN'T contain O_CREAT, the call to __open_2
is useless.

Change-Id: Ifcd29c4fb25e25656961d7552d672e161f0cfdbd

Merge "FORTIFY_SOURCE: add fgets support."

am 67636eea: am 40e7ed58: Unhide rtld_db_dlactivity()

* commit '67636eea20f7789e6689ee8cf6017e7d48735ca1':
  Unhide rtld_db_dlactivity()

am 40e7ed58: Unhide rtld_db_dlactivity()

* commit '40e7ed58d73eae59d0cf2fed61284d16692e307b':
  Unhide rtld_db_dlactivity()