Improved (or warned about lack of) error handling.

It would be safer for dumpstate to exit when execvp on a child fails; a
common occurrence is when a list of command arguments is missing NULL.

dumpstate should be more robust to detect those missing NULL-terminated
args, but that will be addressed in a future change.

BUG: 27804637
BUG: 27832567
Change-Id: Ibcbe46041a86b16e365fbb40613b8c4bdf39744c

Merge "Installd: Separate profile merging and compilation" into nyc-dev

Merge "Fixed send_broadcast and fork handling issues." into nyc-dev

Fixed send_broadcast and fork handling issues.

send_broadcast() was calling run_command_always(char*, bool, int, char*) passing:
    run_command_always(NULL, 20, true, am_args);

I.e., it was passing "true" (!1) as the boolean, and 1 (true) as the
timeout value; the proper call should be:

    run_command_always(NULL, true, 20, am_args);

Also, the code handling failure on child calls was calling _exit(),
which quits dumpstate - it should simply return the function instead.

BUG: 27804637

Change-Id: If8c9b40eddc8b76f92e6d11078dfe446a39ad4d4

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2 am: 72ca325 am: f6d2288
am: 8c1ee7b

* commit '8c1ee7bbecf3dd96c21ba4b0663bcddb25c0a287':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2 am: 72ca325
am: f6d2288

* commit 'f6d2288e5893b17639dbd36338ffc4a84a184425':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b am: 4e067d2
am: 72ca325

* commit '72ca3255ba44366d93a4e9bf3cbf1672086ef4c6':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44 am: b38df7b
am: 4e067d2

* commit '4e067d26119e6b995b795eb2b0a921b24595ab47':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd am: 5ad8d44
am: b38df7b

* commit 'b38df7b4e9d93686c62d9ec654bba3be6e561d56':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52 am: 3d38d6c am: 6e89bd1
am: e07748c

* commit 'e07748c9520d8a21bde134789c5b4087b02bffef':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours am: 162fecd
am: 5ad8d44

* commit '5ad8d443776e74a56e0e461ae6d333cf52e4f591':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52 am: 3d38d6c
am: 6e89bd1

* commit '6e89bd1aa1844999dccbfa716da82cfef21d434c':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36 am: 1e2ce52
am: 3d38d6c

* commit '3d38d6ca5cc95f9ab82a8dba72065edeba527a62':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17 am: 74d2c4b  -s ours
am: 162fecd

* commit '162fecdc2c98844faaa6cb43a4fbe854f511b077':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33 am: e564a36
am: 1e2ce52

* commit '1e2ce52e97a27f5d9cd68c809f63e56847c658ac':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 41e7b17
am: 74d2c4b  -s ours

* commit '74d2c4b4c97dbbcf43a9f8870007593d61beb547':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2 am: b4f0f33
am: e564a36

* commit 'e564a36b5ca8ef3cacaebb4707b0e8e38b6d7d2d':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6 am: a202396 am: df806c5
am: ecc7934

* commit 'ecc7934d98959a615a8d7cd6c685b7851e1f02f3':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: 12bafe2
am: b4f0f33

* commit 'b4f0f339c714be74cf0ae17e4c88620130b02121':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6 am: a202396
am: df806c5

* commit 'df806c5c9a488f02d385e03bf6ec261bf8ffd879':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder am: ce156f6
am: a202396

* commit 'a202396ca27e8d5a658585861cfcd4277d978362':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder
am: 41e7b17

* commit '41e7b1780f106d2eb4304b1f9cf060ce44177cae':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder
am: ce156f6

* commit 'ce156f60adb32c7cbebe50e313f51e564f93fe99':
  Fix issue #27252896: Security Vulnerability -- weak binder

Fix issue #27252896: Security Vulnerability -- weak binder
am: 12bafe2

* commit '12bafe205c236da50f34ed85adf62a5631525ab6':
  Fix issue #27252896: Security Vulnerability -- weak binder

Installd: Separate profile merging and compilation

Create a new merge_profiles command to check and merge profiles.

(Almost) always pass a profile to a dexopt pass.

Require compilation filter input to dexopt and use it for dex2oat
(except for hard overrides like safe-mode).

Bug: 27689078
Change-Id: I1257857cc15c17e2271d1261ea4cc80752270fcb

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
(manually cherry picked and resolved conflicts from commit
c11146106f94e07016e8e26e4f8628f9a0c73199)

Merge "Flushes stdout when execvp fails." into nyc-dev

Flushes stdout when execvp fails.

BUG: 27804637
Change-Id: Idf414a1d08bf51854475c74767588a4814c8d806

Merge "dumpsys: fix error condition" into nyc-dev

dumpsys: fix error condition

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

Merge "Handled failure of execvp and increases am timeout." into nyc-dev

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 "Add final crop implementation" into nyc-dev

Merge "Delete the entire profile directory when the app is uninstalled" into nyc-dev

Merge "sensorservice: Strengthen compiler warnings." into 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>

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
(cherry picked from commit c11146106f94e07016e8e26e4f8628f9a0c73199)

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
(cherry picked from commit c11146106f94e07016e8e26e4f8628f9a0c73199)

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

sensorservice: Strengthen compiler warnings.

We turn on Wall and Werror for our sensorservice module, and
turn on Wextra for sensorservice and libsensorservice.

This doesn't trigger any new warnings, so this is just a
future-proofing move.

Change-Id: I6574632cd83866e2925697664c33ab9537a4993b

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

Delete the entire profile directory when the app is uninstalled

We were deleting just the files themselves. We need to delete the
directory as well.

Bug: 27696217
Change-Id: I202a52c711a2132ba10b926f4d0e9ec953f25b43

Add debug dump if about to segfault

Bug 27674961

Change-Id: I82242d2c1b7cc9646d233bab896f718652e9ce2c

Add final crop implementation

Bug 26559810

Change-Id: Idaccd13cd625c92d18665ddecebdbb266ea365f3

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 "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 "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