Add DeleteKey and DeleteAllKeys test to Keymaster VTS tests

The tests added verify that a generated key is no longer usable
after it was
 - explicictely deleted with deleteKey, or
 - all keys where deleted with deleteAllKeys.

Bug: 32022681
Test: make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
      vts-tradefed run commandAndExit vts --skip-all-system-status-check
      --primary-abi-only --skip-preconditions
      --module VtsHalKeymasterV3_0TargetTest -l INFO
Change-Id: I172ee5d3ccf1850c3c92d3a1a9e991f46b07a0e3

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

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

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

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

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

Check result of registerAsService in dumpstate.

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

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

This reverts commit 8e10baee45a1bfab4094869159b8ede270b6d416.

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

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)

Add VTS tests for Sap Apis.

Bug: 32022332
Test: vts
Change-Id: I9a55978aa485a59510090f46a8e5edfb87bccc0e

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

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.

Bug: 36453077
Test: make
Change-Id: Ic77394d4d2e1ccedba3a28230d870c5d6108bd40

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

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

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

audiohal: Add crash tests for some methods of IEffect

For methods that can't be meaningfully tested, add tests
verifying that they at least don't crash.

Bug: 32022706
Change-Id: I234c2ae4b6bc173450cf50e8749ea966cbc2d2a2
Test: this is a test

Merge "Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation" am: 2d0b636b29 am: 0efe019342
am: 56fc4104b9

Change-Id: Ia37f47663ce7932c9d294512b7af011849e27c97

Merge "Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation" am: 2d0b636b29
am: 0efe019342

Change-Id: I6a2cc30754413eccb59ba775c14863852a89fefc

Merge "Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation"
am: 2d0b636b29

Change-Id: I342068f8bbcf6a97c322bcf0049fed83b519bcda

Merge "Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation"

Merge "[WIFI] Restore initialization code" am: 8f4416aabb am: 3ae14f0ead
am: 058ed46d36

Change-Id: I8c2a70819cf1acf8b6edd8743f7c536a580ce66a

Merge "[WIFI] Restore initialization code" am: 8f4416aabb
am: 3ae14f0ead

Change-Id: I96f5c77e7f9ee3b84f4a6cf7973638d3d4bba68d

Merge "Add background scan routines and callbacks to broadcast radio HAL."

Merge "Improved GNSS HAL VTS test coverage"

Merge "[WIFI] Restore initialization code"
am: 8f4416aabb

Change-Id: I9f0f828d542e337f185c4a423748f57ef53a00ce

Merge "[WIFI] Restore initialization code"

Merge "Handle hidl transaction errors in ConfigStore"

Merge "removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)" am: d5934d06f1 am: 1d5ccbd5ea
am: a0032e3990  -s ours

Change-Id: I9281cd8f52e698505d921d4f1094c823ab102b1d

Merge "removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)" am: d5934d06f1
am: 1d5ccbd5ea

Change-Id: Idccb5e03b143ec2462479e32d0bd20555d32cd63

Merge "removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)"
am: d5934d06f1

Change-Id: I958c1285dbb5e3c6db8145ac743cedfb61cd544c

Merge "removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)"

[WIFI] Restore initialization code

Restore the usage of WIFI-specific main function. The main
function was inadvertantly disabled by moving it into a static
library - which caused the default main to be used.

Need a custom main since need to stop WIFI before tests start
to enforce a clean starting point.

Bug: 35276551
Test: confirms custom main routine is called
Change-Id: I978827bc0f0f16d9ca4f159fbf42d0bc53f5df99

Merge "Verify all radio classes, instead of FM only."

Improved GNSS HAL VTS test coverage

Calling all extension APIs, and checking some exist
Calling delete and inject methods
Checking measurement capabilties

Bug:32022567
Test: Passes on Pixel XL with O
Change-Id: I86d5416b08c91e4d96eff6ba8f88e50e0544540b

Add background scan routines and callbacks to broadcast radio HAL.

Bug: b/34054813
Test: existing VTS pass, none added.
Change-Id: I2009e73a9e5edc4752366fd84f67b5a745856441

Remove viral readproc group dependency.

This has been copy pasted and isn't actually required.

Test: all hals manually tested to work on internal marlin (which have them binderized).
  Note: tv hals not tested (and they never have been). Filed b/36562029
  Note: for thermal hal, could only run VTS/check logs for errors.
Test: lshal outputs proper stuff
Fixes: 31928447
Change-Id: Id1be584dc0fa2d70e9189b922335146bf6d1382d

Merge "Nfc: Null ptr checks in HAL" am: 3b4d423d42 am: 0ee41f6f8c
am: 87e7c64385

Change-Id: I8706702a3f55ecb6c034e8aa02b3ab4011c2c006

Merge "Nfc: Null ptr checks in HAL" am: 3b4d423d42
am: 0ee41f6f8c

Change-Id: Ifc05d12c32e646ceb0f2e810a23d205cdfe7cdc7

