Merge "IGnssDebug.hal field fixes" into oc-dev
am: 27c2f822ba

Change-Id: I1577e85983e09b517659c2ec0ced522c997d2000

Merge "IGnssDebug.hal field fixes" into oc-dev

Merge "Add VTS tests for drm+crypto HALs" into oc-dev
am: ebf7f3213b

Change-Id: Ifd1772a32a764cf54fdf9d77d50f22c3431f1d83

Merge "Add VTS tests for drm+crypto HALs" into oc-dev

Add VTS tests for drm+crypto HALs

Tests: drm vts tests are passing

related-to-bug: 32815560

Change-Id: I2b36f27fbb42eba37f3e5a26acea0e359e60b3af

IGnssDebug.hal field fixes

Modifying some fields in this API to align with values
that work with partner implementations, and better
capture the intent of this new API.

Bug: 30955615
Test: Builds and works with hidl-gen
Change-Id: Ied53cbeed2703ce757a246b45339ff2b9dfeb410

Merge "Check result of registerAsService in dumpstate." into oc-dev
am: 5221194a30

Change-Id: I83beb7c1a92135a441b4453931d70d4f5c9deec5

Merge "Check result of registerAsService in dumpstate." into oc-dev

Merge "Simplify EVS HAL and move to "agressive opens"" into oc-dev
am: 9ee6abd72c

Change-Id: I4cb679b8efe5292947d3a06631ca1acdbaf33943

Merge "Simplify EVS HAL and move to "agressive opens"" into oc-dev

Merge "Fix getKeyRequest legacy HAL returning invalid status" into oc-dev
am: 140e6a9600

Change-Id: Ib51672608147fe181f7d8726e7fcc2525bd6f30a

Merge "Fix getKeyRequest legacy HAL returning invalid status" into oc-dev

Fix getKeyRequest legacy HAL returning invalid status

bug: 36006198

Test: drm hal vts test
Change-Id: Ie3125b4459add4fd30dc418f92ab0f51d8fcc73b

Merge "Add VTS tests for Sap Apis." into oc-dev
am: cfb39b413f  -s ours

Change-Id: Ie0a71b5a69cffa899895105d5e10bcba8a6b9d9b

Merge "Enable the clang-format pre-submit hook" am: 3b9e23513e am: 68a27cc5ab
am: 03ac2b1697

Change-Id: I4b26f5ca596b0ed07d28367a58b086fb26459fd8

Merge "Enable the clang-format pre-submit hook" am: 3b9e23513e
am: 68a27cc5ab

Change-Id: Idb9ac5526f341064982be84fb44f523fbef4521d

Merge "Enable the clang-format pre-submit hook"
am: 3b9e23513e

Change-Id: Idf1651f9e76c8b7e6dfbc70dab8ae567a5f53937

Merge "Enable the clang-format pre-submit hook"

Merge "Add VTS tests for Sap Apis." into oc-dev

Simplify EVS HAL and move to "agressive opens"

This adapts the API implementation to allow a duplicate "open" operation
to automatically close any previous connections to the device.  This
works around a binder level issue that can cause destructors triggered
by remote clients to be delivered out of order to the server.

This was originally change ag/1969959 on master, but has been
recreated on oc-dev (cherry-picking was broken at the time).
The original master change will be abandoned in favor of this getting
merged down from oc-dev.

Test:  Run Vts test (added in following change)
Change-Id: I7b417998e59a4d592fbb91811c4101f39097c5dd

Enable the clang-format pre-submit hook

Bug: 36699008
Test: Try to upload a patch with formatting errors.
Change-Id: I4af7d142ccbbaf39eb236547487c59c5ff8f2cd6

Merge "Change sytle of radio and sap tests to clang google" am: 7441c791b6 am: d0de215476
am: 22164f3d4b

Change-Id: Iacedf1a43dacc012bd68e184fc4256b29fed59d5

Merge "Change sytle of radio and sap tests to clang google" am: 7441c791b6
am: d0de215476

Change-Id: I3ad7fd7d584a2b6ffd8a2050680dd4479c9b0343

Merge "Change sytle of radio and sap tests to clang google"
am: 7441c791b6

Change-Id: I0aa6ad8fcf24862278705a1f0a9b43cb08370c80

audiohal: Fix UAF of HAL devices in Stream objects
am: 936279e1ff

