On Tue May 21, 2024 at 8:47 AM EEST, Bharat Bhushan wrote:
>
>
> > -----Original Message-----
> > From: Jarkko Sakkinen <[email protected]>
> > Sent: Tuesday, May 21, 2024 8:47 AM
> > To: Herbert Xu <[email protected]>
> > Cc: [email protected]; [email protected];
> > [email protected]; James Prestwood <[email protected]>;
> > David Woodhouse <[email protected]>; Eric Biggers
> > <[email protected]>; James Bottomley
> > <[email protected]>; Jarkko Sakkinen
> > <[email protected]>; David S. Miller <[email protected]>; open
> > list:CRYPTO API <[email protected]>; open list <linux-
> > [email protected]>; Peter Huewe <[email protected]>; Jason
> > Gunthorpe <[email protected]>; James Bottomley
> > <[email protected]>; Mimi Zohar
> > <[email protected]>; David Howells <[email protected]>; Paul Moore
> > <[email protected]>; James Morris <[email protected]>; Serge E. Hallyn
> > <[email protected]>; open list:SECURITY SUBSYSTEM <linux-security-
> > [email protected]>
> > Subject: [EXTERNAL] [PATCH v2 5/6] tpm: tpm2_key: Extend parser to
> > TPM_LoadableKey
> >
> > ----------------------------------------------------------------------
> > Extend parser to TPM_LoadableKey. Add field for oid to struct tpm2_key
> > so that callers can differentiate different key types.
> >
> > Signed-off-by: Jarkko Sakkinen <[email protected]>
> > ---
> > drivers/char/tpm/tpm2_key.c | 14 +++++++++++---
> > include/crypto/tpm2_key.h | 2 ++
> > security/keys/trusted-keys/trusted_tpm2.c | 4 ++++
> > 3 files changed, 17 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/char/tpm/tpm2_key.c b/drivers/char/tpm/tpm2_key.c
> > index 0112362e432e..59797dc232f1 100644
> > --- a/drivers/char/tpm/tpm2_key.c
> > +++ b/drivers/char/tpm/tpm2_key.c
> > @@ -32,16 +32,24 @@ int tpm2_key_type(void *context, size_t hdrlen,
> > const void *value, size_t vlen)
> > {
> > enum OID oid = look_up_OID(value, vlen);
> > -
> > - if (oid != OID_TPMSealedData) {
> > + struct tpm2_key *key = context;
> > +
> > + switch (oid) {
> > + case OID_TPMSealedData:
> > + pr_info("TPMSealedData\n");
> > + break;
> > + case OID_TPMLoadableKey:
> > + pr_info("TPMLodableKey\n");
These should be pr_debug() (forgot to change).
> > + break;
> > + default:
> > char buffer[50];
> > -
> > sprint_oid(value, vlen, buffer, sizeof(buffer));
> > pr_debug("OID is \"%s\" which is not TPMSealedData\n",
> > buffer);
>
> Maybe extend this print to say "neither TPMSealedData nor TPMLodableKey"
Right, I tried to apply minimal delta to patches where existing code
needs to be carved to a new form :-)
I think it could be just "OID \"%s\" is unknown"?
BR, Jarkko
