GitLab

For keymaster2, exportKey() will return need upgrade error when
using old patch_level keyblob, an upgradeKeyBlob() call needed in
exportKey(), like begin() and getKeyCharacteristics().

* changes:
  ExportResult implement Parcelable interface
  OperationResult implements Parcelable interface

Only BoringSSL is supported anymore. The OpenSSL code is not tested.

Test: make -j32
Bug: 31464605
Change-Id: I6394bcf71f9e0b17bd8cbb50f6868df03aa00780

memory allocated in readKeymasterBlob() was not freed.
The memory leak happens when the clientId
and appData for getKeyCharacteristics() or exportKey() is non-null.

Change-Id: Ie0340e122bf25a2385175892e5ff8535cca8710a
Signed-off-by: Bin Chen <pierr.chen@gmail.com>

Change-Id: I25427aba88baa1ae3cf9b99813a7f9fb8c70b052
Signed-off-by: Bin Chen <pierr.chen@gmail.com>

The open/write/close/rename pattern can result in
zero-length files on some file systems if crash or power
loss occurs. Prefer open/write/fsync/close/rename.

BUG: https://code.google.com/p/android/issues/detail?id=207607
Change-Id: I2742c3ab967d5cf254b22cffb5164881315623d5

Simplify the code and prepare for the switch to cpp-aidl generated
interface, proxy and stub file.

Change-Id: If4cd831b2bbeaa305393a4732b72231ce596f5ef
Signed-off-by: Bin Chen <pierr.chen@gmail.com>

Simplify the code and prepare for the switch to cpp-aidl generated
    interface, proxy and stub file.

Change-Id: I6873ee376642e62aff8cd88da7bdcb6f16d30df7
Signed-off-by: Bin Chen <pierr.chen@gmail.com>

Simplify the code and prepare for the switch to cpp-aidl generated
interface, proxy and stub file.

Change-Id: Ia18ee6e47df540100f10be013084226dfc5cec7c
Signed-off-by: Bin Chen <pierr.chen@gmail.com>

am: 9c6612e2

Change-Id: Ic137fde859285c3a693b3e4b7642900d2ff34ada

According to Google coding style, no VLA since
it is not part of c++ standard and could blow up
the stack.

Change-Id: Ie79b56cd9a02ecebba76d88af727c6accbcfd45c
Signed-off-by: Bin Chen <pierr.chen@gmail.com>

am: 4f987e61

Change-Id: I18dfe082b23f37795503185fd40521ea35b4d3a9

* Use const reference type for parameters, local variables,
  and for-loop index variables to avoid unnecessary copy.

Bug: 30407689
Bug: 30413223
Bug: 30413862
Change-Id: I3b9383f34e466ca6b5290bad802d535443fd0187
Test: build with WITH_TIDY=1

am: 590a1dad

Change-Id: Ie0f3f1be4ee7ba9fbe249b951b452642c505895a

Declare explicit conversion constructors.

Bug: 28341362
Change-Id: Ibd0d6b7ec63ccb8f910e9da15771ff4d0031f4c4
Test: build with clang-tidy

am: 4ca5d689

* commit '4ca5d689':
  Fix misc-macro-parentheses warnings.

Change-Id: Ib57ef065e1471cc0ed84355827f66a643c2f259f

am: 8f8ed7f3

* commit '8f8ed7f3':
  Fix misc-macro-parentheses warnings.

Change-Id: I16a1cd31d9558f5f35c3ccf34b49cd4af43f69b9

Add parentheses around macro arguments used beside binary operators.

Bug: 28705665
Change-Id: I4ea2db63c18e40dabc10d42bfa5c936a71d6c628

am: e44c3373

* commit 'e44c3373':
  Fix google-explicit-constructor warnings.

Change-Id: I4f11b3d5dd790e7dbd1aad1ffe896aff26781afa

am: e2184060

* commit 'e2184060':
  Fix google-explicit-constructor warnings.

Change-Id: Ic2ac1a5556d2fc746e1b50b11936d9a536783ae0

Bug: 28341362
Change-Id: I116d2667550eec7a080bd5fcf073a8e7206c7c53

am: 1140ef67

* commit '1140ef67':
  Move keystore to foreground cpuset.

Change-Id: I7729240d6f98e9c76e0845c272de639339ea09ce

Bug: 24949295
Change-Id: Iaade68c5216d0efabdfdec4ff2b61d778d319324

am: c5e91c2e

* commit 'c5e91c2e':
  Remove RSA_FLAG_EXT_PKEY from android_engine.cpp.

Change-Id: Ia232e2ce912486f5810bf025fff44cad8974c05a

am: 193059de

* commit '193059de':
  Remove RSA_FLAG_EXT_PKEY from android_engine.cpp.

Change-Id: Ic302ba70c7ab6e8536d5552ae35d7fe511c87c18

Change-Id: I8c3344ad03202f8ce2a8985b135dfcd53d9a6524

am: 52e5773e

* commit '52e5773e':
  Export libbinder headers from libkeystore_binder

Change-Id: I48a9827528d1c950e9497dda9e383a3804a564a6