Merge "servicemanager: log pid and uid on selinux denial"

Merge "dumpstate: add -v printable to logcat"

dumpstate: add -v printable to logcat

In the case where binary data lands in an utf-8 log buffer,
ensure that the content is made printable.

Bug: 23517551
Change-Id: I41696372d07228875d01a19f0c7ab2cb7494b7ef

Merge "dumpstate: execute procrank using su"

dumpstate: execute procrank using su

procrank is a setuid binary which only exists on userdebug/eng
builds. Instead of executing the setuid binary, run the binary using
the su command. This eliminates one more setuid binary, and allows
the tightening of the SELinux policy.

Bug: 18342188
Change-Id: I81d06196a29489feaefb2fea0cf5c4ff1e273d2f

servicemanager: log pid and uid on selinux denial

The audit logs for servicemanager were missing the pid and uid of the source.
This is useful for debugging.

Before:
... SELinux : avc:  denied  { find } for service=android.security.keystore scontext=u:r:system_app:s0 ...
After:
... SELinux : avc:  denied  { find } for service=android.security.keystore pid=1252 uid=1000 scontext=u:r:system_app:s0 ...

Change-Id: Id1a6f38b99f11a31315439620ead2f01108b18b2
Signed-off-by: William Roberts <william.c.roberts@intel.com>

am bd55551d: Merge "Add more methods to IPowerManager."

* commit 'bd55551d80832f62931ccb2d54d9a239822eeb20':
  Add more methods to IPowerManager.

Merge "Add more methods to IPowerManager."

Add more methods to IPowerManager.

Add the full list of transaction IDs from IPowerManager.aidl
to IPowerManager.h and make BpPowerManager support calling
goToSleep, reboot, shutdown, and crash. These are currently
needed by or likely to be needed by Brillo.

Bug: 22122485
Change-Id: I19abd3587c9d53b28ec150210e07f97517ee4ff4

am 77f147ff: Merge "frameworks/native: remove superfluous mention of SharedBuffer in Region.h"

* commit '77f147ff566d350dc23c3a9807ea1262f056ff2f':
  frameworks/native: remove superfluous mention of SharedBuffer in Region.h

Merge "frameworks/native: remove superfluous mention of SharedBuffer in Region.h"

frameworks/native: remove superfluous mention of SharedBuffer in Region.h

Towards deprecation of SharedBuffer

Change-Id: I9c853c86fb057280013405cb8c3c725e13175ea4

am 28b1d678: Merge "Installd: Fully decouple pre- and post-bootcomplete properties"

* commit '28b1d678a4721e9f02b7b9498492fb3d623f21be':
  Installd: Fully decouple pre- and post-bootcomplete properties

Merge "Installd: Fully decouple pre- and post-bootcomplete properties"

Installd: Fully decouple pre- and post-bootcomplete properties

Do not fall back to dalvik.vm.dex2oat-threads if
dalvik.vm.boot-dex2oat-threads is not set. Also do not ever use
the value of dalvik.vm.image-dex2oat-threads. Instead use the
default behavior of dex2oat in that situation.

Change-Id: I42e60996341414b95b8df2d243d5e07f7f5ff9b6

am 905c1446: Merge "Allow using the JIT"

* commit '905c1446ea602b4649a248bfe878626cae01a87a':
  Allow using the JIT

Merge "Allow using the JIT"

Allow using the JIT

Instead of the JIT only being available for eng builds [or configurable
via environment variable], allow the JIT to be enabled programatically.

Change-Id: Id3c5ae227ed400e489bb723a56c516dfc12acd89

am d529cd47: Merge "Pass dexopt flags as integer"

* commit 'd529cd470afa98512bf2eab7a58dae5a9b6e4847':
  Pass dexopt flags as integer

Merge "Pass dexopt flags as integer"

Pass dexopt flags as integer

Instead of using a series of booleans, create a single flags integer
that contains all of the dexopt options.

