mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-31 05:51:53 -06:00
crypto: cipher: introduce context free function
Refactors the qcrypto_cipher_free(), splits it into two parts. One is gcrypt/nettle__cipher_free_ctx() to free the special context. This makes code more clear, what's more, it would be used by the later patch. Reviewed-by: Gonglei <arei.gonglei@huawei.com> Signed-off-by: Longpeng(Mike) <longpeng2@huawei.com> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
This commit is contained in:
parent
6887dc6700
commit
cc5eff0186
2 changed files with 32 additions and 17 deletions
|
@ -64,6 +64,22 @@ struct QCryptoCipherGcrypt {
|
||||||
uint8_t *iv;
|
uint8_t *iv;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
static void gcrypt_cipher_free_ctx(QCryptoCipherGcrypt *ctx,
|
||||||
|
QCryptoCipherMode mode)
|
||||||
|
{
|
||||||
|
if (!ctx) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
gcry_cipher_close(ctx->handle);
|
||||||
|
if (mode == QCRYPTO_CIPHER_MODE_XTS) {
|
||||||
|
gcry_cipher_close(ctx->tweakhandle);
|
||||||
|
}
|
||||||
|
g_free(ctx->iv);
|
||||||
|
g_free(ctx);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
QCryptoCipher *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
|
QCryptoCipher *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
|
||||||
QCryptoCipherMode mode,
|
QCryptoCipherMode mode,
|
||||||
const uint8_t *key, size_t nkey,
|
const uint8_t *key, size_t nkey,
|
||||||
|
@ -228,11 +244,7 @@ QCryptoCipher *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
|
||||||
return cipher;
|
return cipher;
|
||||||
|
|
||||||
error:
|
error:
|
||||||
gcry_cipher_close(ctx->handle);
|
gcrypt_cipher_free_ctx(ctx, mode);
|
||||||
if (cipher->mode == QCRYPTO_CIPHER_MODE_XTS) {
|
|
||||||
gcry_cipher_close(ctx->tweakhandle);
|
|
||||||
}
|
|
||||||
g_free(ctx);
|
|
||||||
g_free(cipher);
|
g_free(cipher);
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
@ -240,17 +252,10 @@ QCryptoCipher *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
|
||||||
|
|
||||||
void qcrypto_cipher_free(QCryptoCipher *cipher)
|
void qcrypto_cipher_free(QCryptoCipher *cipher)
|
||||||
{
|
{
|
||||||
QCryptoCipherGcrypt *ctx;
|
|
||||||
if (!cipher) {
|
if (!cipher) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
ctx = cipher->opaque;
|
gcrypt_cipher_free_ctx(cipher->opaque, cipher->mode);
|
||||||
gcry_cipher_close(ctx->handle);
|
|
||||||
if (cipher->mode == QCRYPTO_CIPHER_MODE_XTS) {
|
|
||||||
gcry_cipher_close(ctx->tweakhandle);
|
|
||||||
}
|
|
||||||
g_free(ctx->iv);
|
|
||||||
g_free(ctx);
|
|
||||||
g_free(cipher);
|
g_free(cipher);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -249,6 +249,19 @@ bool qcrypto_cipher_supports(QCryptoCipherAlgorithm alg,
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
static void nettle_cipher_free_ctx(QCryptoCipherNettle *ctx)
|
||||||
|
{
|
||||||
|
if (!ctx) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
g_free(ctx->iv);
|
||||||
|
g_free(ctx->ctx);
|
||||||
|
g_free(ctx->ctx_tweak);
|
||||||
|
g_free(ctx);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
QCryptoCipher *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
|
QCryptoCipher *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
|
||||||
QCryptoCipherMode mode,
|
QCryptoCipherMode mode,
|
||||||
const uint8_t *key, size_t nkey,
|
const uint8_t *key, size_t nkey,
|
||||||
|
@ -440,10 +453,7 @@ void qcrypto_cipher_free(QCryptoCipher *cipher)
|
||||||
}
|
}
|
||||||
|
|
||||||
ctx = cipher->opaque;
|
ctx = cipher->opaque;
|
||||||
g_free(ctx->iv);
|
nettle_cipher_free_ctx(ctx);
|
||||||
g_free(ctx->ctx);
|
|
||||||
g_free(ctx->ctx_tweak);
|
|
||||||
g_free(ctx);
|
|
||||||
g_free(cipher);
|
g_free(cipher);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue