Skip to content

AesCtrDecryptor: Move TEE_crypto_init() call into AesCtrDecryptor obj… - #1

Open
petegriffin wants to merge 7 commits into
masterfrom
sdp-mediadrm-support
Open

AesCtrDecryptor: Move TEE_crypto_init() call into AesCtrDecryptor obj…#1
petegriffin wants to merge 7 commits into
masterfrom
sdp-mediadrm-support

Conversation

@petegriffin

Copy link
Copy Markdown
Owner

…ect.

TEE_crypto_init() allocates memory for IV and key currently.
It is linked to the lifetime of the AES decryptor object
so makes no sense to be called from SessionLibrary object.

Also this change allows the AesCtrDecryptorUnitTests to be run
without any modification making maintaiing the OPTEE ClearKey
and rebasing to new versions of AOSP easier.

Signed-off-by: Peter Griffin peter.griffin@linaro.org

Also call TEE_copy_secure_memory() function
if running in secure mode. This allows unencrypted
data to be copied into a secure buffer by the TEE.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
…ect.

TEE_crypto_init() allocates memory for IV and key currently.
It is linked to the lifetime of the AES decryptor object
so makes no sense to be called from SessionLibrary object.

Also this change allows the AesCtrDecryptorUnitTests to be run
without any modification making maintaiing the OPTEE ClearKey
and rebasing to new versions of AOSP easier.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
@petegriffin
petegriffin force-pushed the sdp-mediadrm-support branch from 0c1444d to d949196 Compare April 5, 2018 08:31
Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Because aosp default sw clearkey plugin may also be installed,
it is useful to have the plugin name in the logs to ensure
you are using the correct plugin.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant