From: Stephan Mueller Subject: [PATCH v2 6/6] Add DRBG test code to testmgr Date: Mon, 17 Mar 2014 08:39:39 +0100 Message-ID: <2717703.La8pfUukd1@myon.chronox.de> References: <2396177.vxvG2ljJL8@myon.chronox.de> <3319954.xS7rx9A3Ha@myon.chronox.de> <2836686.FjyLUn6aXI@myon.chronox.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: aquini@redhat.com, jeremy.wayne.powell@gmail.com To: linux-kernel@vger.kernel.org, linux-crypto@vger.kernel.org Return-path: In-Reply-To: <2836686.FjyLUn6aXI@myon.chronox.de> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org The DRBG test code implements the CAVS test approach. As discussed for the test vectors, all DRBG types are covered with testing. However, not every backend cipher is covered with testing. To prevent the testmgr from logging missing testing, the NULL test is registered for all backend ciphers not covered with specific test cases. All currently implemented DRBG types and backend ciphers are definined in SP800-90A. Therefore, the fips_allowed flag is set for all. Changes to v1: * Changes due to modification of drbg.c as documented in PATCH 1 * Fix coding style and apply scripts/checkpatch.pl Signed-off-by: Stephan Mueller --- diff --git a/crypto/testmgr.c b/crypto/testmgr.c index 7795550..baa6cb7 100644 --- a/crypto/testmgr.c +++ b/crypto/testmgr.c @@ -27,6 +27,7 @@ #include #include #include +#include #include "internal.h" @@ -108,6 +109,11 @@ struct cprng_test_suite { unsigned int count; }; +struct drbg_test_suite { + struct drbg_testvec *vecs; + unsigned int count; +}; + struct alg_test_desc { const char *alg; int (*test)(const struct alg_test_desc *desc, const char *driver, @@ -121,6 +127,7 @@ struct alg_test_desc { struct pcomp_test_suite pcomp; struct hash_test_suite hash; struct cprng_test_suite cprng; + struct drbg_test_suite drbg; } suite; }; @@ -1712,6 +1719,101 @@ static int alg_test_cprng(const struct alg_test_desc *desc, const char *driver, return err; } + +static int drbg_cavs_test(struct drbg_testvec *test, int pr, + const char *driver, u32 type, u32 mask) +{ + int ret = -EAGAIN; + struct crypto_rng *drng; + struct drbg_test_data test_data; + struct drbg_string addtl, pers, testentropy; + unsigned char *buf = kzalloc(test->expectedlen, GFP_KERNEL); + + if (!buf) + return -ENOMEM; + + drng = crypto_alloc_rng(driver, type, mask); + if (IS_ERR(drng)) { + printk(KERN_ERR "alg: drbg: could not allocate DRNG handle for" + "%s\n", driver); + kzfree(buf); + return -ENOMEM; + } + + test_data.testentropy = &testentropy; + drbg_string_fill(&testentropy, test->entropy, test->entropylen); + drbg_string_fill(&pers, test->pers, test->perslen); + ret = crypto_drbg_reset_test(drng, &pers, &test_data); + if (ret) { + printk(KERN_ERR "alg: drbg: Failed to reset rng\n"); + goto outbuf; + } + + drbg_string_fill(&addtl, test->addtla, test->addtllen); + if (pr) { + drbg_string_fill(&testentropy, test->entpra, test->entprlen); + ret = crypto_drbg_get_bytes_addtl_test(drng, + buf, test->expectedlen, &addtl, &test_data); + } else { + ret = crypto_drbg_get_bytes_addtl(drng, + buf, test->expectedlen, &addtl); + } + if (ret <= 0) { + printk(KERN_ERR "alg: drbg: could not obtain random data for" + "driver %s\n", driver); + goto outbuf; + } + + drbg_string_fill(&addtl, test->addtlb, test->addtllen); + if (pr) { + drbg_string_fill(&testentropy, test->entprb, test->entprlen); + ret = crypto_drbg_get_bytes_addtl_test(drng, + buf, test->expectedlen, &addtl, &test_data); + } else { + ret = crypto_drbg_get_bytes_addtl(drng, + buf, test->expectedlen, &addtl); + } + if (ret <= 0) { + printk(KERN_ERR "alg: drbg: could not obtain random data for" + "driver %s\n", driver); + goto outbuf; + } + + ret = memcmp(test->expected, buf, test->expectedlen); + +outbuf: + crypto_free_rng(drng); + kzfree(buf); + return ret; +} + + +static int alg_test_drbg(const struct alg_test_desc *desc, const char *driver, + u32 type, u32 mask) +{ + int err = 0; + int pr = 0; + int i = 0; + struct drbg_testvec *template = desc->suite.drbg.vecs; + unsigned int tcount = desc->suite.drbg.count; + + if ((0 == memcmp(driver, "drbg(pr(", 8)) || + (0 == memcmp(driver, "drbg_pr_", 8))) + pr = 1; + + for (i = 0; i < tcount; i++) { + err = drbg_cavs_test(&template[i], pr, driver, type, mask); + if (err) { + printk(KERN_ERR "alg: drbg: Test %d failed for %s\n", + i, driver); + err = -EINVAL; + break; + } + } + return err; + +} + static int alg_test_null(const struct alg_test_desc *desc, const char *driver, u32 type, u32 mask) { @@ -2273,6 +2375,171 @@ static const struct alg_test_desc alg_test_descs[] = { .alg = "digest_null", .test = alg_test_null, }, { + .alg = "drbg(nopr(ctr(aes128)))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_nopr_ctr_aes128_tv_template, + .count = ARRAY_SIZE(drbg_nopr_ctr_aes128_tv_template) + } + } + }, { + .alg = "drbg(nopr(ctr(aes192)))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_nopr_ctr_aes192_tv_template, + .count = ARRAY_SIZE(drbg_nopr_ctr_aes192_tv_template) + } + } + }, { + .alg = "drbg(nopr(ctr(aes256)))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_nopr_ctr_aes256_tv_template, + .count = ARRAY_SIZE(drbg_nopr_ctr_aes256_tv_template) + } + } + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(nopr(hmac(sha256))) test */ + .alg = "drbg(nopr(hmac(sha1)))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + .alg = "drbg(nopr(hmac(sha256)))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_nopr_hmac_sha256_tv_template, + .count = + ARRAY_SIZE(drbg_nopr_hmac_sha256_tv_template) + } + } + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(nopr(hmac(sha256))) test */ + .alg = "drbg(nopr(hmac(sha384)))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(nopr(hmac(sha256))) test */ + .alg = "drbg(nopr(hmac(sha512)))", + .test = alg_test_null, + .fips_allowed = 1, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(nopr(sha256)) test */ + .alg = "drbg(nopr(sha1))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + .alg = "drbg(nopr(sha256))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_nopr_sha256_tv_template, + .count = ARRAY_SIZE(drbg_nopr_sha256_tv_template) + } + } + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(nopr(sha256)) test */ + .alg = "drbg(nopr(sha384))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(nopr(sha256)) test */ + .alg = "drbg(nopr(sha512))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + .alg = "drbg(pr(ctr(aes128)))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_pr_ctr_aes128_tv_template, + .count = ARRAY_SIZE(drbg_pr_ctr_aes128_tv_template) + } + } + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(aes128)) test */ + .alg = "drbg(pr(ctr(aes192)))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(aes128)) test */ + .alg = "drbg(pr(ctr(aes256)))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(hmac(sha256))) test */ + .alg = "drbg(pr(hmac(sha1)))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + .alg = "drbg(pr(hmac(sha256)))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_pr_hmac_sha256_tv_template, + .count = ARRAY_SIZE(drbg_pr_hmac_sha256_tv_template) + } + } + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(hmac(sha256))) test */ + .alg = "drbg(pr(hmac(sha384)))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(hmac(sha256))) test */ + .alg = "drbg(pr(hmac(sha512)))", + .test = alg_test_null, + .fips_allowed = 1, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(sha256)) test */ + .alg = "drbg(pr(sha1))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + .alg = "drbg(pr(sha256))", + .test = alg_test_drbg, + .fips_allowed = 1, + .suite = { + .drbg = { + .vecs = drbg_pr_sha256_tv_template, + .count = ARRAY_SIZE(drbg_pr_sha256_tv_template) + } + } + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(sha256)) test */ + .alg = "drbg(pr(sha384))", + .fips_allowed = 1, + .test = alg_test_null, + }, { + /* There is no need to specifically test the DRBG with every + * backend cipher -- covered by drbg(pr(sha256)) test */ + .alg = "drbg(pr(sha512))", + .fips_allowed = 1, + .test = alg_test_null, + }, { .alg = "ecb(__aes-aesni)", .test = alg_test_null, .fips_allowed = 1,