Revert "source4/heimdal/lib/krb5/keytab_memory.c debug"
authorStefan Metzmacher <metze@samba.org>
Sat, 20 Jul 2019 11:19:37 +0000 (11:19 +0000)
committerStefan Metzmacher <metze@samba.org>
Thu, 6 Aug 2020 12:03:26 +0000 (14:03 +0200)
This reverts commit 3a21895e76637755c06e9455c49447bc35878bb1.

source4/heimdal/lib/krb5/keytab_memory.c

index 7956d32b4904dcd8c3d955a4252753ec226a4abd..0ee684d363824aec0e4d84f3e2ed2cd82be38544 100644 (file)
 
 #include "krb5_locl.h"
 
-static void krb5_keytab_entry_dump(krb5_context context,
-                                  const krb5_keytab_entry *entry,
-                                  const char *name,
-                                  int idx,
-                                  const char *func)
-{
-    const uint8_t *data = entry->keyblock.keyvalue.data;
-    //_krb5_debug(context, 1000,
-    //         "DEBUG,%s:%s[%d]: entry[kvno=%u,type=%u,len=%zu,%p, %02x%02x%02x%02x...%02x%02x%02x%02x",
-    //            func,name, idx,
-    //            entry->vno,
-    //            entry->keyblock.keytype,
-    //            entry->keyblock.keyvalue.length,
-    //            data,
-    //            data[0],
-    //            data[1],
-    //            data[2],
-    //            data[3],
-    //            data[entry->keyblock.keyvalue.length-4],
-    //            data[entry->keyblock.keyvalue.length-3],
-    //            data[entry->keyblock.keyvalue.length-2],
-    //            data[entry->keyblock.keyvalue.length-1]);
-    dprintf(2,
-               "%s:%s[%d]: entry[kvno=%u,type=%u,len=%zu,%p, %02X%02X%02X%02X...%02X%02X%02X%02X\n",
-                func,name, idx,
-                entry->vno,
-                entry->keyblock.keytype,
-                entry->keyblock.keyvalue.length,
-                data,
-                data[0],
-                data[1],
-                data[2],
-                data[3],
-                data[entry->keyblock.keyvalue.length-4],
-                data[entry->keyblock.keyvalue.length-3],
-                data[entry->keyblock.keyvalue.length-2],
-                data[entry->keyblock.keyvalue.length-1]);
-}
-
 /* memory operations -------------------------------------------- */
 
 struct mkt_data {
@@ -192,15 +153,10 @@ mkt_next_entry(krb5_context context,
               krb5_keytab_entry *entry,
               krb5_kt_cursor *c)
 {
-    krb5_error_code ret;
     struct mkt_data *d = id->data;
     if(c->fd >= d->num_entries)
        return KRB5_KT_END;
-    ret = krb5_kt_copy_entry_contents(context, &d->entries[c->fd], entry);
-    krb5_keytab_entry_dump(context, &d->entries[c->fd], "stored", c->fd, "mkt_next_entry");
-    c->fd++;
-    krb5_keytab_entry_dump(context, entry, "entry", -1, "mkt_next_entry");
-    return ret;
+    return krb5_kt_copy_entry_contents(context, &d->entries[c->fd++], entry);
 }
 
 static krb5_error_code KRB5_CALLCONV
@@ -216,7 +172,6 @@ mkt_add_entry(krb5_context context,
              krb5_keytab id,
              krb5_keytab_entry *entry)
 {
-    krb5_error_code ret;
     struct mkt_data *d = id->data;
     krb5_keytab_entry *tmp;
     tmp = realloc(d->entries, (d->num_entries + 1) * sizeof(*d->entries));
@@ -226,16 +181,8 @@ mkt_add_entry(krb5_context context,
        return ENOMEM;
     }
     d->entries = tmp;
-    ret = krb5_kt_copy_entry_contents(context, entry,
-                                      &d->entries[d->num_entries]);
-    if (ret != 0) {
-        return ret;
-    }
-
-    krb5_keytab_entry_dump(context, entry, "entry", -1, "mkt_add_entry");
-    krb5_keytab_entry_dump(context, &d->entries[d->num_entries], "stored", d->num_entries, "mkt_add_entry");
-    d->num_entries++;
-    return ret;
+    return krb5_kt_copy_entry_contents(context, entry,
+                                      &d->entries[d->num_entries++]);
 }
 
 static krb5_error_code KRB5_CALLCONV