X-Git-Url: https://git.distorted.org.uk/u/mdw/catacomb/blobdiff_plain/7675ae6bdd4a8372b734ebcb90ee9b6fe13cc14a..78614e02310dbe879d55f0a68e47349db074ff61:/key-io.c

diff --git a/key-io.c b/key-io.c
index 7ed0c2c..5c70909 100644
--- a/key-io.c
+++ b/key-io.c
@@ -7,7 +7,7 @@
  * (c) 1999 Straylight/Edgeware
  */
 
-/*----- Licensing notice --------------------------------------------------* 
+/*----- Licensing notice --------------------------------------------------*
  *
  * This file is part of Catacomb.
  *
@@ -15,12 +15,12 @@
  * it under the terms of the GNU Library General Public License as
  * published by the Free Software Foundation; either version 2 of the
  * License, or (at your option) any later version.
- * 
+ *
  * Catacomb is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU Library General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU Library General Public
  * License along with Catacomb; if not, write to the Free
  * Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
@@ -196,12 +196,12 @@ int key_merge(key_file *f, const char *file, FILE *fp,
      *
      * There are currently six fields of interest:
      *
-     *   * The key's identification (id, tag and type).
-     *   * The actual key data itself.
-     *   * The key expiry time.
-     *   * The key deletion time.
-     *   * The attributes field.
-     *   * Any further comments.
+     *	 * The key's identification (id, tag and type).
+     *	 * The actual key data itself.
+     *	 * The key expiry time.
+     *	 * The key deletion time.
+     *	 * The attributes field.
+     *	 * Any further comments.
      *
      * All but the last field can contain no spaces.
      */
@@ -221,7 +221,7 @@ int key_merge(key_file *f, const char *file, FILE *fp,
 
     /* --- Extract the key data into the block --- */
 
-    if (key_read(vf[1], &k->k, 0)) {
+    if ((k->k = key_read(vf[1], 0)) == 0) {
       if (rep)
 	rep(file, line, "bad key data", arg);
       goto skip_1;
@@ -239,12 +239,12 @@ int key_merge(key_file *f, const char *file, FILE *fp,
       char *qq;
 
       if (!q) {
-	if (k->k.e != KENC_BINARY) {
+	if (k->k->e != KENC_BINARY) {
 	  if (rep)
 	    rep(file, line, "new-style key encoding but no keyid", arg);
 	  goto skip_2;
 	}
-	k->id = crc32(0, k->k.u.k.k, k->k.u.k.sz);
+	k->id = crc32(0, k->k->u.k.k, k->k->u.k.sz);
 	k->type = xstrdup(vf[0]);
 	k->tag = 0;
       } else {
@@ -286,7 +286,7 @@ int key_merge(key_file *f, const char *file, FILE *fp,
 	goto skip_3;
       }
     }
- 
+
     /* --- Parse up the attributes, if specified --- */
 
     sym_create(&k->a);
@@ -313,7 +313,7 @@ int key_merge(key_file *f, const char *file, FILE *fp,
       xfree(k->tag);
     xfree(k->type);
   skip_2:
-    key_destroy(&k->k);
+    key_drop(k->k);
   skip_1:
     DESTROY(k);
   skip_0:;
@@ -347,14 +347,14 @@ int key_extract(key_file *f, key *k, FILE *fp, const key_filter *kf)
 
   /* --- Skip the key if it's deleted or unselected--- */
 
-  if (KEY_EXPIRED(t, k->del) || !key_match(&k->k, kf))
+  if (KEY_EXPIRED(t, k->del) || !key_match(k->k, kf))
     return (0);
 
   /* --- Encode the key and write the easy stuff --- */
 
   key_fulltag(k, &d);
   DPUTC(&d, ' ');
-  key_write(&k->k, &d, kf);
+  key_write(k->k, &d, kf);
   DPUTC(&d, ' ');
   dstr_write(&d, fp);
   DRESET(&d);
@@ -469,7 +469,7 @@ void key_discard(key_file *f)
     key_attr *a;
     key *k = (key *)b;
 
-    key_destroy(&k->k);
+    if (k->k) key_drop(k->k);
     xfree(k->type);
     xfree(k->tag);
     if (k->c)
@@ -514,10 +514,9 @@ int key_close(key_file *f)
  *		@uint32 id@ = keyid to set
  *		@const char *type@ = the type of this key
  *		@time_t exp@ = when the key expires
- *		@int *err@ = where to store the error condition
+ *		@key *kk@ = where to put the key pointer
  *
- * Returns:	Key block containing new data, or null if it couldn't be
- *		done.
+ * Returns:	Error code (one of the @KERR@ constants).
  *
  * Use:		Attaches a new key to a key file.  You must have a writable
  *		key file for this to work.
@@ -532,11 +531,9 @@ int key_close(key_file *f)
  *		key'.  Be careful with `forever' keys.  If I were you, I'd
  *		use a more sophisticated key management system than this for
  *		them.
- *
- *		You have to set the actual key yourself.
  */
 
-key *key_new(key_file *f, uint32 id, const char *type, time_t exp, int *err)
+int key_new(key_file *f, uint32 id, const char *type, time_t exp, key **kk)
 {
   key *k = 0;
   time_t t = time(0);
@@ -554,21 +551,21 @@ key *key_new(key_file *f, uint32 id, const char *type, time_t exp, int *err)
     k = CREATE(key);
     k->id = id;
     k->tag = 0;
-    k->type = xstrdup(type);
     k->exp = k->del = exp;
     k->c = 0;
-    k->k.e = 0;
+    k->type = (char *)type; /* temporarily */
     sym_create(&k->a);
-    if ((e = insert(f, k)) == 0)
-      f->f |= KF_MODIFIED;
-    else {
-      xfree(k->type);
+    if ((e = insert(f, k)) != 0)
       DESTROY(k);
-      k = 0;
+    else {
+      k->k = key_newstring(KCAT_SHARE, "<unset>");
+      k->type = xstrdup(type);
+      *kk = k;
+      f->f |= KF_MODIFIED;
     }
   }
-  *err = e;
-  return (k);
+
+  return (e);
 }
 
 /*----- That's all, folks -------------------------------------------------*/