Change-Id: Id78777accd072722b04d6b0bdc8bf9c219cec861

Merge "Change sytle of radio and sap tests to clang google"

Merge "Bluetooth: Replace assert() with CHECK()" am: b26fea2629 am: 17c9a90e0a
am: 5953a0a43b

Change-Id: If57ba4fbc2a9426170ed164de638b6546cedd063

Merge "Bluetooth: Replace assert() with CHECK()" am: b26fea2629
am: 17c9a90e0a

Change-Id: Iec662c64cbe65038c6399837b1b4361da6ab8749

Merge "Bluetooth: Replace assert() with CHECK()"
am: b26fea2629

Change-Id: If0285fbce60e230fa947d8400fa0387dbb1bc551

Merge "Bluetooth: Replace assert() with CHECK()"

Check result of registerAsService in dumpstate.

Test: see change I35f5d4498abd7369b03a04081e8aa26a3933328e
Fixes: 36704363
Change-Id: I5cc8f32592a77994e756438b8d1b7f3255acef19

Change sytle of radio and sap tests to clang google

Test: make VtsHalRadioV1_0TargetTest, make VtsHalSapV1_0TargetTest
Change-Id: Ie13efe0d049ade7839881c14ed2904d8acd99b93

Merge "Add VTS tests for Sap Apis." am: eb245db746 am: 4d7ec24e3e
am: 8fe3f47882

Change-Id: Ic2f17907d19f258fd599e35c6e5c8e592e4c0894

Merge "Add VTS tests for Sap Apis." am: eb245db746
am: 4d7ec24e3e

Change-Id: I8f4fdb2b70972d4052dd3b04c63cf37718389f0e

Merge "Add VTS tests for Sap Apis."
am: eb245db746

Change-Id: I6aaeba6da6cb43639e434e2f0ecc10a5731c2b47

Merge "Add VTS tests for Sap Apis."

Bluetooth: Replace assert() with CHECK()

Test: Bluetooth starts/stops, VtsHalBluetoothV1_0TargetTest passes
Change-Id: I1ecdf4ed7507fc8e2f00c35f136cb13e145713e5

audiohal: Fix UAF of HAL devices in Stream objects

Stream objects used to hold a pointer to underlying HAL device
object which they didn't own. Since destruction of server side
objects is asynchronous, it was possible that a Device object
gets destroyed before Stream objects, making all the HAL device
object pointer to become stale.

Fixed by adding a strong reference to Device objects into Stream
objects.

Bug: 36702804
Change-Id: I3da3611afbb91d6fd6410ac5b8af2a2eebfa6dac
Test: ran Loopback app and HAL VTS tests
(cherry picked from commit 96d3573cda6f76bcbfc277e69d94914a565218d8)

Revert "Revert "Add keymaster VTS tests.""
am: 63e15f0790

Change-Id: Ic93a1b3587a0d8edbdc13ca4a831b77ce75d8b14

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify am: 51d023c48e am: c6578f0d30
am: c0af573e8c

Change-Id: I318f0c6cb5de5024de58883b28b968ba22e4d2b2

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify am: 51d023c48e
am: c6578f0d30

Change-Id: I997b319b75d00e69a96b0057de6e0c22867a5258

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify
am: 51d023c48e

Change-Id: Ifaad9a1c7d0994b43c208ca4c7899daab2baf9ec

Revert "Revert "Add keymaster VTS tests.""

This reverts commit 8e10baee45a1bfab4094869159b8ede270b6d416.

Test: Tests pass, except known failures with in-progress vendor fixes
Change-Id: Ia95c1444100e123fa65d095b001878cf9040d0c4

RenderScript VTS cleanup and fixes
am: da8c2c1e2a

Change-Id: I41227692b20c8833134b0130d1ff90d650cf29d9

Fix RS HIDL server, pass data by bytes instead of by elements.
am: 1791d9bfab

Change-Id: I0b6c83825db9bda2f263b3604483e5e798994b15

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify

VtsHalBluetoothV1_0TargetTest uses VtsHalHidlCallbackBase.
Callback class extends VtsHalHidlCallbackBase and use waitForCallback, notifyFromCallback, and setWaitTimeout

The original Bluetooth test contains the following use cases:
  * Wait for specific callback
  * Set timeout for specific callback waiting
The modified version supports the above use cases.

Bug: 36036059
Test: Test pass locally
Change-Id: I961baaf7f1b09fa9c134629cc7280565a2664ed5

Merge "RenderScript VTS cleanup and fixes" am: 7b36175068 am: 1a2850fbc4
am: 643c439e89

Change-Id: I211e1f18f8b35cb7826ae5d390074c520c2ffcc4

Merge "Fix RS HIDL server, pass data by bytes instead of by elements." am: f2b214dc8d am: 1e7e3611e7
am: 815ab992ef

Change-Id: Ida0d3c5cbae560c1587488ef64b5cddfbc07eb10

Merge "RenderScript VTS cleanup and fixes" am: 7b36175068
am: 1a2850fbc4

Change-Id: I1c6e607430a4e597400f0a61a04b98840238567a

Merge "Fix RS HIDL server, pass data by bytes instead of by elements." am: f2b214dc8d
am: 1e7e3611e7

Change-Id: I2e821b90ca667689e2ceb71d618de8ab41ec6af2

Merge "RenderScript VTS cleanup and fixes"
am: 7b36175068

Change-Id: I03e7a64f197e7b47f7f7dbcd893cc3125fc06850

Merge "Fix RS HIDL server, pass data by bytes instead of by elements."
am: f2b214dc8d

Change-Id: I753fcfd32f0837a099bf157c837b6e7759a9ab45

RenderScript VTS cleanup and fixes

This CL fixes the remaining VTS test cases for the RenderScript
vts-hidl-hal test and cleans up and improves the working test cases.
The goal is to have at least one passing test case per HIDL HAL entry.

Bug: 35915961

Test: mm and run on angler
Change-Id: I4b6e25831ce7d2abb3148c39eb48bde10ed04646
(cherry picked from commit 54fa8d428552a6202d196b6abf6a844bb277ac98)

Fix RS HIDL server, pass data by bytes instead of by elements.

  Our current stack:
  API->API_TO_HAL_translator->HAL
          ->HAL_TO_Implementation_translator->Implementation

  For most APIs:
  - API passes objectCount.
  - HAL expects objectCount.
  - Implementation expects objectCount.

  For APIs like ScriptGroupCreate:
  - API passes byteCount. And unfortunately, these APIs are part of
    NDK, we could not make them also passing objectCount like others.
  - HAL expects objectCount.
  - Implementation expects byteCount.

  So that both API_TO_HAL_translator and
  HAL_TO_Implementation_translator should correctly convert input
  objectCount/byteCount to byteCount/objectCount.

  This CL only fixes the HAL_TO_Implementation_translator part,
  whereas aosp/356395 fixes the API_TO_HAL_translator part. Both
  parts were mistakenly using byteCount as objectCount, causing
  potential out-of-bound access.

Bug: 36404879
Test: mm on angler
Change-Id: I28541a8926aeafece40e2a3f664bda67e26a34a2
(cherry picked from commit fd14e27b8997da6b453174af2af2e1cf66e01b5d)

Merge "RenderScript VTS cleanup and fixes"

Merge "Fix RS HIDL server, pass data by bytes instead of by elements."

Fix RS HIDL server, pass data by bytes instead of by elements.

  Our current stack:
  API->API_TO_HAL_translator->HAL
          ->HAL_TO_Implementation_translator->Implementation

  For most APIs:
  - API passes objectCount.
  - HAL expects objectCount.
  - Implementation expects objectCount.

  For APIs like ScriptGroupCreate:
  - API passes byteCount. And unfortunately, these APIs are part of
    NDK, we could not make them also passing objectCount like others.
  - HAL expects objectCount.
  - Implementation expects byteCount.

  So that both API_TO_HAL_translator and
  HAL_TO_Implementation_translator should correctly convert input
  objectCount/byteCount to byteCount/objectCount.

  This CL only fixes the HAL_TO_Implementation_translator part,
  whereas aosp/356395 fixes the API_TO_HAL_translator part. Both
  parts were mistakenly using byteCount as objectCount, causing
  potential out-of-bound access.

Bug: 36404879
Test: mm on angler
Change-Id: I28541a8926aeafece40e2a3f664bda67e26a34a2

Add VTS tests for Sap Apis.

Bug: 32022332
Test: vts
Change-Id: I9a55978aa485a59510090f46a8e5edfb87bccc0e