Merge "Nfc: Null ptr checks in HAL"
am: 3b4d423d42

Change-Id: Icede34b74cd253d6921ce38c5abb0d3f80ee1bd7

Merge "Nfc: Null ptr checks in HAL"

Handle hidl transaction errors in ConfigStore

Test: surface flinger not failing, android now bootsup

Bug: b/36445794
Change-Id: I22fa7aab9fa92bc04333aaa0eef45891ebeba8e7

Verify all radio classes, instead of FM only.

Also, a bit of code cleanup.

Bug: b/33382424, b/32022455
Test: VTS
Change-Id: Ib314027b256a6ca4bfa75e9de2ba70990a00b4da

Merge "Camera: Add doc for combined stream usage flags"

removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)

HIDL_FETCH_... is required only for passthrough mode, but Configstore
does not provide passthrough mode.

Bug: 35325577
Test: build and check configstore works

Change-Id: I35e916ad4a6c321303a3888dabb4884b17146e16
Merged-In: Iac3ed57568f18c3a8a7f7423a361b0fe45aa54ab

removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)

HIDL_FETCH_... is required only for passthrough mode, but Configstore
does not provide passthrough mode.

Bug: 35325577
Test: build and check configstore works
Change-Id: Iac3ed57568f18c3a8a7f7423a361b0fe45aa54ab

Merge "Add logic for VHAL versions > 2.0 to forward property information to 2.0 Use this infrastructure to move OBD2_LIVE_FRAME support over to the Vehicle HAL 2.0 layer."

Merge "audiohal: Remove stale TODOs"

Merge "[AWARE] Add Passphrase data-path security" am: 9e71d7d44f am: 5a05398da8
am: e3ee58a4e7

Change-Id: I86680b7dc7285658d5b71a8e8bc5fbf504522b96

Merge "Bluetooth: Protect pointers and callbacks" am: 54906b6ccf am: ce78f9faab
am: 4cfd84a049

Change-Id: I6325f5c93c1edd58c7813c5ab7bec9bb8b3f6aef

Merge "[AWARE] Add Passphrase data-path security" am: 9e71d7d44f
am: 5a05398da8

Change-Id: I5ae40c7077410ad34c3face4aa1fad62dfe5887c

Merge "Bluetooth: Protect pointers and callbacks" am: 54906b6ccf
am: ce78f9faab

Change-Id: I6500b50c6158a2f2d577e223e598748f80d8a0c4

Merge "[AWARE] Add Passphrase data-path security"
am: 9e71d7d44f

Change-Id: I4f0e514d2f49156942c6818379347c369c79889b

Merge "Bluetooth: Protect pointers and callbacks"
am: 54906b6ccf

Change-Id: Idb2e6f6fc796c95552b112060b9866a81073dc37

Merge "[AWARE] Add Passphrase data-path security"

Merge "Add manufacturer and model to device ID attestation"

Merge "Bluetooth: Protect pointers and callbacks"

Nfc: Null ptr checks in HAL

Adds null pointer checks in the default implementation.

Test: compiles
Bug: 36149477
Change-Id: I6195ce92f5cefd4b82874866d08014830f400192
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Merge "use toString instead of to_string" am: 6a73f61b96 am: a1b886fd90
am: 35937ab9fa

Change-Id: I0cb0cbff94b206655061c078848c0ff95147fcfc

Merge "use toString instead of to_string" am: 6a73f61b96
am: a1b886fd90

Change-Id: I5dda7824fcd2988f349a5d1c9df680f9be6700cb

Merge "use toString instead of to_string"
am: 6a73f61b96

Change-Id: Ia28bfdf70b7a79c96822aa3f0402bc0290ad7b3d

Merge "use toString instead of to_string"

Merge "Add VTS for couple of missing APIs and some cleanup for earlier APIs." am: 60449e68d1 am: 516a71e295
am: 0f8a067a04

Change-Id: Ifbdbf6c739e140eb10c8fafdeaa31b92f281a41b

Merge "Add VTS for couple of missing APIs and some cleanup for earlier APIs." am: 60449e68d1
am: 516a71e295

Change-Id: I55111cddfd5101bfdccc82bdfa07013187d6f95f

Merge "Add VTS for couple of missing APIs and some cleanup for earlier APIs."
am: 60449e68d1

Change-Id: I0bcc97f30baf408e6eb6b96f7f37b3b536953fc7

Bluetooth: Protect pointers and callbacks

Test: kill com.android.bluetooth
Bug: 36150800
Change-Id: I9533731c9ce3776c84e6c1d3fdf3e266b328a084

Merge "Add VTS for couple of missing APIs and some cleanup for earlier APIs."

Merge "Check for unimplemented keymaster function delete_key"

use toString instead of to_string

Bug: 35703683
Test: hidl_test_java passes

Change-Id: Ib7d6807f6b9ddcd7b0271b17a0a52be0ee07ec6c