Fix issue #27252896: Security Vulnerability -- weak binder

Sending transaction to freed BBinder through weak handle
can cause use of a (mostly) freed object.  We need to try to
safely promote to a strong reference first.

Change-Id: Ic9c6940fa824980472e94ed2dfeca52a6b0fd342

Merge "media: add and clarify OMX float numerical type" into nyc-dev

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9 am: c240884 am: 1899ffd
am: d53600c  -s ours

* commit 'd53600cc8441a9b552727e8f278f9c4b38224806':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938 am: b2428c7 am: ce3faa0
am: 6c42071

* commit '6c42071f84c450a884c72d38a44ab3f45dd16343':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9 am: c240884
am: 1899ffd

* commit '1899ffd6767574f1457b3ab5779e525ecc360d2a':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938 am: b2428c7
am: ce3faa0

* commit 'ce3faa072a58dc01cecffa61d7df0a1d8bca212b':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9
am: c240884

* commit 'c240884dccdf9f40725113ab88ebf615325dab19':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938
am: b2428c7

* commit 'b2428c71c20a2e67a257eba540e6de2ca0fbae22':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours
am: e98a9a9

* commit 'e98a9a910692cf7dee3bac2b58cf5f08c59db38f':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469
am: 46da938

* commit '46da9385ad96fb1c2baab64fdea01eb3b0b41dd0':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372
am: fdd9851  -s ours

* commit 'fdd9851ba326132c11d177bf5152b0c66cd87764':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857
am: 1ed2469

* commit '1ed24698aa431aa4c3198fa3b8e830f1c29812a6':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours
am: 6fdc372

* commit '6fdc37205c32dcbd23718d68e45806c76f1536f8':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f
am: f16b857

* commit 'f16b85752d7344814663c658175cce1a933e976b':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours
am: 12a01ba  -s ours

* commit '12a01ba5f4f3f8ba122dfbf689e976019f353b3d':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables
am: 93c617f

* commit '93c617fd2a5e7910e0ba5c0ed6da152d30920679':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74
am: dc5d0f4  -s ours

* commit 'dc5d0f46de1f8a800b3af340ca57278989df151a':
  DO NOT MERGE BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev
am: b4eac74

* commit 'b4eac742c9e3f0238d5d03b237b2038df885ed2c':
  DO NOT MERGE BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev

media: add and clarify OMX float numerical type

Bug: 27172163
Change-Id: Idb399c05d14ee51f944251430d3561479feee6e3

Merge "MemoryDealer: expose internal allocation alignment" into nyc-dev

Merge "Added -q flag to showmap." into nyc-dev

Merge "Add user activity event type for accessibility." into nyc-dev

Added -q flag to showmap.

dumpstate calls showmap for each pid, and since most of them are empty,
it ends up polluting logcat with entries like:

03-17 14:49:05.974 12160 12160 E dumpstate: command '/system/xbin/su
root showmap -q 9867' failed: No such file or directory

BUG: 26906985
Change-Id: I7e064b2ec8901b6ccee49b228be96ff2847c703c

MemoryDealer: expose internal allocation alignment

Bug: 27722308
Change-Id: Ie4ee1f54edcb711d39f8419a160af1495af0152e

BQ: Allow shared buffer to be the first queued buffer

- When single-buffer mode is enabled the shared buffer will now be
  either the first buffer dequeued or the first buffer queued.
  Whichever comes first.

Bug 24940410

Change-Id: I9f7e2630640e37fe1fb967aed41a3025567247a8

Merge "Native: Sensor: For ndk sensor enable request, enable sensor with SENSOR_DELAY_NORMAL delay instead of 0 delay, by default " into nyc-dev

Native: Sensor: For ndk sensor enable request, enable sensor with
SENSOR_DELAY_NORMAL delay instead of 0 delay, by default

In case of one or more sensor applications running in background
(say Auto Rotation, Auto Brightness, some sensor ndk daemon like
panning, etc), which are using the common sensor (say accelerometer)
running at UI delay(66 ms), and then some 3rd party sensor ndk daemon
starts(say Direct Call feature, GPS ndk daemon for correcting latitude
and longitude using sensor data while using navigation, etc), which
enables the same sensor (accelerometer) but does not set the delay,
then that sensor chip will start to generate data at minimum delay
possible because of passing 0 delay value during enable.

Ideally, this is wrong behaviour because none of the sensor-ndk daemon
or android application has requested for minimum delay yet and so sensor
should generate data still at 66ms only. This in turn will make
significant increment in current consumption because of fastest sampling
rate.

Also, as you know, ASensorEventQueue_enableSensor api and
ASensorEventQueue_setEventRate api are not deprecated and is still used
irrespective of HAL 1_3 or latest 1_4. In fact, these are the only two
api currently available for enabling and setting delay. Direct Call,
panning, Smart alert, gps daemon, etc are such examples which uses these
ndk api

Small Code Snippet:
AutoRotation starts:
mSensorManager.registerListener(
        listener1, mAccelerometer, SensorManager.SENSOR_DELAY_UI);

AutoBrightness start:
mSensorManager.registerListener(
        listener2, mAccelerometer, SensorManager.SENSOR_DELAY_NORMAL);
mSensorManager.registerListener(
        listener2, mLight, SensorManager.SENSOR_DELAY_NORMAL);

Panning:
ASensorEventQueue_enableSensor(mSensorEventQueue1, mAccelerometer);
ASensorEventQueue_setEventRate(
        mSensorEventQueue1, mAccelerometer, SENSOR_DELAY_UI);
(Here, first 0 delay will be set, but UI delay will override it with
function call setEventRate)
... more possible android-apps/sensor-ndk daemon

So currently, accelerometer is running at 66 ms delay
because DELAY_UI (66ms) is less than DELAY_NORMAL (200ms)

Now, sensor gps ndk daemon starts, enables the sensor but does not set
delay:
ASensorEventQueue_enableSensor(mSensorEventQueue2, mAccelerometer);
--> This call will request for 0ms delay so minimum delay will be set
(say 10ms)

Therefore now, accelerometer will run at 10 ms delay which is wrong
behavior because delay should still be 66 ms ideally. Also, running
sensor at lowest delay unnecessarily increases current consumption.
(Unless requested for minimum delay like in games)

Change-Id: Ib18d0d77f80b2621a204d491c3f61eed9a8b7251
Signed-off-by: Aniroop Mathur <a.mathur@samsung.com>
Signed-off-by: Aniroop Mathur <aniroop.mathur@gmail.com>

Merge "Sensor: Add batch support for native applications" into nyc-dev

Merge "Fix eglSurfaceAttrib return value" into nyc-dev

Merge "loader: cache top-of-chain PhysicalDevice" into nyc-dev

Add user activity event type for accessibility.

Bug: 19531976
Change-Id: I95096dbf252556e677a8a9f3bffe8abaab353ab4

Merge "Improve logging of child commands." into nyc-dev

Improve logging of child commands.

When execvpe() is called it doesn't return, so the child error handling
should be done at the parent, otherwise the failure is not printed in the bugreport.

BUG: 27583193
Change-Id: I6161e4c72b7b0da7b82bff7fd72d7a5aaf4d6e89

Merge "Otapreopt: Load properties from files" into nyc-dev

loader: cache top-of-chain PhysicalDevice

Bug: 27615575
If a layer wraps the VkPhysicalDevice object the
loader would be unable to figure out what VkInstance
it was associated with since it no longer matches
the VkPhysicalDevice queried from the driver.
Now cache the VkPhysicalDevice's from the top of
the chain as well to resolve this.

Change-Id: I7d04884dc3aa8623ce2f67e4f6575f74b82e2834

Merge "Otapreopt: Remove b_mount_path" into nyc-dev

Otapreopt: Load properties from files

Refactor file parsing into a helper.

Load system properties, completely, from the right sources. Add code
to parse init.environ.rc for environment variables.

Bug: 25612095
Change-Id: I3e5b2ff620ccb374c775ed1922baa0d9b8ed5974

Sensor: Add batch support for native applications

Currently, it is not possible for native applications to use the
sensor in batch mode. As it is a 'C' header file so function
overloading is not possible. Therefore, lets add a new function to
enable the sensor, set the delay and batch timeout, all together.

Change-Id: I2671ba37f48bae63945bf24e9f4ea009d33793f7
Signed-off-by: Aniroop Mathur <a.mathur@samsung.com>

Fix eglSurfaceAttrib return value

Bug 24940410

Change-Id: Ibd6be5861fe48c5d2230d48a7b65ae0b21a01b9b

Merge "atrace:  Adding dumping message to logcat." into nyc-dev

Installd: Make extract-only = interpret-only

Test the impact of verifying and quickening at install time and
post-OTA boot time.

Bug: 26833007
Bug: 27688727
Bug: 27689078
Change-Id: Ib2ebe71747f2b8dc7a2864f3d02b5c2217d29d6c

Otapreopt: Remove b_mount_path

The final chosen design runs the tool in a chroot, no B mount path
required.

Bug: 25612095
Change-Id: Iba920455799da1aff97f759afc784ce43c4bf0b1

DO NOT MERGE BQ: fix some uninitialized variables

Bug 27555981
Bug 27556038

Change-Id: I436b6fec589677d7e36c0e980f6e59808415dc0e

BQ: fix some uninitialized variables

Bug 27555981
Bug 27556038

Change-Id: I436b6fec589677d7e36c0e980f6e59808415dc0e

Merge "Revert "Add a callback thread to ConsumerBase"" into nyc-dev

Merge "Otapreopt: Adapt for actual A/B" into nyc-dev

Revert "Add a callback thread to ConsumerBase"

This reverts commit 22b5702b40eca423171124e69c3d4ad03a3e6900.

Bug 27557176

Change-Id: Icc0e79af82329e6414ed44a99e90631cfcf140fa

Otapreopt: Adapt for actual A/B

Add postinstall script for update_engine hook.

Add otapreopt_chroot as a gateway between installd and otapreopt.
Installd will fork and run otapreopt_chroot, which has the permission
to set up a chroot in /postinstall and run otapreopt from the B
partition.

Bug: 25612095
Change-Id: I4264598da00053ced87c849c738ddc0bc5437304

atrace:  Adding dumping message to logcat.

Change-Id: Ida9e7ce8a05f8c5e44245ad0c383453d2ea7d79f

Added zip location on log message.

BUG: 27660862
Change-Id: I6e9dc816955762fde0d4ddb9ac4cbf12b607cbbd

Merge "installd: Avoid potential use of null 'src'." into nyc-dev

DO NOT MERGE Tracing for binder locks/transactions am: 97e5be36e3
am: 10213888ad  -s ours

* commit '10213888add66cda37de12c002fec27e82c96ba6':
  DO NOT MERGE Tracing for binder locks/transactions

DO NOT MERGE Tracing for binder locks/transactions
am: 97e5be36e3

* commit '97e5be36e3b0256b6b671ff93c1cce870dfeddb3':
  DO NOT MERGE Tracing for binder locks/transactions

installd: Avoid potential use of null 'src'.

In create_cache_path, we have code checking whether 'src' is null,
implying it's possibile it could be null here.  To be safe, we move
our strlen call until after this check.

Change-Id: I6a67cdad8ebf8bdce1562ce5f0cbf98b084807ef

DO NOT MERGE Tracing for binder locks/transactions

Added logic to pull traces from the binder driver.

Change-Id: I9aa00b4650fc3f90d7bd55328dfef11e2514f7b4

Merge "frameworks/native: Make Thread::run threadName argument required" into nyc-dev

Pass -n to netstat.

netstat does lots of DNS lookups, which will slow down the dump
process and creates the risk that the dump will be incomplete.

Bug: 26154009
Change-Id: Ibefa259bf74e85ab8dbdfac76f82c101d47686a7

frameworks/native: Make Thread::run threadName argument required

Bug: 27557176
Change-Id: I009046d04cc127ee280c7b2038c04fab540fc5e6

Merge "svcmgr: don't send reply to one-way txn." into nyc-dev

Merge "omx: add OMX_EventDataSpaceChanged" into nyc-dev

Merge "media: update DescribeColorAspectsParams" into nyc-dev

Merge "Installd: Do not log on missing B artifact" into nyc-dev

Installd: Do not log on missing B artifact

For now, just swallow the warning.

Bug: 25612095
Change-Id: Ib7131464c7eb85c153cb91bf5ff9dc2a6127dd4c

svcmgr: don't send reply to one-way txn.

svcmgr currently assumes that all incoming transactions are synchronous
and expecting a reply.  Change the BR_TRANSACTION handling code to not
send a reply to one-way transactions.

Bug: 27111612
Change-Id: If2e56d3eb53b41d059dbe7d5d10e764343c0262c

Merge "Remove the route command from dumpstate." into nyc-dev

Remove the route command from dumpstate.

Its output is irrelevant and misleading, because:

1. It only looks at the "main" routing table, which is a last
   resort routing table that is trumped by all other networks and
   cannot even be used by non-root users.
2. The proper way to understand device routing is to look at the
   output of "ip rule" and "ip route", both of which are already
   in the bugreport.

Bug: 26154009
Change-Id: Ice19799639daa156a0d4dbb69915031e5582a8ed

Merge "loader: Include instance where required for GIPA" into nyc-dev

Merge "Pass system layers through chain." into nyc-dev

loader: Include instance where required for GIPA

Bug: 27579008
GetInstanceProcAddr is only speced to allow VK_NULL_HANDLE
for the instance for a few calls.

Change-Id: If350e23edc336bff3b499631ba9a68b2921c6969

Return 16 values when sending on-change sensor\'s last event am: 95f13901ed
am: e4be205b7b

* commit 'e4be205b7b64be5d183b3a0f9724ec28a2d7e7cf':
  Return 16 values when sending on-change sensor's last event

Return 16 values when sending on-change sensor\'s last event
am: 95f13901ed

* commit '95f13901edc77650f4bb8d727169e553af28a9d1':
  Return 16 values when sending on-change sensor's last event

Merge "Revert "DispSync: workaround HH hardware vsync issue"" into nyc-dev

Merge "HWC2: Track move of transform into Geometry" into nyc-dev

Revert "DispSync: workaround HH hardware vsync issue"

This reverts commit c734d95c575830ffc97fc5d2ae14b020c95d05de, which was
a workaround for a bug on a device which is now obsolete.

Bug: 25845510
Change-Id: I3244bfd3f758c52dfe76e9d5b0a72bbcffdf5a6f

HWC2: Track move of transform into Geometry

Changes a line where we missed the move of transform into Geometry to
fix the HWC2 build.

Bug: 26454664
Change-Id: Ib44ec8705b81da2aa1b99e61449609371e533297

Merge "Do not queue trivial buffer for unsupported screenshot request." into nyc-dev

Do not queue trivial buffer for unsupported screenshot request.

When screenshot is not supported, we queue an empty buffer to the
requesting producer. Besides returning a useless screenshot, this
overwrites the valuable error code, returning ERROR_OK whenever we
successfully return the trivial buffer.

Instead, refuse to queue the trivial buffer, and return the original
error code.

This replaces the workaround proposed in
I6da5d2fdecdef6c87d4dd7b353e2464678800110

BUG:27505438
Change-Id: I597a9be25071d2a6ddafb7d39cc1b09fb48d5fd0

Merge "ui: Fix Rect::reduce" into nyc-dev

ui: Fix Rect::reduce

- Properly handle the case where the Rect to be excluded is completely
  outside the other Rect.

Bug 27415039

Change-Id: I3331d5b3ab231d023348079c781b194d24ac37dd

Merge "Improved usage of stderr for logging." into nyc-dev

omx: add OMX_EventDataSpaceChanged

This is used internally by IOMX to signal a dataspace change by a surface
encoder.

Bug: 25684127
Change-Id: I42d4ec1db422fa62c251ce197495c6e8ab1430b3

media: update DescribeColorAspectsParams

Bug: 25684127
Change-Id: Id16c50dab510d5277edc0d84f11d0419a46642ae

Pass system layers through chain.

Previously if any layers were activated via properties the layers did not see
those as arguments. After this the layers have a complete list of what layers
are enabled. This is then used in the layers to guarantee ordering.

Bug: 27126420

Change-Id: I3a6a923e2564abc3f7c43b110d892539e4d4bfdb

media: move non-OMX structs to VideoAPI.h from HardwareAPI.h

HardwareAPI.h is to be used by OMX only.
Also add asString support for MediaImage and ColorAspect enums.

Bug: 25684127
Change-Id: I839c2ee0815278cdabe74c7cc9c9dc32cb38df76

Improved usage of stderr for logging.

Dumpstate have some macros that write logs to both ALOG and stderr,
which is useful when the latter is redirected to a file.

But prior to this change, only the log statements from dumpstate.cpp
were using it, and utils.cpp was logging to stderr directly.

This change not only uses MYLOGE for all error messages, but also closes
stderr at the very end (so it doesn't miss statements like the
parameters sent to send_broadcast) and uses MYLOGD for the overall progress.

BUG: 27437378
Change-Id: I373a523e20c98d8f445822103784fa9900c6c684

Merge "Fix warning when force-disconnect but there is no connected API" into nyc-dev

Fix warning when force-disconnect but there is no connected API

It's not really an error, we can return early.

bug: 27295820
Change-Id: I450ca615cf9c33b9aaef9c31eec8217c97516df5

media: move asString into android namespace

Change-Id: I98c1f32de97b5dff0faf06c60127ef1c1b254604

Fix name of argument

Follow-up to change I92cc374b5b6d95ca7755e08b95a9bc9060df2178.

Change-Id: I62cc634ee456c6789b48ab8d8542dffa87f4cc72

Add 'rmprofiles' command to installd

Adds a new command to installd which clears all profile data of
a given package.

Bug: 27516490
Change-Id: I92cc374b5b6d95ca7755e08b95a9bc9060df2178

Merge "egl: Add EGL extension for front buffer auto-refresh mode" into nyc-dev

Merge "Implement the stub for glCreateShaderProgramv" into nyc-dev

libvulkan: Fix double-free, refactor instance destruction

Fixes dEQP-VK.api.object_management.alloc_callback_fail.instance.
Since we were calling DestroyInstance_Bottom from both
CreateInstance_Bottom and CreateInstance_Top failure paths, we were
calling the driver's DestroyInstance twice.

To avoid such bugs, this change clears the driver instance handle to
VK_NULL_HANDLE after calling the driver DestroyInstance.

But the real fix in this change is to make creation and destruction
symmetric. Now DestroyInstance_Bottom only cleans up the things that
were initialized/allocated in CreateInstance_Bottom, and is only
called from CreateInstance_Bottom failure paths and from a dispatched
vkDestroyInstance. Similarly, DestroyInstance_Top and failure paths in
CreateInstance_Top call DestroyInstance (formerly TeardownInstance) to
clean up things initialized/allocated in CreateInstance_Top. The
direct calls from *_Top functions to DestroyInstance_Bottom are gone
-- *_Top functions should only reach *_Bottom functions via dispatch,
so the call goes through enabled layers.

Bug: 27493757
Change-Id: I4e9f8508297813415499dc17803fff49ce9abdcf
(cherry picked from commit 15cd1e269fd2dacef8b95006928b122b9dabbeea)

dumpsys: switch to using a pipe to work around selinux denial.

Some services can't write to the socket that we were passing in, due to
selinux policies. Use a pipe for now.

Bug: http://b/27500248
Change-Id: Ic681121597a1717d31ef356bd6fe2c2daaa86d94

Implement the stub for glCreateShaderProgramv

Bug 26851015

Change-Id: Ide7c46e0b5525d082dc591e92b4a8d8b21665708

Merge "Revert "SF: detachLayer from Client earlier on destruction"" into nyc-dev

Merge "Removed temporary 'v1-dev1-dumpsys-split' version and bumped version to 'v1-dev2'." into nyc-dev

Revert "SF: detachLayer from Client earlier on destruction"

This reverts commit e338df1c2ff4ed2cd575a0e3193ef77c9cd9cb72.

Bug 27330889

Change-Id: I7e62ac2623bd0e1304f36a895ee2b5f9160e2a7e

Merge "Latch transform with geometry state." into nyc-dev