Add VTS tests for Sap Apis.

Bug: 32022332
Test: vts
Change-Id: I9a55978aa485a59510090f46a8e5edfb87bccc0e

Merge "Revert "Add keymaster VTS tests."" into oc-dev
am: bf0f670580

Change-Id: I28bfff5b7e2a1109d32f43a84cfa362562c13bd5

Merge "Revert "Add keymaster VTS tests."" into oc-dev

Revert "Add keymaster VTS tests."

This reverts commit dda22ea8845f6906d4bccc13bf142d96a43976c5.

Change-Id: Ie6a60e17ed5258840eba3d3f6b8092636160b2c2

Merge "Convert all comments into "doxygen-ready" comments." into oc-dev
am: 8074912d8c

Change-Id: Ic81578a5730c614e31c529b04fe96a2852da89ea

Merge "Convert all comments into "doxygen-ready" comments." into oc-dev

Add keymaster VTS tests.
am: dda22ea884

Change-Id: I6d683d35598134e29cf4f2083050282f6fba2718

Convert all comments into "doxygen-ready" comments.

Bug: 36453077
Test: mma
Change-Id: I0b1f77dfae5d2258969e33d85ecf45401ffbdfaa

Add keymaster VTS tests.

Note: Recent Google devices have keymaster implementations that don't
pass this test suite.  See https://goo.gl/6hsGwa for a summary.

Bug: 32022681
Test: This is the test suite.
Change-Id: Ib200b68e0c7844df02eb9f086385d6c36e306d45

Convert all comments into "doxygen-ready" comments.
am: 675ae49ab1

Change-Id: I86aa5511421ad5bd7ec29776fbc8738b0437219b

Convert all comments into "doxygen-ready" comments. am: 3e4acb8b3a am: 64fcbfe9aa
am: 211c03e585

Change-Id: If163f588c87dff16b2f1173871c85d6ce4b0e422

Convert all comments into "doxygen-ready" comments. am: 3e4acb8b3a
am: 64fcbfe9aa

Change-Id: I9c97cfecf806ebe0e33d9b92c0a5e923cd8c560b

Convert all comments into "doxygen-ready" comments.
am: 3e4acb8b3a

Change-Id: I615de8ebda7c08f29e52dc78f65c5408d3a7d58f

Convert all comments into "doxygen-ready" comments.

Bug: 36453077
Test: make
Change-Id: Ic77394d4d2e1ccedba3a28230d870c5d6108bd40

Convert all comments into "doxygen-ready" comments.

Bug: 36453077
Test: make
Change-Id: Ic77394d4d2e1ccedba3a28230d870c5d6108bd40

Merge "Use gralloc usage conversion library" into oc-dev
am: 2f44ec6c86

Change-Id: I692ea8e24c0d87e7729e52c1e35d7d40eab12a0f

Merge "Use gralloc usage conversion library" into oc-dev

Update makefiles for hwtypes.h
am: 561b602a99

Change-Id: I11a2aa2bfeca6a60bc13f768e98abacc69b97df8

Use gralloc usage conversion library

Converting from ..graphics.allocator@2.0 usage to gralloc0 usage isn't
quite as simple as ORing and truncating, which is what the default
implementation was doing. Switch to using library functions that do it
correctly.

Test: boot bullhead
Change-Id: I40ae00e9aad92b374f281569207972b7461a3e55

Update makefiles for hwtypes.h

Test: pass
Change-Id: I8e17eb8fec5d2f04c526476a866c8dd54ccb556c

Merge "Add missing LOW_LATENCY_OFFBODY_DETECT handling"

Merge "Fingerprint vts tests."

Merge "Revert "Add auth token parsing to IKeymasterDevice.hal""

Revert "Add auth token parsing to IKeymasterDevice.hal"

This reverts commit 62f63c7ddbd08737e298a97975754225e5da0126.

Reason for revert: b/36637075

Bug: 36637075
Change-Id: Ie0e8d0b480047a7c68f266e7e5d8a31722f85128

Merge "Sync the constants in HAL and framework"

wifi(implementation): Stop loading vendor library statically am: 34e4bc4dd2 am: c17e4e07ac
am: 0e5e51175c

Change-Id: Ib1e6cc90a4388e49fda3608fb193e15fb66aef80

wifi(implementation): Stop loading vendor library statically am: 34e4bc4dd2
am: c17e4e07ac

