crypto: testmgr - white space removal on __test_skcipher
This patch inverts two if conditions to remove code blocks indentation. Several white space clean-ups follow. Signed-off-by: Cristian Stoica <cristian.stoica@freescale.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
This commit is contained in:
parent
5f2b424e5c
commit
a1aa44a2eb
1 changed files with 146 additions and 157 deletions
303
crypto/testmgr.c
303
crypto/testmgr.c
|
@ -966,73 +966,69 @@ static int __test_skcipher(struct crypto_ablkcipher *tfm, int enc,
|
||||||
else
|
else
|
||||||
memset(iv, 0, MAX_IVLEN);
|
memset(iv, 0, MAX_IVLEN);
|
||||||
|
|
||||||
if (!(template[i].np) || (template[i].also_non_np)) {
|
if (template[i].np && !template[i].also_non_np)
|
||||||
j++;
|
continue;
|
||||||
|
|
||||||
ret = -EINVAL;
|
j++;
|
||||||
if (WARN_ON(align_offset + template[i].ilen >
|
|
||||||
PAGE_SIZE))
|
|
||||||
goto out;
|
|
||||||
|
|
||||||
data = xbuf[0];
|
ret = -EINVAL;
|
||||||
|
if (WARN_ON(align_offset + template[i].ilen > PAGE_SIZE))
|
||||||
|
goto out;
|
||||||
|
|
||||||
|
data = xbuf[0];
|
||||||
|
data += align_offset;
|
||||||
|
memcpy(data, template[i].input, template[i].ilen);
|
||||||
|
|
||||||
|
crypto_ablkcipher_clear_flags(tfm, ~0);
|
||||||
|
if (template[i].wk)
|
||||||
|
crypto_ablkcipher_set_flags(tfm, CRYPTO_TFM_REQ_WEAK_KEY);
|
||||||
|
|
||||||
|
ret = crypto_ablkcipher_setkey(tfm, template[i].key,
|
||||||
|
template[i].klen);
|
||||||
|
if (!ret == template[i].fail) {
|
||||||
|
pr_err("alg: skcipher%s: setkey failed on test %d for %s: flags=%x\n",
|
||||||
|
d, j, algo, crypto_ablkcipher_get_flags(tfm));
|
||||||
|
goto out;
|
||||||
|
} else if (ret)
|
||||||
|
continue;
|
||||||
|
|
||||||
|
sg_init_one(&sg[0], data, template[i].ilen);
|
||||||
|
if (diff_dst) {
|
||||||
|
data = xoutbuf[0];
|
||||||
data += align_offset;
|
data += align_offset;
|
||||||
memcpy(data, template[i].input, template[i].ilen);
|
sg_init_one(&sgout[0], data, template[i].ilen);
|
||||||
|
}
|
||||||
|
|
||||||
crypto_ablkcipher_clear_flags(tfm, ~0);
|
ablkcipher_request_set_crypt(req, sg, (diff_dst) ? sgout : sg,
|
||||||
if (template[i].wk)
|
template[i].ilen, iv);
|
||||||
crypto_ablkcipher_set_flags(
|
ret = enc ? crypto_ablkcipher_encrypt(req) :
|
||||||
tfm, CRYPTO_TFM_REQ_WEAK_KEY);
|
crypto_ablkcipher_decrypt(req);
|
||||||
|
|
||||||
ret = crypto_ablkcipher_setkey(tfm, template[i].key,
|
switch (ret) {
|
||||||
template[i].klen);
|
case 0:
|
||||||
if (!ret == template[i].fail) {
|
break;
|
||||||
pr_err("alg: skcipher%s: setkey failed on test %d for %s: flags=%x\n",
|
case -EINPROGRESS:
|
||||||
d, j, algo,
|
case -EBUSY:
|
||||||
crypto_ablkcipher_get_flags(tfm));
|
ret = wait_for_completion_interruptible(
|
||||||
goto out;
|
&result.completion);
|
||||||
} else if (ret)
|
if (!ret && !((ret = result.err))) {
|
||||||
continue;
|
reinit_completion(&result.completion);
|
||||||
|
|
||||||
sg_init_one(&sg[0], data, template[i].ilen);
|
|
||||||
if (diff_dst) {
|
|
||||||
data = xoutbuf[0];
|
|
||||||
data += align_offset;
|
|
||||||
sg_init_one(&sgout[0], data, template[i].ilen);
|
|
||||||
}
|
|
||||||
|
|
||||||
ablkcipher_request_set_crypt(req, sg,
|
|
||||||
(diff_dst) ? sgout : sg,
|
|
||||||
template[i].ilen, iv);
|
|
||||||
ret = enc ?
|
|
||||||
crypto_ablkcipher_encrypt(req) :
|
|
||||||
crypto_ablkcipher_decrypt(req);
|
|
||||||
|
|
||||||
switch (ret) {
|
|
||||||
case 0:
|
|
||||||
break;
|
break;
|
||||||
case -EINPROGRESS:
|
|
||||||
case -EBUSY:
|
|
||||||
ret = wait_for_completion_interruptible(
|
|
||||||
&result.completion);
|
|
||||||
if (!ret && !((ret = result.err))) {
|
|
||||||
reinit_completion(&result.completion);
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
/* fall through */
|
|
||||||
default:
|
|
||||||
pr_err("alg: skcipher%s: %s failed on test %d for %s: ret=%d\n",
|
|
||||||
d, e, j, algo, -ret);
|
|
||||||
goto out;
|
|
||||||
}
|
}
|
||||||
|
/* fall through */
|
||||||
|
default:
|
||||||
|
pr_err("alg: skcipher%s: %s failed on test %d for %s: ret=%d\n",
|
||||||
|
d, e, j, algo, -ret);
|
||||||
|
goto out;
|
||||||
|
}
|
||||||
|
|
||||||
q = data;
|
q = data;
|
||||||
if (memcmp(q, template[i].result, template[i].rlen)) {
|
if (memcmp(q, template[i].result, template[i].rlen)) {
|
||||||
pr_err("alg: skcipher%s: Test %d failed on %s for %s\n",
|
pr_err("alg: skcipher%s: Test %d failed on %s for %s\n",
|
||||||
d, j, e, algo);
|
d, j, e, algo);
|
||||||
hexdump(q, template[i].rlen);
|
hexdump(q, template[i].rlen);
|
||||||
ret = -EINVAL;
|
ret = -EINVAL;
|
||||||
goto out;
|
goto out;
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1047,116 +1043,109 @@ static int __test_skcipher(struct crypto_ablkcipher *tfm, int enc,
|
||||||
else
|
else
|
||||||
memset(iv, 0, MAX_IVLEN);
|
memset(iv, 0, MAX_IVLEN);
|
||||||
|
|
||||||
if (template[i].np) {
|
if (!template[i].np)
|
||||||
j++;
|
continue;
|
||||||
|
|
||||||
crypto_ablkcipher_clear_flags(tfm, ~0);
|
j++;
|
||||||
if (template[i].wk)
|
|
||||||
crypto_ablkcipher_set_flags(
|
|
||||||
tfm, CRYPTO_TFM_REQ_WEAK_KEY);
|
|
||||||
|
|
||||||
ret = crypto_ablkcipher_setkey(tfm, template[i].key,
|
crypto_ablkcipher_clear_flags(tfm, ~0);
|
||||||
template[i].klen);
|
if (template[i].wk)
|
||||||
if (!ret == template[i].fail) {
|
crypto_ablkcipher_set_flags(tfm, CRYPTO_TFM_REQ_WEAK_KEY);
|
||||||
pr_err("alg: skcipher%s: setkey failed on chunk test %d for %s: flags=%x\n",
|
|
||||||
d, j, algo,
|
ret = crypto_ablkcipher_setkey(tfm, template[i].key,
|
||||||
crypto_ablkcipher_get_flags(tfm));
|
template[i].klen);
|
||||||
|
if (!ret == template[i].fail) {
|
||||||
|
pr_err("alg: skcipher%s: setkey failed on chunk test %d for %s: flags=%x\n",
|
||||||
|
d, j, algo, crypto_ablkcipher_get_flags(tfm));
|
||||||
|
goto out;
|
||||||
|
} else if (ret)
|
||||||
|
continue;
|
||||||
|
|
||||||
|
temp = 0;
|
||||||
|
ret = -EINVAL;
|
||||||
|
sg_init_table(sg, template[i].np);
|
||||||
|
if (diff_dst)
|
||||||
|
sg_init_table(sgout, template[i].np);
|
||||||
|
for (k = 0; k < template[i].np; k++) {
|
||||||
|
if (WARN_ON(offset_in_page(IDX[k]) +
|
||||||
|
template[i].tap[k] > PAGE_SIZE))
|
||||||
goto out;
|
goto out;
|
||||||
} else if (ret)
|
|
||||||
continue;
|
|
||||||
|
|
||||||
temp = 0;
|
q = xbuf[IDX[k] >> PAGE_SHIFT] + offset_in_page(IDX[k]);
|
||||||
ret = -EINVAL;
|
|
||||||
sg_init_table(sg, template[i].np);
|
memcpy(q, template[i].input + temp, template[i].tap[k]);
|
||||||
|
|
||||||
|
if (offset_in_page(q) + template[i].tap[k] < PAGE_SIZE)
|
||||||
|
q[template[i].tap[k]] = 0;
|
||||||
|
|
||||||
|
sg_set_buf(&sg[k], q, template[i].tap[k]);
|
||||||
|
if (diff_dst) {
|
||||||
|
q = xoutbuf[IDX[k] >> PAGE_SHIFT] +
|
||||||
|
offset_in_page(IDX[k]);
|
||||||
|
|
||||||
|
sg_set_buf(&sgout[k], q, template[i].tap[k]);
|
||||||
|
|
||||||
|
memset(q, 0, template[i].tap[k]);
|
||||||
|
if (offset_in_page(q) +
|
||||||
|
template[i].tap[k] < PAGE_SIZE)
|
||||||
|
q[template[i].tap[k]] = 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
temp += template[i].tap[k];
|
||||||
|
}
|
||||||
|
|
||||||
|
ablkcipher_request_set_crypt(req, sg, (diff_dst) ? sgout : sg,
|
||||||
|
template[i].ilen, iv);
|
||||||
|
|
||||||
|
ret = enc ? crypto_ablkcipher_encrypt(req) :
|
||||||
|
crypto_ablkcipher_decrypt(req);
|
||||||
|
|
||||||
|
switch (ret) {
|
||||||
|
case 0:
|
||||||
|
break;
|
||||||
|
case -EINPROGRESS:
|
||||||
|
case -EBUSY:
|
||||||
|
ret = wait_for_completion_interruptible(
|
||||||
|
&result.completion);
|
||||||
|
if (!ret && !((ret = result.err))) {
|
||||||
|
reinit_completion(&result.completion);
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
/* fall through */
|
||||||
|
default:
|
||||||
|
pr_err("alg: skcipher%s: %s failed on chunk test %d for %s: ret=%d\n",
|
||||||
|
d, e, j, algo, -ret);
|
||||||
|
goto out;
|
||||||
|
}
|
||||||
|
|
||||||
|
temp = 0;
|
||||||
|
ret = -EINVAL;
|
||||||
|
for (k = 0; k < template[i].np; k++) {
|
||||||
if (diff_dst)
|
if (diff_dst)
|
||||||
sg_init_table(sgout, template[i].np);
|
q = xoutbuf[IDX[k] >> PAGE_SHIFT] +
|
||||||
for (k = 0; k < template[i].np; k++) {
|
offset_in_page(IDX[k]);
|
||||||
if (WARN_ON(offset_in_page(IDX[k]) +
|
else
|
||||||
template[i].tap[k] > PAGE_SIZE))
|
|
||||||
goto out;
|
|
||||||
|
|
||||||
q = xbuf[IDX[k] >> PAGE_SHIFT] +
|
q = xbuf[IDX[k] >> PAGE_SHIFT] +
|
||||||
offset_in_page(IDX[k]);
|
offset_in_page(IDX[k]);
|
||||||
|
|
||||||
memcpy(q, template[i].input + temp,
|
if (memcmp(q, template[i].result + temp,
|
||||||
template[i].tap[k]);
|
template[i].tap[k])) {
|
||||||
|
pr_err("alg: skcipher%s: Chunk test %d failed on %s at page %u for %s\n",
|
||||||
if (offset_in_page(q) + template[i].tap[k] <
|
d, j, e, k, algo);
|
||||||
PAGE_SIZE)
|
hexdump(q, template[i].tap[k]);
|
||||||
q[template[i].tap[k]] = 0;
|
|
||||||
|
|
||||||
sg_set_buf(&sg[k], q, template[i].tap[k]);
|
|
||||||
if (diff_dst) {
|
|
||||||
q = xoutbuf[IDX[k] >> PAGE_SHIFT] +
|
|
||||||
offset_in_page(IDX[k]);
|
|
||||||
|
|
||||||
sg_set_buf(&sgout[k], q,
|
|
||||||
template[i].tap[k]);
|
|
||||||
|
|
||||||
memset(q, 0, template[i].tap[k]);
|
|
||||||
if (offset_in_page(q) +
|
|
||||||
template[i].tap[k] < PAGE_SIZE)
|
|
||||||
q[template[i].tap[k]] = 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
temp += template[i].tap[k];
|
|
||||||
}
|
|
||||||
|
|
||||||
ablkcipher_request_set_crypt(req, sg,
|
|
||||||
(diff_dst) ? sgout : sg,
|
|
||||||
template[i].ilen, iv);
|
|
||||||
|
|
||||||
ret = enc ?
|
|
||||||
crypto_ablkcipher_encrypt(req) :
|
|
||||||
crypto_ablkcipher_decrypt(req);
|
|
||||||
|
|
||||||
switch (ret) {
|
|
||||||
case 0:
|
|
||||||
break;
|
|
||||||
case -EINPROGRESS:
|
|
||||||
case -EBUSY:
|
|
||||||
ret = wait_for_completion_interruptible(
|
|
||||||
&result.completion);
|
|
||||||
if (!ret && !((ret = result.err))) {
|
|
||||||
reinit_completion(&result.completion);
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
/* fall through */
|
|
||||||
default:
|
|
||||||
pr_err("alg: skcipher%s: %s failed on chunk test %d for %s: ret=%d\n",
|
|
||||||
d, e, j, algo, -ret);
|
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
temp = 0;
|
q += template[i].tap[k];
|
||||||
ret = -EINVAL;
|
for (n = 0; offset_in_page(q + n) && q[n]; n++)
|
||||||
for (k = 0; k < template[i].np; k++) {
|
;
|
||||||
if (diff_dst)
|
if (n) {
|
||||||
q = xoutbuf[IDX[k] >> PAGE_SHIFT] +
|
pr_err("alg: skcipher%s: Result buffer corruption in chunk test %d on %s at page %u for %s: %u bytes:\n",
|
||||||
offset_in_page(IDX[k]);
|
d, j, e, k, algo, n);
|
||||||
else
|
hexdump(q, n);
|
||||||
q = xbuf[IDX[k] >> PAGE_SHIFT] +
|
goto out;
|
||||||
offset_in_page(IDX[k]);
|
|
||||||
|
|
||||||
if (memcmp(q, template[i].result + temp,
|
|
||||||
template[i].tap[k])) {
|
|
||||||
pr_err("alg: skcipher%s: Chunk test %d failed on %s at page %u for %s\n",
|
|
||||||
d, j, e, k, algo);
|
|
||||||
hexdump(q, template[i].tap[k]);
|
|
||||||
goto out;
|
|
||||||
}
|
|
||||||
|
|
||||||
q += template[i].tap[k];
|
|
||||||
for (n = 0; offset_in_page(q + n) && q[n]; n++)
|
|
||||||
;
|
|
||||||
if (n) {
|
|
||||||
pr_err("alg: skcipher%s: Result buffer corruption in chunk test %d on %s at page %u for %s: %u bytes:\n",
|
|
||||||
d, j, e, k, algo, n);
|
|
||||||
hexdump(q, n);
|
|
||||||
goto out;
|
|
||||||
}
|
|
||||||
temp += template[i].tap[k];
|
|
||||||
}
|
}
|
||||||
|
temp += template[i].tap[k];
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue