Merge "Replace cutils/log.h and rationalize log/log.h and android/log.h"

Replace cutils/log.h and rationalize log/log.h and android/log.h

- cutils/log.h treat as deprecated and lead by example
- android/log.h to be used instead of log/log.h if required
- add system includes that are assumed as side effects of log.h
- modules that use SLOG should use SLOG for all logging (installd)
- define LOG_TAG first
- remove logging infrastructure if not used

Test: build
Bug: 31289077
Change-Id: Iea147a0104c7ab7f12451304131d6500f42141e7

Merge "Update for hiding Status from Return<T> object."

Update for hiding Status from Return<T> object.

Test: compiles

Change-Id: Ie32514f34efaa1188f34f662b194b60d83c20bf0

Merge "Only log restorecon changes for existing dirs."

Only log restorecon changes for existing dirs.

If we just created an app private data directory, we're always going
to be relabeling it, so don't log in that case.  Continue logging if
something caused us to relabel an existing app data directory.

Test: builds, boots, only logs for existing dirs
Bug: 31958779
Change-Id: Id03f7c18aed191a4c7fb9dcf9380136c93798a4b

Merge "Remove ~1/3 of the installd calls at boot."

Remove ~1/3 of the installd calls at boot.

When preparing CE storage for an app, we always perform a second call
to extract any newly created CE directory inode.  Let's simplify this
and just return the inode number from the createAppData() call.

Test: builds, boots, reads CE inodes after wipe
Bug: 33463450
Change-Id: I68485e4b9b49e9da7b3ac7c66f50c6abc750b7e3

Merge "Sensor: Add more comments for ASensorEventQueue_registerSensor() function"

Merge "Handle invalid shared GIDs."

Handle invalid shared GIDs.

Shared GIDs are only available inside a range from 50000-59999, so
system UIDs (outside the normal app range) don't have a valid GID to
use.  The best we can do is skip the profile setup in those cases.

Test: builds
Bug: 27948817
Change-Id: I8c38a0e3631eafd07e96da3e37e641b3cce195a6

Merge "libvulkan: add NULL check to DestroySwapchainKHR"

libvulkan: add NULL check to DestroySwapchainKHR

vkDestroySwapchainKHR is required to silently ignore NULL handles
that are provided to it to destroy.
This adds an early return if swapchain_handle is NULL.

Test: dEQP-VK.wsi.android.swapchain.destroy.null_handle (VK 1.0.2 CTS)

Change-Id: Ic230f114a680210cb0e0de931b17e4d70fb27f44

Merge "Fix unused-parameter warning"

Merge "Fix more warnings in framworks/native/opengl/include"

Fix unused-parameter warning

This is exposed when frameworks/av/include is moved from being a system
include directory to a normal include directory.

Test: m -j checkbuild
Change-Id: I127ebe25162f3b6dbb9fc564a6c2c851d0c1636f

Fix more warnings in framworks/native/opengl/include

Remove two more visability attributes being applied to typedefs.
The warnings were being hidden by the use of -isystem to include
frameworks/native/opengl/include.

Bug: 31752268
Test: m -j native

Change-Id: Ic6854d934a8d0d03b23e596d5230a6deb69c86bc

Sensor: Add more comments for ASensorEventQueue_registerSensor() function

Lets add more comments to clarify that ASensorEventQueue_disableSensor is
to be used same as before for deactivating the sensor with respect to
sensor activation with ASensorEventQueue_registerSensor function.

Test: Check clarification in comments / spell check

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

Merge "egl: add EGL_IMG_context_priority to the whitelist"

Merge "Make atrace not depend on /dev/hwbinder."

Merge "atrace: introduce tracing category for adb."

Merge "Split all dexopt-related code into dexopt.cpp."

Make atrace not depend on /dev/hwbinder.

Bug: 31262344
Test: compiles
Change-Id: I2b4fa6815723507dd04afda19e99e5a15053dc66

Merge "Allow both RENDER and TEXTURE flags when creating native client buffers."

Merge "Minor shuffling of installd error reporting."

Split all dexopt-related code into dexopt.cpp.

Only a few minor changes to return bool instead of binder::Status.

Test: builds, boots, apps install/uninstall fine
Change-Id: I538024c0e9da8ca2c0983a6e779c34bd40752053

atrace: introduce tracing category for adb.

Bug: http://b/31289465
Test: systrace
Change-Id: Ida5e898206e5b224771393d2d2ba609183dc3b90

Minor shuffling of installd error reporting.

We actually want to early-return when creating user data fails, and
we want to return an exception when getAppDataInode() is called with
unsupported flags.

Test: builds, boots, apps install/uninstall fine
Bug: 13758960, 30944031
Change-Id: I69816275a0be7a73a8ae7f3d90f2cdea80d8ebfc

