* keytab.c (add_principal): Don't call kadm5_free_key_data; that's

	Just Wrong here and was causing coredumps.

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@10102 dc483132-0cff-0310-8789-dd5450dbe970

diff --git a/src/kadmin/cli/ChangeLog b/src/kadmin/cli/ChangeLog
index 0869c123659eaf1ca2f6f3323392f686984b13ac..3bf88a6827f3a1eafef1b65c1bc05d45c31808a4 100644 (file)
--- a/src/kadmin/cli/ChangeLog
+++ b/src/kadmin/cli/ChangeLog
@@ -1,3 +1,8 @@
+Fri Jun 27 17:37:07 1997  Tom Yu  <tlyu@mit.edu>
+
+       * keytab.c (add_principal): Don't call kadm5_free_key_data; that's
+       Just Wrong here and was causing coredumps.
+
 Wed May 28 13:33:40 1997  Barry Jaspan  <bjaspan@mit.edu>
 
        * keytab.c (add_principal): use kadm5_free_key_data

diff --git a/src/kadmin/cli/keytab.c b/src/kadmin/cli/keytab.c
index 56724491f04b3204b1e3535d73fe4cfd037e8c99..df4f6ce257cd15f2eb1d22a22613d0303ac51f82 100644 (file)
--- a/src/kadmin/cli/keytab.c
+++ b/src/kadmin/cli/keytab.c
@@ -255,7 +255,11 @@ int add_principal(void *handle, char *keytab_str, krb5_keytab keytab,
      }
 
 cleanup:
-     kadm5_free_key_data(handle, &nkeys, &keys);
+     if (nkeys) {
+         for (i = 0; i < nkeys; i++)
+              krb5_free_keyblock_contents(context, &keys[i]);
+         free(keys);
+     }
      if (princ)
          krb5_free_principal(context, princ);