]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Clean up cksum handling in CMAC token functions
authorGreg Hudson <ghudson@mit.edu>
Fri, 29 Mar 2013 05:57:31 +0000 (01:57 -0400)
committerGreg Hudson <ghudson@mit.edu>
Fri, 29 Mar 2013 05:59:33 +0000 (01:59 -0400)
In krb5int_dk_cmac_encrypt, cksum wasn't used.  In
krb5int_dk_cmac_decrypt, cksum needs to be initialized since we clean
it up.

src/lib/crypto/krb/enc_dk_cmac.c

index 508f9f8234edd906e1a50e978ca3b915956aeb37..e27c862ad16dbd83f040ea64dbf61435408c7c79 100644 (file)
@@ -89,7 +89,6 @@ krb5int_dk_cmac_encrypt(const struct krb5_keytypes *ktp, krb5_key key,
     const struct krb5_enc_provider *enc = ktp->enc;
     krb5_error_code ret;
     krb5_crypto_iov *header, *trailer, *padding;
-    krb5_data cksum = empty_data();
     krb5_key ke = NULL, ki = NULL;
 
     /* E(Confounder | Plaintext | Pad) | Checksum */
@@ -129,7 +128,6 @@ krb5int_dk_cmac_encrypt(const struct krb5_keytypes *ktp, krb5_key key,
 cleanup:
     krb5_k_free_key(NULL, ke);
     krb5_k_free_key(NULL, ki);
-    zapfree(cksum.data, cksum.length);
     return ret;
 }
 
@@ -141,7 +139,7 @@ krb5int_dk_cmac_decrypt(const struct krb5_keytypes *ktp, krb5_key key,
     const struct krb5_enc_provider *enc = ktp->enc;
     krb5_error_code ret;
     krb5_crypto_iov *header, *trailer;
-    krb5_data cksum;
+    krb5_data cksum = empty_data();
     krb5_key ke = NULL, ki = NULL;
 
     /* E(Confounder | Plaintext | Pad) | Checksum */