[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 for-2.10 01/18] crypto: cipher: introduce con
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH v2 for-2.10 01/18] crypto: cipher: introduce context free function |
Date: |
Fri, 21 Apr 2017 11:55:37 +0000 |
> -----Original Message-----
> From: longpeng
> Sent: Monday, April 17, 2017 9:33 AM
> To: address@hidden
> Cc: Gonglei (Arei); Huangweidong (C); address@hidden;
> address@hidden; address@hidden; address@hidden; longpeng
> Subject: [PATCH v2 for-2.10 01/18] 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.
>
> Signed-off-by: Longpeng(Mike) <address@hidden>
> ---
> crypto/cipher-gcrypt.c | 31 ++++++++++++++++++-------------
> crypto/cipher-nettle.c | 18 ++++++++++++++----
> 2 files changed, 32 insertions(+), 17 deletions(-)
>
Reviewed-by: Gonglei <address@hidden>
> diff --git a/crypto/cipher-gcrypt.c b/crypto/cipher-gcrypt.c
> index 6487eca..0ecffa2 100644
> --- a/crypto/cipher-gcrypt.c
> +++ b/crypto/cipher-gcrypt.c
> @@ -64,6 +64,22 @@ struct QCryptoCipherGcrypt {
> 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,
> QCryptoCipherMode mode,
> const uint8_t *key, size_t nkey,
> @@ -228,11 +244,7 @@ QCryptoCipher
> *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
> return cipher;
>
> error:
> - gcry_cipher_close(ctx->handle);
> - if (cipher->mode == QCRYPTO_CIPHER_MODE_XTS) {
> - gcry_cipher_close(ctx->tweakhandle);
> - }
> - g_free(ctx);
> + gcrypt_cipher_free_ctx(ctx, mode);
> g_free(cipher);
> return NULL;
> }
> @@ -240,17 +252,10 @@ QCryptoCipher
> *qcrypto_cipher_new(QCryptoCipherAlgorithm alg,
>
> void qcrypto_cipher_free(QCryptoCipher *cipher)
> {
> - QCryptoCipherGcrypt *ctx;
> if (!cipher) {
> return;
> }
> - ctx = cipher->opaque;
> - gcry_cipher_close(ctx->handle);
> - if (cipher->mode == QCRYPTO_CIPHER_MODE_XTS) {
> - gcry_cipher_close(ctx->tweakhandle);
> - }
> - g_free(ctx->iv);
> - g_free(ctx);
> + gcrypt_cipher_free_ctx(cipher->opaque, cipher->mode);
> g_free(cipher);
> }
>
> diff --git a/crypto/cipher-nettle.c b/crypto/cipher-nettle.c
> index dfc9030..e04e3a1 100644
> --- a/crypto/cipher-nettle.c
> +++ b/crypto/cipher-nettle.c
> @@ -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,
> QCryptoCipherMode mode,
> const uint8_t *key, size_t nkey,
> @@ -440,10 +453,7 @@ void qcrypto_cipher_free(QCryptoCipher *cipher)
> }
>
> ctx = cipher->opaque;
> - g_free(ctx->iv);
> - g_free(ctx->ctx);
> - g_free(ctx->ctx_tweak);
> - g_free(ctx);
> + nettle_cipher_free_ctx(ctx);
> g_free(cipher);
> }
>
> --
> 1.8.3.1
>
- [Qemu-devel] [PATCH v2 for-2.10 00/18] crypto: add afalg-backend support, Longpeng(Mike), 2017/04/16
- [Qemu-devel] [PATCH v2 for-2.10 04/18] crypto: cipher: introduce qcrypto_cipher_ctx_new for builtin-backend, Longpeng(Mike), 2017/04/16
- [Qemu-devel] [PATCH v2 for-2.10 01/18] crypto: cipher: introduce context free function, Longpeng(Mike), 2017/04/16
- Re: [Qemu-devel] [PATCH v2 for-2.10 01/18] crypto: cipher: introduce context free function,
Gonglei (Arei) <=
- [Qemu-devel] [PATCH v2 for-2.10 08/18] crypto: hmac: introduce qcrypto_hmac_ctx_new for gcrypt-backend, Longpeng(Mike), 2017/04/16
- [Qemu-devel] [PATCH v2 for-2.10 10/18] crypto: hmac: introduce qcrypto_hmac_ctx_new for glib-backend, Longpeng(Mike), 2017/04/16
- [Qemu-devel] [PATCH v2 for-2.10 18/18] tests: crypto: add hmac speed benchmark support, Longpeng(Mike), 2017/04/16
- [Qemu-devel] [PATCH v2 for-2.10 02/18] crypto: cipher: introduce qcrypto_cipher_ctx_new for gcrypt-backend, Longpeng(Mike), 2017/04/16
- [Qemu-devel] [PATCH v2 for-2.10 12/18] crypto: introduce some common functions for af_alg backend, Longpeng(Mike), 2017/04/16