Change-Id: I35542aa73ca57b0e765d19b1339b1429849c1ae8

am 7f413889: Merge "Installd: Take boot status as dexopt parameter"

* commit '7f413889a3f0bff0c159141abb498650305f2089':
  Installd: Take boot status as dexopt parameter

Merge "Installd: Take boot status as dexopt parameter"

Installd: Take boot status as dexopt parameter

Expect the boot status explicitly as a parameter so that we do not
have to rely on dev.bootcomplete, which isn't meaningfully set
when the device needs the decryption screen on boot.

Bug: 23898216
(cherry picked from commit 72ebebed876a62e719b098e43d9d516361bde029)
Change-Id: I9b34298caf70b1e5d40970cc0d04c469016a80a7

am 0830bb16: Merge "Enable integer overflow sanitization in libinput."

* commit '0830bb169717f920817e66afb18abe2c86d20161':
  Enable integer overflow sanitization in libinput.

am bffbd09f: Merge "Eliminate multiple benign overflow conditions."

* commit 'bffbd09f65d3bc594d844a185d75700e8b4789eb':
  Eliminate multiple benign overflow conditions.

am 64b83133: Merge "Remove usage of SharedBuffer"

* commit '64b83133d5ec40828a10e57cfbb37c642bb7f510':
  Remove usage of SharedBuffer

Merge "Enable integer overflow sanitization in libinput."

Enable integer overflow sanitization in libinput.

Bug: 24171356
Change-Id: I1412b78171c60b0252560956a672f2d92a7011e6

Merge "Eliminate multiple benign overflow conditions."

Merge "Remove usage of SharedBuffer"

am ad2a1592: Merge "Enable multiple benign overflow conditions."

* commit 'ad2a1592cf01a5393ad645fea81b6c36ee11f73e':
  Enable multiple benign overflow conditions.

am 146f31b6: Merge "Eliminate benign overflow condition triggered upon loop termination in Input.cpp"

* commit '146f31b6c44f557270779106e95c769783270fed':
  Eliminate benign overflow condition triggered upon loop termination in Input.cpp

Merge "Enable multiple benign overflow conditions."

Merge "Eliminate benign overflow condition triggered upon loop termination in Input.cpp"

Eliminate multiple benign overflow conditions.

In InputTransport.cpp, there are multiple loops in which loop
termination occurs when the value becomes zero. These termination
conditions are all written value-- > 0, which, since value is
unsigned, result in an unsigned integer overflow when value is 0.
These loops were refactored to eliminate these conditions.

Bug: 24171356
Change-Id: Ie135c4306d1f2cef2778e295242305ed5139221a

Enable multiple benign overflow conditions.

In VelocityTracker.cpp, there are multiple loops in which loop
termination occurs when the value becomes zero. These termination
conditions are all written value-- > 0, which, since value is
unsigned, result in an integer overflow condition when value is 0.
These loop were refactored to eliminate these conditions.

Bug: 24171356
Change-Id: Ie44940cfef8a122ee1aff59c021274ba39a368bf

Eliminate benign overflow condition triggered upon loop termination in Input.cpp

In readFromParcel, a while loop is terminated when sampleCount = 0. The
decrement operation that was here would decrease sampleCount, an unsigned value,
below 0, triggering an unsigned integer overflow. The while loop was refactored
to eliminate this condition.

Bug: 24171356
Change-Id: I7669f54a41d11548b33e322b025431c6f6038952

am 251c8b3f: Merge "constify generated JNINativeMethod fn ptr tables"

* commit '251c8b3ff842363243c8eea25b369bd805af7aad':
  constify generated JNINativeMethod fn ptr tables

Merge "constify generated JNINativeMethod fn ptr tables"

constify generated JNINativeMethod fn ptr tables

Change-Id: Icbad34e298f6d8e60ef12e9a91200994249fe2f5

Remove usage of SharedBuffer

- Remove getSharedBuffer() from Region
- Don't use SharedBuffer for memory management in HWCLayerVersion1,
  instead keep shallow copies of the Regions.

Bug 23962051

(cherry picked from commit d814cf2a3e3a2fdb73efa80539fe8af0a93da1db)

Change-Id: I8fa17beed57936648c7b4defc9219dff1d5b337f

resolved conflicts for 5b5c81c4 to stage-aosp-master

Change-Id: I0be665105d46cb3b934cb00890822d4dfae18f64

With SANITIZE_TARGET, load EGL libs from the instrumented path.

Bug: 21785137
Change-Id: Iac56419a9ca776c5704bed44d3f0a1a1edd9d172
(cherry picked from commit c2466e6f005e9cbeea7764e9d8864fa6bca17633)

Merge "asan: update condition to work with multiple SANITIZE_TARGET values."

asan: update condition to work with multiple SANITIZE_TARGET values.

The goal is to enable SANITIZE_TARGET='address coverage', which
will be used by LLVMFuzzer.

Bug: 22850550
Change-Id: Iaa300aa20798f2e9aa613842b463edf8aeebe3a7

am c5a6696f: Merge "Installd: Allow different behavior before bootcomplete"

* commit 'c5a6696f3e51e253b78baacdec99ed34e79bc3fd':
  Installd: Allow different behavior before bootcomplete

Merge "Installd: Allow different behavior before bootcomplete"

Installd: Allow different behavior before bootcomplete

Check dev.bootcomplete in dex2oat(). Use the information for
two changes.

Only switch to the background when we're post bootcomplete.
This will ensure better utilization after upgrades.

Add a second dex2oat-threads property that is used pre
bootcomplete. A separation of these phases allows using less
cores when the device is up, freeing up resources for other
purposes, e.g., avoid jank. The precedence during boot is the
boot property, or the image property if the former doesn't
exist, or the default property as a fallback.

Bug: 23898216
Bug: 24004256

(cherry picked from commit 21948005f0294a64a42932b21a33ae75295dc3d9)

Change-Id: I5063f3fc4b437cbe88c4e94584e01c1c78eccc4d

am c0ed1282: Merge "Update libpowermanager to expose enum and export headers."

* commit 'c0ed128263f753696cbcb24a5d1a45bc894ae336':
  Update libpowermanager to expose enum and export headers.

Merge "Update libpowermanager to expose enum and export headers."

Update libpowermanager to expose enum and export headers.

Move libpowermanager's transaction ID enum (needed when
implementing BnPowerManager::onTransact()) into
IPowerManager.h and update the library's Android.mk file to
export its headers.

Bug: 22122485
Change-Id: Ie97198c3d2ebe8e880125e19a001ef7162009858

am f2e53c5e: Merge "Remove BnServiceManager."

* commit 'f2e53c5e1f3fd27c9131f20b2f8147c82372f746':
  Remove BnServiceManager.

Merge "Remove BnServiceManager."

Remove BnServiceManager.

