On 05/21/2016 10:22 AM, Mikhail Feoktistov wrote:
> ---
>  src/util/vircrypto.c | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
> 

Ugh... Sorry about this - forgot about the unused attributes <sigh>

I should have just gone with the entire function w/ the
HAVE_GNUTLS_CIPHER_ENCRYPT macro instead of the intermixed
ATTRIBUTE_UNUSED as those would have a tendency for someone to come
along and try to patch again pointing out they are used without
realizing it was the gnutls macro that needed to be broader...

I'll send something short with 2 patches that will do that. I started
working on the next task and realized I needed a slight adjustment anyway...

John
> 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:
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to