crypto/builtin: Merge qcrypto_cipher_aes_{ecb,xts}_{en,de}crypt

There's no real reason we need two separate helper functions here.
Standardize on the function signature required for xts_encrypt.
Rename to do_aes_{en,de}crypt_ecb, since the helper does not
itself do anything with respect to xts.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Richard Henderson 2020-08-28 10:05:18 -07:00 committed by Daniel P. Berrangé
parent 838e4631cb
commit 8ee47cddbe

View file

@ -72,65 +72,38 @@ static void qcrypto_cipher_free_aes(QCryptoCipher *cipher)
g_free(cipher); g_free(cipher);
} }
static void do_aes_encrypt_ecb(const void *vctx,
static void qcrypto_cipher_aes_ecb_encrypt(const AES_KEY *key, size_t len,
const void *in, uint8_t *out,
void *out, const uint8_t *in)
size_t len)
{ {
const uint8_t *inptr = in; const QCryptoCipherBuiltinAESContext *ctx = vctx;
uint8_t *outptr = out;
/* We have already verified that len % AES_BLOCK_SIZE == 0. */ /* We have already verified that len % AES_BLOCK_SIZE == 0. */
while (len) { while (len) {
AES_encrypt(inptr, outptr, key); AES_encrypt(in, out, &ctx->enc);
inptr += AES_BLOCK_SIZE; in += AES_BLOCK_SIZE;
outptr += AES_BLOCK_SIZE; out += AES_BLOCK_SIZE;
len -= AES_BLOCK_SIZE; len -= AES_BLOCK_SIZE;
} }
} }
static void do_aes_decrypt_ecb(const void *vctx,
static void qcrypto_cipher_aes_ecb_decrypt(const AES_KEY *key, size_t len,
const void *in, uint8_t *out,
void *out, const uint8_t *in)
size_t len)
{ {
const uint8_t *inptr = in; const QCryptoCipherBuiltinAESContext *ctx = vctx;
uint8_t *outptr = out;
/* We have already verified that len % AES_BLOCK_SIZE == 0. */ /* We have already verified that len % AES_BLOCK_SIZE == 0. */
while (len) { while (len) {
AES_decrypt(inptr, outptr, key); AES_decrypt(in, out, &ctx->dec);
inptr += AES_BLOCK_SIZE; in += AES_BLOCK_SIZE;
outptr += AES_BLOCK_SIZE; out += AES_BLOCK_SIZE;
len -= AES_BLOCK_SIZE; len -= AES_BLOCK_SIZE;
} }
} }
static void qcrypto_cipher_aes_xts_encrypt(const void *ctx,
size_t length,
uint8_t *dst,
const uint8_t *src)
{
const QCryptoCipherBuiltinAESContext *aesctx = ctx;
qcrypto_cipher_aes_ecb_encrypt(&aesctx->enc, src, dst, length);
}
static void qcrypto_cipher_aes_xts_decrypt(const void *ctx,
size_t length,
uint8_t *dst,
const uint8_t *src)
{
const QCryptoCipherBuiltinAESContext *aesctx = ctx;
qcrypto_cipher_aes_ecb_decrypt(&aesctx->dec, src, dst, length);
}
static int qcrypto_cipher_encrypt_aes(QCryptoCipher *cipher, static int qcrypto_cipher_encrypt_aes(QCryptoCipher *cipher,
const void *in, const void *in,
void *out, void *out,
@ -142,8 +115,7 @@ static int qcrypto_cipher_encrypt_aes(QCryptoCipher *cipher,
switch (cipher->mode) { switch (cipher->mode) {
case QCRYPTO_CIPHER_MODE_ECB: case QCRYPTO_CIPHER_MODE_ECB:
qcrypto_cipher_aes_ecb_encrypt(&ctxt->state.aes.key.enc, do_aes_encrypt_ecb(&ctxt->state.aes.key, len, out, in);
in, out, len);
break; break;
case QCRYPTO_CIPHER_MODE_CBC: case QCRYPTO_CIPHER_MODE_CBC:
AES_cbc_encrypt(in, out, len, AES_cbc_encrypt(in, out, len,
@ -153,8 +125,8 @@ static int qcrypto_cipher_encrypt_aes(QCryptoCipher *cipher,
case QCRYPTO_CIPHER_MODE_XTS: case QCRYPTO_CIPHER_MODE_XTS:
xts_encrypt(&ctxt->state.aes.key, xts_encrypt(&ctxt->state.aes.key,
&ctxt->state.aes.key_tweak, &ctxt->state.aes.key_tweak,
qcrypto_cipher_aes_xts_encrypt, do_aes_encrypt_ecb,
qcrypto_cipher_aes_xts_decrypt, do_aes_decrypt_ecb,
ctxt->state.aes.iv, ctxt->state.aes.iv,
len, out, in); len, out, in);
break; break;
@ -177,8 +149,7 @@ static int qcrypto_cipher_decrypt_aes(QCryptoCipher *cipher,
switch (cipher->mode) { switch (cipher->mode) {
case QCRYPTO_CIPHER_MODE_ECB: case QCRYPTO_CIPHER_MODE_ECB:
qcrypto_cipher_aes_ecb_decrypt(&ctxt->state.aes.key.dec, do_aes_decrypt_ecb(&ctxt->state.aes.key, len, out, in);
in, out, len);
break; break;
case QCRYPTO_CIPHER_MODE_CBC: case QCRYPTO_CIPHER_MODE_CBC:
AES_cbc_encrypt(in, out, len, AES_cbc_encrypt(in, out, len,
@ -188,8 +159,8 @@ static int qcrypto_cipher_decrypt_aes(QCryptoCipher *cipher,
case QCRYPTO_CIPHER_MODE_XTS: case QCRYPTO_CIPHER_MODE_XTS:
xts_decrypt(&ctxt->state.aes.key, xts_decrypt(&ctxt->state.aes.key,
&ctxt->state.aes.key_tweak, &ctxt->state.aes.key_tweak,
qcrypto_cipher_aes_xts_encrypt, do_aes_encrypt_ecb,
qcrypto_cipher_aes_xts_decrypt, do_aes_decrypt_ecb,
ctxt->state.aes.iv, ctxt->state.aes.iv,
len, out, in); len, out, in);
break; break;