[libvirt] [PATCH] util: compilation fix if build without GNUTLS library
Roman Bogorodskiy
bogorodskiy at gmail.com
Sun May 22 06:03:34 UTC 2016
Mikhail Feoktistov wrote:
> ---
> src/util/vircrypto.c | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/src/util/vircrypto.c b/src/util/vircrypto.c
> index 5183d49..4f288f0 100644
> --- a/src/util/vircrypto.c
> +++ b/src/util/vircrypto.c
> @@ -24,6 +24,7 @@
> #include "virlog.h"
> #include "virerror.h"
> #include "viralloc.h"
> +#include "virrandom.h"
>
> #include "md5.h"
> #include "sha256.h"
> @@ -220,14 +221,14 @@ virCryptoEncryptDataAESgnutls(gnutls_cipher_algorithm_t gnutls_enc_alg,
> */
> int
> virCryptoEncryptData(virCryptoCipher algorithm,
> - uint8_t *enckey,
> + uint8_t *enckey ATTRIBUTE_UNUSED,
> size_t enckeylen,
> - uint8_t *iv,
> + uint8_t *iv ATTRIBUTE_UNUSED,
> size_t ivlen,
> - uint8_t *data,
> - size_t datalen,
> - uint8_t **ciphertext,
> - size_t *ciphertextlen)
> + uint8_t *data ATTRIBUTE_UNUSED,
> + size_t datalen ATTRIBUTE_UNUSED,
> + uint8_t **ciphertext ATTRIBUTE_UNUSED,
> + size_t *ciphertextlen ATTRIBUTE_UNUSED)
> {
> switch (algorithm) {
> case VIR_CRYPTO_CIPHER_AES256CBC:
ACK; I've shortened commit message title a little and pushed to fix
build.
However, I'm wondering if there should be a better fix in long term to
avoid ATTRIBUTE_UNUSED for attributes that possibly could be used, e.g.
either using something like
#ifndef HAVE_GNUTLS_CIPHER_ENCRYPT
# define GNUTLS_ATTRIBUTE ATTRIBUTE_UNUSED
#else
# define GNUTLS_ATTRIBUTE
or just adding a stub for virCryptoEncryptData() as the actual
implementation does not *seem* to do anything useful without gnutls
anyway.
Roman Bogorodskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160522/fc9d679f/attachment-0001.sig>
More information about the libvir-list
mailing list