Change-Id: I843085db6544dd9c14feb611b709f48e6284c901

wifi(implementation): Stop loading vendor library statically
am: 34e4bc4dd2

Change-Id: I614a2074fbc2d82618eeb72002724830cf888dab

wifi(implementation): Stop loading vendor library statically

libwifi-hal statically imports the vendor implementation of the legacy
HAL. Since the HIDL daemon is dynamically linking against libwif-hal,
there is no need for it to statically import the vendor implementation
library (one pointed to by: $LIBWIFI-HAL variable).

Bug: 36169790
Test: Compiles & device is able to connect to wifi networks with the
change.

Change-Id: Ie44b58328f9bdde011ba6d4a4d26f786c2d097e7

RenderScript VTS cleanup and fixes

This CL fixes the remaining VTS test cases for the RenderScript
vts-hidl-hal test and cleans up and improves the working test cases.
The goal is to have at least one passing test case per HIDL HAL entry.

Bug: 35915961

Test: mm and run on angler
Change-Id: I4b6e25831ce7d2abb3148c39eb48bde10ed04646

Fingerprint vts tests.

Fix callback mechanism. Add tests for uncovered interfaces.

Test: make vts && adb push
out/host/linux-x86/vts/android-vts/testcases/DATA/nativetest64/VtsHalBiometricsFingerprintV2_1TargetTest/VtsHalBiometricsFingerprintV2_1TargetTest
/data/local/tmp/ && adb shell
./data/local/tmp/VtsHalBiometricsFingerprintV2_1TargetTest
Bug: 32022721
Change-Id: Ibeb955bfb937d9f23f98a2e073235aa745183e94

Sync the constants in HAL and framework

Test: build and flash device.

Bug: 36233336
Change-Id: I8134b8bded5e31561a97938bdf86a43f7c2b6dc5

Merge "audiohal: Add crash tests for some methods of IEffect"

Update makefiles. am: e90c31f125 am: f11981de68
am: 103bada0e1

Change-Id: Ibe6eeeacab5660a8cf70d59dd9a4864d6f70ab80

Update makefiles. am: e90c31f125
am: f11981de68

Change-Id: Icf25d969d83dff9653435d5fff0edb6b274a9d9f

Update makefiles.
am: e90c31f125

Change-Id: I370a9a20b005d7510463594b24e5c4fc8791116a

Update makefiles.

Test: pass
Change-Id: I925baccbe51a32b7b7f3f66cf2918d017ee6b039

Add auth token parsing to IKeymasterDevice.hal

Auth tokens have an unfortunate dual character. To most of the system
they are opaque blobs that are intended only to be obtained from one
HAL (e.g. gatekeeper or fingerprint) and passed to another
HAL (keymaster), but keystore actually needs to extract some bits of
information from them in order to determine which of the available blobs
should be provided for a given keymaster key operation.

This CL adds a method that resolves this dual nature by moving the
responsibility of parsing blobs to the HAL so that no component of the
framework has to make any assumptions about their content and all can
treat them as fully opaque. This still means that the various HAL
implementers have to agree on content, but they also have to agree on an
HMAC key which much be securely distributed to all at every boot, so
asking them to agree on an auth token format is perfectly
acceptable. But now the Android system doesn't have to care about the
format.

Bug: 32962548
Test: CTS tests pass, plus manual testing.
Change-Id: I78aa6e4ea9c5d8f34906b0969909387e2c5894e6

Note that auth token is opaque.

Change-Id: I788ab8c5be630dd818a9a442c487c3451029bf81
Test: N/A, comment-only change
Bug: 34985116

Add missing LOW_LATENCY_OFFBODY_DETECT handling

Bug: 36580454
Test: unlock watch with LLOB sensor and verify no SensorService crash
Change-Id: Ib1d086efd80e829bb844521fa39325f095b1c2c9
Signed-off-by: Ben Fennema <fennema@google.com>

Add digest support and implementation name to getHardwareFeatures

This is needed to support the keystore statistics gathering initiative.
It will allow us to get information about what kinds of keymaster
implementations exist in the ecosystem, and which ones fail in which
ways.

Bug: 36549319
Test: Will add to VTS tests
Change-Id: I49ee4623656060d69a6de7723b11cd715150451a

Merge "Remove viral readproc group dependency."