]> xenbits.xensource.com Git - libvirt.git/commitdiff
util: Alter virCryptoEncryptData for non GNUTLS builds
authorJohn Ferlan <jferlan@redhat.com>
Sun, 22 May 2016 14:07:47 +0000 (10:07 -0400)
committerJohn Ferlan <jferlan@redhat.com>
Tue, 7 Jun 2016 11:38:25 +0000 (07:38 -0400)
Rather than intermixing the ATTRIBUTE_UNUSED - use HAVE_GNUTLS_CIPHER_ENCRYPT
for the whole function instead.

src/util/vircrypto.c

index 4f288f0f8dce83b08741de8bb8eec914acb26a13..aaede94c6394af4479c7ac0f41d07fbddeb994ea 100644 (file)
@@ -200,7 +200,6 @@ virCryptoEncryptDataAESgnutls(gnutls_cipher_algorithm_t gnutls_enc_alg,
     memset(&iv_buf, 0, sizeof(gnutls_datum_t));
     return -1;
 }
-#endif
 
 
 /* virCryptoEncryptData:
@@ -221,14 +220,14 @@ virCryptoEncryptDataAESgnutls(gnutls_cipher_algorithm_t gnutls_enc_alg,
  */
 int
 virCryptoEncryptData(virCryptoCipher algorithm,
-                     uint8_t *enckey ATTRIBUTE_UNUSED,
+                     uint8_t *enckey,
                      size_t enckeylen,
-                     uint8_t *iv ATTRIBUTE_UNUSED,
+                     uint8_t *iv,
                      size_t ivlen,
-                     uint8_t *data ATTRIBUTE_UNUSED,
-                     size_t datalen ATTRIBUTE_UNUSED,
-                     uint8_t **ciphertext ATTRIBUTE_UNUSED,
-                     size_t *ciphertextlen ATTRIBUTE_UNUSED)
+                     uint8_t *data,
+                     size_t datalen,
+                     uint8_t **ciphertext,
+                     size_t *ciphertextlen)
 {
     switch (algorithm) {
     case VIR_CRYPTO_CIPHER_AES256CBC:
@@ -246,7 +245,6 @@ virCryptoEncryptData(virCryptoCipher algorithm,
             return -1;
         }
 
-#ifdef HAVE_GNUTLS_CIPHER_ENCRYPT
         /*
          * Encrypt the data buffer using an encryption key and
          * initialization vector via the gnutls_cipher_encrypt API
@@ -256,9 +254,6 @@ virCryptoEncryptData(virCryptoCipher algorithm,
                                              enckey, enckeylen, iv, ivlen,
                                              data, datalen,
                                              ciphertext, ciphertextlen);
-#else
-        break;
-#endif
 
     case VIR_CRYPTO_CIPHER_NONE:
     case VIR_CRYPTO_CIPHER_LAST:
@@ -270,6 +265,25 @@ virCryptoEncryptData(virCryptoCipher algorithm,
     return -1;
 }
 
+#else
+
+int
+virCryptoEncryptData(virCryptoCipher algorithm,
+                     uint8_t *enckey ATTRIBUTE_UNUSED,
+                     size_t enckeylen ATTRIBUTE_UNUSED,
+                     uint8_t *iv ATTRIBUTE_UNUSED,
+                     size_t ivlen ATTRIBUTE_UNUSED,
+                     uint8_t *data ATTRIBUTE_UNUSED,
+                     size_t datalen ATTRIBUTE_UNUSED,
+                     uint8_t **ciphertext ATTRIBUTE_UNUSED,
+                     size_t *ciphertextlen ATTRIBUTE_UNUSED)
+{
+    virReportError(VIR_ERR_INVALID_ARG,
+                   _("algorithm=%d is not supported"), algorithm);
+    return -1;
+}
+#endif
+
 /* virCryptoGenerateRandom:
  * @nbytes: Size in bytes of random byte stream to generate
  *