]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Use zapfreedata() where appropriate 906/head
authorGreg Hudson <ghudson@mit.edu>
Thu, 14 Mar 2019 15:54:00 +0000 (11:54 -0400)
committerGreg Hudson <ghudson@mit.edu>
Fri, 15 Mar 2019 15:38:34 +0000 (11:38 -0400)
src/lib/krb5/krb/mk_cred.c
src/lib/krb5/krb/mk_priv.c
src/lib/krb5/krb/mk_safe.c
src/lib/krb5/krb/rd_safe.c

index 62224f1cf0e11aade1ac804e8d7f021750735981..c0d7b38125904417c2df4a22ee4af91eb17e0514 100644 (file)
@@ -59,8 +59,7 @@ encrypt_credencpart(krb5_context context, krb5_cred_enc_part *encpart,
     ret = k5_encrypt_keyhelper(context, key, KRB5_KEYUSAGE_KRB_CRED_ENCPART,
                                der_enccred, encdata_out);
 
-    zap(der_enccred->data, der_enccred->length);
-    krb5_free_data(context, der_enccred);
+    zapfreedata(der_enccred);
     return ret;
 }
 
@@ -208,10 +207,7 @@ cleanup:
     krb5_free_data_contents(context, &enc.ciphertext);
     free(lstorage.contents);
     free(rstorage.contents);
-    if (der_krbcred != NULL) {
-        zap(der_krbcred->data, der_krbcred->length);
-        krb5_free_data(context, der_krbcred);
-    }
+    zapfreedata(der_krbcred);
     return ret;
 }
 
index d0231697a21d68205dbb25ed44409096483ce56f..b537a2ca18a596b72cc0d5faf03a8885e5fcccf9 100644 (file)
@@ -50,7 +50,7 @@ create_krbpriv(krb5_context context, const krb5_data *userdata,
     krb5_error_code ret;
     krb5_priv privmsg;
     krb5_priv_enc_part encpart;
-    krb5_data *der_encpart, *der_krbpriv;
+    krb5_data *der_encpart = NULL, *der_krbpriv;
     size_t enclen;
 
     memset(&privmsg, 0, sizeof(privmsg));
@@ -97,10 +97,7 @@ create_krbpriv(krb5_context context, const krb5_data *userdata,
 cleanup:
     zapfree(privmsg.enc_part.ciphertext.data,
             privmsg.enc_part.ciphertext.length);
-    if (der_encpart != NULL) {
-        zap(der_encpart->data, der_encpart->length);
-        krb5_free_data(context, der_encpart);
-    }
+    zapfreedata(der_encpart);
     return ret;
 }
 
index 8a106160e43b46dac0daa1858a74323cff56b8ba..151a06e61e6fcd97b8bb669205bed19a1df63163 100644 (file)
@@ -78,8 +78,7 @@ create_krbsafe(krb5_context context, const krb5_data *userdata, krb5_key key,
     ret = krb5_k_make_checksum(context, sumtype, key,
                                KRB5_KEYUSAGE_KRB_SAFE_CKSUM, der_krbsafe,
                                &safe_checksum);
-    zap(der_krbsafe->data, der_krbsafe->length);
-    krb5_free_data(context, der_krbsafe);
+    zapfreedata(der_krbsafe);
     if (ret)
         return ret;
 
index 17f2b278f6333a7903b73ffe8db72293fae6de44..7b7a30cd4ee89fb1cb97cd5b2067184902674205 100644 (file)
@@ -117,10 +117,7 @@ read_krbsafe(krb5_context context, krb5_auth_context ac,
     safe_cksum = NULL;
 
 cleanup:
-    if (der_zerosafe != NULL) {
-        zap(der_zerosafe->data, der_zerosafe->length);
-        krb5_free_data(context, der_zerosafe);
-    }
+    zapfreedata(der_zerosafe);
     krb5_free_data(context, safe_body);
     krb5_free_safe(context, krbsafe);
     krb5_free_checksum(context, safe_cksum);