linux-kernel - Re: SLB9670 TPM module crash

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <D1CQGK8GY5UO.179S3ITUXL00Z@kernel.org>
Date: Sat, 18 May 2024 14:27:54 +0300
From: "Jarkko Sakkinen" <jarkko@...nel.org>
To: "Jarkko Sakkinen" <jarkko@...nel.org>, "Parthiban"
 <parthiban@...umiz.com>, <James.Bottomley@...senPartnership.com>
Cc: <linux-integrity@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
 <peterhuewe@....de>, <jgg@...pe.ca>
Subject: Re: SLB9670 TPM module crash

On Sat May 18, 2024 at 2:25 PM EEST, Jarkko Sakkinen wrote:
> On Sat May 18, 2024 at 2:21 PM EEST, Parthiban wrote:
> > Dear James Bottomley,
> >
> > The following crash is observed in the current mainline kernel and I have tried the
> > git bisect to narrow it down. Bisect points to the below commit, which got merged as
> > part of [1]. I tried reverting the below commit and the TPM loads fine.
> >
> > commit 1b6d7f9eb150305dcb0da4f7101a8d30dcdf0497
> > Author: James Bottomley <James.Bottomley@...senPartnership.com>
> > Date:   Mon Apr 29 16:28:07 2024 -0400
> >
> >     tpm: add session encryption protection to tpm2_get_random()
> >     
> >     If some entity is snooping the TPM bus, they can see the random
> >     numbers we're extracting from the TPM and do prediction attacks
> >     against their consumers.  Foil this attack by using response
> >     encryption to prevent the attacker from seeing the random sequence.
> >     
> >     Signed-off-by: James Bottomley <James.Bottomley@...senPartnership.com>
> >     Reviewed-by: Jarkko Sakkinen <jarkko@...nel.org>
> >     Tested-by: Jarkko Sakkinen <jarkko@...nel.org>
> >     Signed-off-by: Jarkko Sakkinen <jarkko@...nel.org>
> >
> >  drivers/char/tpm/tpm2-cmd.c | 21 +++++++++++++++++----
> >  1 file changed, 17 insertions(+), 4 deletions(-)
> >
> > [   11.551988] tpm_tis_spi spi0.1: 2.0 TPM (device-id 0x1B, rev-id 22)
> > [   11.563036] spi_master spi0: will run message pump with realtime priority
>
>
> Explanation and workaround: https://lore.kernel.org/linux-integrity/D1C1KL7Q27P9.39BH0Z4EMBBUG@kernel.org/

Oops completely wrong for this issue! Sorry I overlooked.

So fix is in progress for __request_module() issue. See this
discussion for reference:

https://lore.kernel.org/linux-integrity/119dc5ed-f159-41be-9dda-1a056f29888d@notapiano/

BR, Jarkko