diff options
author | Jarod Wilson <jarod@redhat.com> | 2011-01-28 23:14:35 -0500 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2011-01-28 23:14:35 -0500 |
commit | 18c0ebd2d8194cce4b3f67e2903fa01bea892cbc (patch) | |
tree | 4cc000697114e61cde326ce9f6e09ed6f939240f | |
parent | 2918aa8d1d4e7b4586a5a89dc8406e1d431f5129 (diff) |
crypto: testmgr - mark ghash as fips_allowed
A self-test failure in fips mode means a panic. Well, gcm(aes)
self-tests currently fail in fips mode, as gcm is dependent on ghash,
which semi-recently got self-test vectors added, but wasn't marked as a
fips_allowed algorithm. Because of gcm's dependence on what is now seen
as a non-fips_allowed algorithm, its self-tests refuse to run.
Previously, ghash got a pass in fips mode, due to the lack of any test
vectors at all, and thus gcm self-tests were able to run. After this
patch, a 'modprobe tcrypt mode=35' no longer panics in fips mode, and
successful self-test of gcm(aes) is reported.
Signed-off-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r-- | crypto/testmgr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/crypto/testmgr.c b/crypto/testmgr.c index 521fdb2f7cfd..2854865f2434 100644 --- a/crypto/testmgr.c +++ b/crypto/testmgr.c | |||
@@ -2077,6 +2077,7 @@ static const struct alg_test_desc alg_test_descs[] = { | |||
2077 | }, { | 2077 | }, { |
2078 | .alg = "ghash", | 2078 | .alg = "ghash", |
2079 | .test = alg_test_hash, | 2079 | .test = alg_test_hash, |
2080 | .fips_allowed = 1, | ||
2080 | .suite = { | 2081 | .suite = { |
2081 | .hash = { | 2082 | .hash = { |
2082 | .vecs = ghash_tv_template, | 2083 | .vecs = ghash_tv_template, |