Re: [PATCH] keys, trusted: fix missing support for TPM 2.0 in trusted_update()

From: Jarkko Sakkinen
Date: Mon Oct 16 2017 - 07:47:09 EST


On Mon, Oct 16, 2017 at 10:53:13AM +0800, Boshi Wang wrote:
> Call tpm_seal_trusted() in trusted_update() for TPM 2.0 chips.
>
> Signed-off-by: Boshi Wang <wangboshi@xxxxxxxxxx>
> ---
> security/keys/trusted.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/security/keys/trusted.c b/security/keys/trusted.c
> index ddfaebf..563fe5f 100644
> --- a/security/keys/trusted.c
> +++ b/security/keys/trusted.c
> @@ -1065,6 +1065,11 @@ static int trusted_update(struct key *key, struct key_preparsed_payload *prep)
> size_t datalen = prep->datalen;
> char *datablob;
> int ret = 0;
> + int tpm2;
> +
> + tpm2 = tpm_is_tpm2(TPM_ANY_NUM);
> + if (tpm2 < 0)
> + return tpm2;
>
> if (test_bit(KEY_FLAG_NEGATIVE, &key->flags))
> return -ENOKEY;
> @@ -1110,7 +1115,10 @@ static int trusted_update(struct key *key, struct key_preparsed_payload *prep)
> dump_payload(p);
> dump_payload(new_p);
>
> - ret = key_seal(new_p, new_o);
> + if (tpm2)
> + ret = tpm_seal_trusted(TPM_ANY_NUM, new_p, new_o);
> + else
> + ret = key_seal(new_p, new_o);
> if (ret < 0) {
> pr_info("trusted_key: key_seal failed (%d)\n", ret);
> kzfree(new_p);
> --
> 2.10.1
>
>
>

Reviewed-by: Jarkko Sakkinen <jarkko.sakkinen@xxxxxxxxxxxxxxx>

The sealing code for TPM 1.2 should be moved to driver/char/tpm (has
been in my backlog for couple of years now).

/Jarkko