Allow both RENDER and TEXTURE flags when creating native client buffers.

Bug: 33057441
Test: Cherrypick from internal repo.
Change-Id: I64ebf6bb5cb534d7f67a9338ebac4bf23a47d607

Merge "Return real error strings from installd."

Return real error strings from installd.

Now that we've moved installd to Binder, we can return nice detailed
error strings explaining why a call failed.  This is particularly
valuable when we record the error message into the PackageManager
persistent log, because up until now those errors were limited to
an unhelpful "installd returned -1" message.

Also perform uniform enforcement of all incoming package name and
UUID arguments.

Test: builds, boots, apps install/uninstall fine
Bug: 13758960, 30944031
Change-Id: Ic1f65ce8c10b1329e01d6a49d72cafa879c4d8bc

Merge "service_manager.c: remove is_selinux_enabled calls"

Merge "Filename refactoring, remove DEXOPT_OTA flag."

Merge "Final push of installd to Binder; goodbye socket!"

service_manager.c: remove is_selinux_enabled calls

SELinux is always enabled. No need to check for it.

Test: Device boots and service manager works without errors.
Change-Id: I12be7336e7c31a5f5796ea1383f3dc934eb3c97c

Filename refactoring, remove DEXOPT_OTA flag.

Test: builds, boots
Bug: 13758960, 30944031
Change-Id: Ib60899fc70b12f76fad9dfad596b513777752703

Final push of installd to Binder; goodbye socket!

Move last two straggling installd commands to Binder and destroy the
socket-based communication channel forever.

Test: builds, boots, apps install fine, pre-OTA dexopt works
Bug: 13758960, 30944031
Change-Id: I2ecd604fb6627b9028a7e32f87d0367e4a4afde3

Merge "Yet another set of installd Binder methods."

Yet another set of installd Binder methods.

Pretty straightforward refactoring.

Test: builds, boots, apps install fine
Bug: 13758960, 30944031
Change-Id: I6e79fe1a3663807c24eb28dca684b539df398c13

Merge "Move more installd methods to Binder."

Merge "Revert "Move installd back to Android.mk.""

Move more installd methods to Binder.

Test: builds, boots, apps install fine
Bug: 13758960, 30944031
Change-Id: I1fcaaaaad34c2f9a3d6a15443ef1764186669865

Revert "Move installd back to Android.mk."

This reverts commit 00b6f68c5c72252542a1e8c5dda4b420ed02c947 and
converts the new entries to Android.bp now that Soong supports AIDL.

Test: mmma -j framework/native/cmds/installd
Test: /data/nativetest64/installd_utils_test/installd_utils_test; fails
      the same before/after
Change-Id: I327eb0bc28fa84cda669ec0df59da82ee10caf1f

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Merge "Shuffle installd Binder implementation around."

Merge "DO NOT MERGE. Check and restorecon cache/code_cache directories."

Merge changes I0c8d4018,Ic9dfb243,I7e442b06
am: 17892731ec

Change-Id: Iadd662bf024e3acd58cf5c747d1546e457483a7b

DO NOT MERGE. Recursively restorecon when SELinux label changes.
am: 7db6041d15  -s ours

Change-Id: Ie64c33ed2eaded73fe043955e0f954bc76f166f7

DO NOT MERGE. Ignore setxattr() failures on tmpfs.
am: 4ed6507cfb  -s ours

Change-Id: I5a8fa5606aaeaa75539a38227e78420acd4bf5bc

DO NOT MERGE. Record "cache" inodes to clear while CE is locked.
am: 9a998f4762  -s ours

Change-Id: Id413a5bceff13dc717fa82209302662b77276165

Merge changes I0c8d4018,Ic9dfb243,I7e442b06

* changes:
  DO NOT MERGE. Recursively restorecon when SELinux label changes.
  DO NOT MERGE. Ignore setxattr() failures on tmpfs.
  DO NOT MERGE. Record "cache" inodes to clear while CE is locked.

Shuffle installd Binder implementation around.

Instead of writing wrapper methods in InstalldNativeService.cpp,
this change shifts the Binder implementation over into the existing
commands.cpp file.  This will let us migrate all methods over, and
then perform one final file rename to InstalldNativeService.cpp.

The downside of this approach is that we no longer have a giant kill
switch to quickly switch back.

Moves the moveCompleteApp() method over to Binder.

Test: builds, boots, apps install fine
Bug: 13758960, 30944031
Change-Id: I53550e05bc2b65155c3de18424f67b1a95450a6f

DO NOT MERGE. Check and restorecon cache/code_cache directories.

To speed up boot times, we recently relaxed SELinux restorecon logic
to only consider relabeling app storage when the top level SELinux
label changed.

However, if an app manually deletes either their cache or code_cache
directories, installd will helpfully recreate those directories at
the next boot, but they'll be stuck with incorrect SELinux labels
which an app can't fix.  (Our historically aggressive restorecons had
relabeled them, which is why we didn't observe until now.)

This change checks the labels of the cache/code_cache directories,
and runs a restorecon if needed, fixing the issue above.

Test: delete cache and verify recreated with correct label
Bug: 32504081
Change-Id: I0114ae4129223e5909b1075d56a9b1145ebc5ef4

DO NOT MERGE. Recursively restorecon when SELinux label changes.

PackageManager has been pretty aggressive about asking installd to
restorecon over app data when it thinks something might have
changed.  However, in the vast majority of cases these are no-op
requests, and we waste a bunch of time recursively walking all
private data, easily costing 60+ seconds on dogfooder devices.

This change updates the initial "create_app_data" command to kick off
a recursive restorecon if it detects that the top-level SELinux label
on the app private data directory changes.  The "create_app_data"
command is designed to ensure that an app's storage is ready, so
PackageManager always calls it at least once per boot before apps
can run.  (This change means that PackageManager no longer needs to
make separate "restorecon_app_data" calls.)

Test: booted, verified that a label change triggered restorecon
Bug: 30768146
Change-Id: I0c8d4018cf8ff888d0ae07a82adc3d61a6002aad

DO NOT MERGE. Ignore setxattr() failures on tmpfs.

Bug: 30210614
Change-Id: Ic9dfb2439b4b1056e1a58552be2e84f4e524321e

DO NOT MERGE. Record "cache" inodes to clear while CE is locked.

There are two situations where we need to clear cached data from
all users, including those whose CE storage might still be locked:

1. When PackageManager is clearing caches to try making room for
package updates.
2. When the device fingerprint changes, we need to clear code caches
for all apps.

To enable this, we now record the inode number of the "cache" and
"code_cache" directories in xattrs of the parent directory.  This is
just enough information to find the cache directories to enable
deleting files inside.  When preparing CE storage for an app, we now
create these two directories and immediately write the inode numbers.

Bug: 26056125
Change-Id: I7e442b0676a695acf962593469793a93b03c8aee

Merge "Check and write some error in atrace."
am: 7b50c02030

Change-Id: I562794bbe7dc52268ad2b477e0dca58a7b79d365

Merge "Check and write some error in atrace."

Merge "Convert vulkan to Android.bp"
am: e421c0710f

Change-Id: Icd0b77b032179c8e8ce5c6009848e23916c990d5

Merge "Convert vulkan to Android.bp"

Merge changes from topic 'bind'
am: 7726057707

Change-Id: I6e0198eceaa85c97bf3b72791e4e5e1df5086091

Initial Binder interface for installd.
am: 9087400f3c

Change-Id: I4912defb2edafb60d429d90679bffe88589fcad9

Convert vulkan to Android.bp

See build/soong/README.md for more information.

Test: mma -j
Change-Id: Ic7f0af683a5ef093df718ed746a73a7fa2a9ce0b

Move installd back to Android.mk.
am: 00b6f68c5c

Change-Id: Iad4819473f6b25f6ac7aa31e23661b564cd49f0e

Merge changes from topic 'bind'

* changes:
  Initial Binder interface for installd.
  Move installd back to Android.mk.

Check and write some error in atrace.

Bug: 33305299

Test: compiles
Change-Id: I88c7ede6cbf68ca10d980e448456bb4ef2a73592

Initial Binder interface for installd.

Define an AIDL for installd, starting with a single method.  Publish
that interface for the system to start using.  We'll circle back and
migrate more methods in future CLs.

Migrating installd to Binder will allow us to get rid of one of
the few lingering text-based command protocols, improving system
maintainability and security.

Test: builds, boots
Bug: 13758960, 30944031
Change-Id: Icdb5320082567e0355d8f76f413f01cfacf0fb99

Move installd back to Android.mk.

An upcoming CL will be compiling an AIDL for installd, which the new
build system doesn't support yet.

Test: builds, boots
Bug: 13758960, 30944031
Change-Id: I47f9ef21eb6040072c402d40ae42a95d7bd0e61d

Merge "Fix includes; use IBase instead of IBinder."
am: bd5dc9df53

Change-Id: Ic75eab82059ff0d1bf483f11d55fb9a6a74dd539

Merge "Fix includes; use IBase instead of IBinder."

Fix includes; use IBase instead of IBinder.

Test: mma
Change-Id: I2604772ff952ed451cc87be330ed131452db8eff

Merge "Check for errors for IServiceManager::get and list."
am: a09202e9ac

Change-Id: I530107b1067810c4107cf22f4408d910190c9f7b

Merge "Check for errors for IServiceManager::get and list."

egl: add EGL_IMG_context_priority to the whitelist

In the CCD 7.0 document EGL_IMG_context_priority was added as an
mandatory egl extension devices must support. However, right now the
extension string is not returned to applications even if the graphics
driver supports it.

Fix this by adding the extension to the whitelist.

Test: check for the extension string returned by eglQueryString

Change-Id: I7833a4e1782e12d845207b234431e10e09c085dc
Signed-off-by: Christian Poetzsch <christian.potzsch@imgtec.com>

Check for errors for IServiceManager::get and list.

Test: compiles
Change-Id: I65b858b1ebd5d45739db05fcadb58ae48fd3337e

Merge "surfaceflinger:Fix potential crash issue when do AIDL FUZZ test"
am: 2792ff4527

Change-Id: Ia8c1ac3f9eff3680ed64235cb01043d0b7c434b9

Merge "surfaceflinger:Fix potential crash issue when do AIDL FUZZ test"

surfaceflinger:Fix potential crash issue when do AIDL FUZZ
test

There will be trigger exception when we provide invalid
inputs parameter for surfaceflinger  JNI interface

Test: 1.Run adb shell and open AIDLer tools;
      2.change to Dir /sdcard/elong.AIDLer
      3.run commond service list > service_list.txt
      4.Rerun AIDL FUZZ test

Change-Id: I0a63adc1e8ae5d9c9e43cb6df6aa13503c40a0c5

Merge "Adjust installd with new dexopt return values."
am: b29c82b67e

Change-Id: Ib0f3811c5891d6871d7b9d44c615954b06f261c6

Merge "Adjust installd with new dexopt return values."

Adjust installd with new dexopt return values.

Bug: 30937355
Test: device boots, mimic system update, apps get updated.
Change-Id: I9539892dfcdd0cc722a292140accd7525ee132d0

Merge "Fix clang static analyzer warnings."
am: b10d705f51

Change-Id: Ibdd7df4af9766b5d00515254cbb8d5e9370a4971

Merge "Fix clang static analyzer warnings."

Merge "atrace: support HAL services as well."
am: 1aea108920

Change-Id: Iac0430699cd15f59ed68b41d20010165a50a50e2

Merge "atrace: support HAL services as well."

atrace: support HAL services as well.

Also poke services that are only registered with
hwbinder, so that they update their props (trace
categories) immediately.

Bug: 31262344
Test: builds, atrace works
Change-Id: I355ddd5f950acc27d598cd972e63f9711426b943

Merge "atrace.rc: move to post-fs trigger"
am: 3749a60b42

Change-Id: Ib2675cf4047d6ce1b9b6285538bb7f7be7ed1649

Merge "atrace.rc: move to post-fs trigger"

atrace.rc: move to post-fs trigger

Test: manually
Bug: 32025203
Change-Id: I787984fc8ab7151ecce9e62a25fa03b0ecf263cd

Merge "Fix warnings in framworks/native/opengl/include"
am: d12c9369d5

Change-Id: Ifbbccbd890d3ab80910586c52f263e903e36dbbd

Merge "Fix warnings in framworks/native/opengl/include"

Merge "Revert "Workaround alloc-dealloc-mismatch error on ASan device boot.""
am: 5c58020c00

Change-Id: I107ce2043fd261b08413418825daf6c35f808228

Merge "Revert "Workaround alloc-dealloc-mismatch error on ASan device boot.""

Merge "Revert "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat."""
am: eda1ddf518

Change-Id: Idbc246744abd62d0bb647f569968457cf5b85583

Merge "Revert "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat."""

Revert "Workaround alloc-dealloc-mismatch error on ASan device boot."

This reverts commit 22819276b147e34a80e8ca696be5b3fda68764b9.

The underlying kernel bug has been fixed.

Test: SANITIZE_TARGET=address build should continue working

Change-Id: I6f3938047d9f60357e7bd9bccd3ef36d7686c73b

Revert "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat.""

Test: device boots, adb shell cmd package compile works.

This reverts commit 5373d3028d53df00c244802f89c8a31a8c1a55a6.

bug:30937355

Change-Id: Iddab9cc8de866a281672fc3c509d43070c3f56de

Merge "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat.""
am: 80a4f63d7e

Change-Id: I02271ca0bbb3ee1d6eb5a8f73d6af0f23e0ab058

Merge "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat.""

Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat."

Mishandled create_vdex_filename.

bug:30937355

This reverts commit 196539e9ea6b096826c3489700588c286d8ca9c2.

Change-Id: Idd1a9f9567240899a103547a36dd036219f303cd