Re: [PATCH] crypto: crypto4xx - Remove insecure and unused rng_alg
From: Aleksander Jan Bajkowski <hidden>
Date: 2026-05-31 10:15:52
Also in:
linux-crypto, lkml, stable
Hi Eric, On 30/05/2026 21:26, Eric Biggers wrote:
On Sat, May 30, 2026 at 05:05:19PM +0200, Aleksander Jan Bajkowski wrote:quoted
Hi Eric, On 30/05/2026 00:04, Eric Biggers wrote:quoted
Remove crypto4xx_rng, as it is insecure and unused: - It has only a 64-bit security strength, which is highly inadequate. This can be seen by the fact that crypto4xx_hw_init() seeds it with only 64 bits of entropy, and the fact that the original commit mentions that it implements ANSI X9.17 Annex C.In addition to a seed, the PRNG also uses ring oscillators as sources of entropy. The entropy should be higher than 64b. This is the Rambus EIP-73d IP core. The same IP core is built into eip93 (EIP-73a), eip97 (EIP-73d), and eip197 (EIP-73d). You can find the documentation online. The complete "container" is actually Rambus EIP-94, and one of its parts is EIP-73d.Just because it may have another source of entropy doesn't mean its security strength is higher than 64 bits. I cannot find any documentation other than https://datasheet.octopart.com/PPC460EX-SUB800T-AMCC-datasheet-11553412.pdf which says "ANSI X9.17 Annex C compliant using a DES algorithm". DES actually has a 56-bit key, so maybe I was over-generous. And according to https://cacr.uwaterloo.ca/hac/about/chap5.pdf ANSI X9.17 has only a 64-bit state anyway. So even if we assume the datasheet is incorrect and the algorithm is actually 3DES which has a longer key, the state is likely still 64-bit.
According to the datasheet, there is no second source of entropy. The PRNG has three built-in LFSRs. Each of them can be initialized independently. The first LFSR is used to generate input data. The second and third are used to generate keys for DES encryption. The output of the first LFSR is encrypted using 3DES with two 64-bit keys. Between individual DES operations, data is XORed with the seed. It sounds like a fairly secure design if properly reseeded. There is also a newer design (EIP73a) based on the same algorithm. The only difference is the replacing of 3DES with AES using a 2TDEA scheme. The DES-based variant is more widely used, even in new SoCs.
So it isn't looking good. And since it's an undocumented proprietary design it shouldn't be given the benefit of the doubt either.
As I mentioned earlier, this IP core is quite well documented[1] (page 198). Half of all SOHO routers have the EIP-73d built in. The algorithm is also described in TRM for some of these SoCs :) List od SoCs with EIP-73d: AMCC PPC405EX/PPC460EX, Intel/Maxlinear GRX350, URX850, Marvell Armada 37x0, 7k, 8k, Mediatek MT7623/MT7981/MT7986/MT7987/MT7988, Qualcomm IPQ975x. [1] https://www.scribd.com/document/734250956/Safexcel-Ip-94-Plb-Sas-v1-5?_gl=1*dng4pf*_up*MQ..*_ga*OTQ4NjkzMTAxLjE3ODAyMjA4ODI.*_ga_Z4ZC50DED6*czE3ODAyMjA4ODEkbzEkZzEkdDE3ODAyMjA4ODEkajYwJGwwJGgw*_ga_8KZ8BV0P5W*czE3ODAyMjA4ODEkbzEkZzEkdDE3ODAyMjA4ODEkajYwJGwwJGgw Best regards, Aleksander