Handled failure of execvp and increases am timeout.

The run_command_always() call fork and runs the command in the child
code, and user to have the above logic:

    if (pid == 0) {
        // boiling code
        execvp(command, (char**) args);
        // execvp's result will be handled after waitpid_with_timeout() below...
        _exit(-1); // ...but it doesn't hurt to force exit, just in case
    }

Recently, the _exit(-1) line was removed and there's been reports of
dumpstate hanging after, because it tries to wait for pid 0 to
finish (and in some cases it even tries to kill it).

Also increased the timeout when running am to send broadcasts.

BUG: 27804637
BUG: 27809534
Change-Id: Ia129253e367e5886d305947e56a643fd660a33bd

merge in nyc-release history after reset to nyc-dev

Merge "media: add HDRStaticInfo and DescribeHDRStaticInfoParams" into nyc-dev

Merge "dumpsys: make timeout value an argument" into nyc-dev

dumpsys: make timeout value an argument

Also rework the args parsing using getopt_long.

Bug: 27788406
Change-Id: I60fd8e614616334d47b8de574f3c81a121a6ac0a
Signed-off-by: Thierry Strudel <tstrudel@google.com>

Merge "MediaResourceMonitor: Change argument type from string to int" into nyc-dev

MediaResourceMonitor: Change argument type from string to int

Bug: 27529962
Change-Id: Ic4228aa74c9730134c61a2668932c6ab3b785afc

Merge "Install non-working WSI functions if not enabled" into nyc-dev

Add debug dump if about to segfault

Bug 27674961

Change-Id: I82242d2c1b7cc9646d233bab896f718652e9ce2c

Merge "Add debug dump if about to segfault" into nyc-dev

Install non-working WSI functions if not enabled

b/27537020
These changes make it so that WSI calls dead end if the
application hasn't explicitly enabled them.

Change-Id: I44f95244ec7fd17855daf1c46e9cb6ab4932840f

merge in nyc-release history after reset to nyc-dev

Add debug dump if about to segfault

Bug 27674961

Change-Id: I82242d2c1b7cc9646d233bab896f718652e9ce2c

Fixed typo.

BUG: 27419521

Change-Id: Ie9f0b24876353e532f4ebe2ff4935938c3c7a781

Merge "Don't use su to when calling am or dumpsys." into nyc-dev

Merge "Capture systrace buffers." into nyc-dev

Capture systrace buffers.

When tracing is enabled (/sys/kernel/debug/tracing/tracing_on = 1),
dumpstate will run 'atrace --async_dump' and redirect its output to a
systrace.txt entry in the zip file.

BUG: 27419521
Change-Id: Ia6de46a691b25febac31331fe0aa8701c9a84ebb

Merge "Fix issue #27252896: Security Vulnerability -- weak binder" into nyc-dev

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 in nyc-release history after reset to nyc-dev

Merge "Removed HardwarePropertiesManagerService helper." into nyc-dev

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

Don't use su to when calling am or dumpsys.

su is not available on user builds anymore, hence the bugreport
notifications would never be sent on those builds. Instead, it should
explicitly drop root using system calls.

BUG: 27583193
Change-Id: Ia6256b241fdd6ab4c059fb764b10b4445ad6551d

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 in nyc-release history after reset to nyc-dev

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

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

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

merge in nyc-release history after reset to nyc-dev

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

merge in nyc-release history after reset to 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

media: add HDRStaticInfo and DescribeHDRStaticInfoParams

also add packed attributes to new structures in VideoAPI.h

Bug: 25684127
Change-Id: I97b626b6f0f0c38edfcdd5d3bd17019284c22578

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

merge in nyc-release history after reset to 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

merge in nyc-release history after reset to nyc-dev

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