As far as I can tell (e.g. https://goo.gl/nxpFBZ from 2009),
BnServiceManager is unused. It's only relevant for
libbinder-based C++ daemons implementing the
IServiceManager.aidl interface. service_manager.c rolls its
own binder code instead.

Change-Id: I837a897d8a201b04bf3ff65c87b0d35d10196199

am 6c8c8138: Merge "Benign unsigned integer overflow in Parcel"

* commit '6c8c8138434d2381a78407e9876121d3bcef55a7':
  Benign unsigned integer overflow in Parcel

am 8c77ec50: Merge "Enable fsanitize integer in ui"

* commit '8c77ec506a83538d85cf68b8095541ca8aa8c788':
  Enable fsanitize integer in ui

Merge "Benign unsigned integer overflow in Parcel"

Merge "Enable fsanitize integer in ui"

Benign unsigned integer overflow in Parcel

The realloc case in continueWrite did not update the
gParcelGlobalAllocCount value when an allocation occurred.

In addition, there are conditions that could cause the
gParcelGlobalAllocCount value to be decremented below 0, resulting
in a benign unsigned integer overflow that can cause corrupted values
to be returned through system profiling mechanisms.

BUG: 23972600
Change-Id: Ibe07db91a811a04b486760eb78d81c926ba8503d

am bf04509f: Merge "Enable fsanitize integer in binder"

* commit 'bf04509ff42fc1e179b518caada5d0b663f2a8ee':
  Enable fsanitize integer in binder

Merge "Enable fsanitize integer in binder"

Enable fsanitize integer in binder

Enables checks for undefined or suspicious integer behavior. If
suspicious behavior is detected, the process is aborted instead of
potentially resulting in a security bug, such as #20674694 and
#19573085.

Bug: 23836686
Change-Id: I14d6d565afcb8a6435fd5a8e8175e00bbe5d9b6c

Enable fsanitize integer in ui

Enables checks for undefined or suspicious integer behavior. If
suspicious behavior is detected, the process is aborted instead of
potentially resulting in a security bug.

Bug: 23834892
Change-Id: I9111bce2ebb36e045f4da1dbf78f90f04a0aa81c

am 477b1e87: Merge "move init.trace.rc to frameworks/native"

* commit '477b1e873a894a4331535bf1a59d76ec0cdd809a':
  move init.trace.rc to frameworks/native

Merge "move init.trace.rc to frameworks/native"

move init.trace.rc to frameworks/native

init.trace.rc will be renamed to atrace.rc and use the LOCAL_INIT_RC
mechanism to be included on /system appropriately.

Bug 23186545

Change-Id: Ic5833b4bb6e6364e4fef6ab4882ae75f8f3856e9

am 40a8bb3b: Merge "Add a tag for tracing system server."

* commit '40a8bb3b2290f77a90735633eab350133f9bb9d2':
  Add a tag for tracing system server.

Merge "Add a tag for tracing system server."

am ce92141c: Merge "Whitelist windows module"

* commit 'ce92141cbdfeae302fcc1d5e4b4b261c783feaf7':
  Whitelist windows module

resolved conflicts for merge of 3821feac to stage-aosp-master

Change-Id: Icc17d2b7dc26eadf99b2c6967cceaa54ae38e617

Merge "Whitelist windows module"

Merge "include/android/sensor: add enum for LINEAR_ACCELERATION"

am 1159b181: Merge "ignore SIGPIPE signal in native services"

* commit '1159b18177c7247884d7c90c11abfe5aae39cb34':
  ignore SIGPIPE signal in native services

Merge "ignore SIGPIPE signal in native services"

ignore SIGPIPE signal in native services

When run dump method via dumpsys command with invalid pipe fd, native
services can crash with SIGPIPE.
So ignore SIGPIPE signal.

Bug: https://code.google.com/p/android/issues/detail?id=184099
Change-Id: Icb15e0dff34a245c08c1b4c4bed51992ae382da4
Signed-off-by: Young-Ho Cha <ganadist@gmail.com>

include/android/sensor: add enum for LINEAR_ACCELERATION

Change-Id: I6e387f905ba87bde810952daf939ba0f529d18b8

Whitelist windows module

Bug: 23566667
Change-Id: I4fa1243421885a0c1a8febbbcefb0d7e9cdcc99b

am cf1fedef: am 2acadb3f: Merge "BatteryProperties: Adds batteryCycleCount, batteryCurrent, batteryFullCharge fields in the the battery properties."

* commit 'cf1fedef98db7a5ae30685cdc47b5b6e107fba50':
  BatteryProperties: Adds batteryCycleCount, batteryCurrent, batteryFullCharge fields in the the battery properties.

am 2acadb3f: Merge "BatteryProperties: Adds batteryCycleCount, batteryCurrent, batteryFullCharge fields in the the battery properties."

* commit '2acadb3f26b3f9b7f2ae7c1bca219ada94066462':
  BatteryProperties: Adds batteryCycleCount, batteryCurrent, batteryFullCharge fields in the the battery properties.

Merge "BatteryProperties: Adds batteryCycleCount, batteryCurrent, batteryFullCharge fields in the the battery properties."

BatteryProperties: Adds batteryCycleCount, batteryCurrent,
batteryFullCharge fields in the the battery properties.

Bug: 22981343
Change-Id: I426b4b7d1eb544d2b1d97d87ae1eba0fdf35c503
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

am d3bfedbb: am f364393a: Merge "libagl: Fix buffer read overrun in eglCreatePbufferSurface"

* commit 'd3bfedbb1fc6296b561e60eac19581a6f1705f9e':
  libagl: Fix buffer read overrun in eglCreatePbufferSurface

am f364393a: Merge "libagl: Fix buffer read overrun in eglCreatePbufferSurface"

* commit 'f364393a01824bf4d805b6fffc78bb00d426eb9f':
  libagl: Fix buffer read overrun in eglCreatePbufferSurface

Merge "libagl: Fix buffer read overrun in eglCreatePbufferSurface"

am 32592270: am 64e13880: Merge "bundle init.rc contents with its service"

* commit '32592270d40d6060e19aa97949aee72c8aca0807':
  bundle init.rc contents with its service

am 64e13880: Merge "bundle init.rc contents with its service"

* commit '64e138803478c22cac598b143b3f109d6220a167':
  bundle init.rc contents with its service

Merge "bundle init.rc contents with its service"

libagl: Fix buffer read overrun in eglCreatePbufferSurface

The code was assuming that EGL_NONE==0, which isn't true.

Bug: 23403170
Change-Id: Ic1eccdef086b4d610bd78dbb6b2ae883c91dc322

am a6d2ff29: am 110ddef1: Merge "binder: Change from C11 <stdatomic.h> to C++11 <atomic>."

* commit 'a6d2ff29c4920b440b16bc6ceaa90710361b9be9':
  binder: Change from C11 <stdatomic.h> to C++11 <atomic>.

am 110ddef1: Merge "binder: Change from C11 <stdatomic.h> to C++11 <atomic>."

* commit '110ddef1da19c03dd5e167680d00f48e1377da9d':
  binder: Change from C11 <stdatomic.h> to C++11 <atomic>.

Merge "binder: Change from C11 <stdatomic.h> to C++11 <atomic>."

bundle init.rc contents with its service

Bug: 23186545
Change-Id: I2a70a39122b146d5bbc9b55156e769782e011bff

binder: Change from C11 <stdatomic.h> to C++11 <atomic>.

Change-Id: Ib831a9ccfcc7089940d37883d860574359c270e2

am 7a1ccf7c: am d974a364: Merge "binder: Add a throughput performance test for binder."

* commit '7a1ccf7c7d1dd0defa1a82d973a8b80f3370c52c':
  binder: Add a throughput performance test for binder.

am d974a364: Merge "binder: Add a throughput performance test for binder."

* commit 'd974a3647bd371886093738ab16485c943cb5bad':
  binder: Add a throughput performance test for binder.

Merge "binder: Add a throughput performance test for binder."

binder: Add a throughput performance test for binder.

Change-Id: I04801cd505a439307f2a963f6267db1831b3829f

am 7bd45d00: am 4997a57a: Merge "Tracing for binder locks/transactions"

* commit '7bd45d0010622e559f7795243ec8e548f641266b':
  Tracing for binder locks/transactions

am 4997a57a: Merge "Tracing for binder locks/transactions"

* commit '4997a57a9ae90bbb058dec90222d423a7de0e7e7':
  Tracing for binder locks/transactions