2002-09-02 Marcus Brinkmann <marcus@g10code.de>

	* conversion.c (_gpgme_decode_c_string): Fix off by one error in last change.
	* rungpg.c (_gpgme_append_gpg_args_from_signers): Move before
	_gpgme_op_edit so its prototype is known early on.

diff --git a/gpgme/ChangeLog b/gpgme/ChangeLog
index 65af2a477ff3742b065f5a44561c612576a2bac5..949737317c4fc447970059ba0635ffdfe505cb6a 100644 (file)
--- a/gpgme/ChangeLog
+++ b/gpgme/ChangeLog
@@ -1,5 +1,9 @@
 2002-09-02  Marcus Brinkmann  <marcus@g10code.de>
 
+       * conversion.c (_gpgme_decode_c_string): Fix off by one error in last change.
+       * rungpg.c (_gpgme_append_gpg_args_from_signers): Move before
+       _gpgme_op_edit so its prototype is known early on.
+
        * conversion.c: New file.
        * util.h: Add prototypes for _gpgme_decode_c_string and
        _gpgme_hextobyte.

diff --git a/gpgme/conversion.c b/gpgme/conversion.c
index 4029db8a1f3f378f0a0161dc6c0fe869350ba936..9e1737f63270c57dc251e4c4c908775fdbc12f56 100644 (file)
--- a/gpgme/conversion.c
+++ b/gpgme/conversion.c
@@ -58,7 +58,7 @@ _gpgme_decode_c_string (const char *src, char **destp)
 
   /* We can malloc a buffer of the same length, because the converted
      string will never be larger.  */
-  dest = xtrymalloc (strlen (src));
+  dest = xtrymalloc (strlen (src) + 1);
   if (!dest)
     return mk_error (Out_Of_Core);
 

diff --git a/gpgme/rungpg.c b/gpgme/rungpg.c
index e0a4e2181784488cad6607aff22a396039a66f4f..f67f68fdebc73a6d61982bde2cfea0ed3ea3f488 100644 (file)
--- a/gpgme/rungpg.c
+++ b/gpgme/rungpg.c
@@ -1439,6 +1439,32 @@ _gpgme_gpg_op_delete (GpgObject gpg, GpgmeKey key, int allow_secret)
 }
 
 
+static GpgmeError
+_gpgme_append_gpg_args_from_signers (GpgObject gpg,
+                                    GpgmeCtx ctx /* FIXME */)
+{
+  GpgmeError err = 0;
+  int i;
+  GpgmeKey key;
+
+  for (i = 0; (key = gpgme_signers_enum (ctx, i)); i++)
+    {
+      const char *s = gpgme_key_get_string_attr (key, GPGME_ATTR_KEYID,
+                                                NULL, 0);
+      if (s)
+       {
+         if (!err)
+           err = _gpgme_gpg_add_arg (gpg, "-u");
+         if (!err)
+           err = _gpgme_gpg_add_arg (gpg, s);
+       }
+      gpgme_key_unref (key);
+      if (err) break;
+    }
+  return err;
+}
+
+
 GpgmeError
 _gpgme_gpg_op_edit (GpgObject gpg, GpgmeKey key, GpgmeData out,
                    GpgmeCtx ctx /* FIXME */)
@@ -1487,32 +1513,6 @@ _gpgme_append_gpg_args_from_recipients (GpgObject gpg,
 }
 
 
-static GpgmeError
-_gpgme_append_gpg_args_from_signers (GpgObject gpg,
-                                    GpgmeCtx ctx /* FIXME */)
-{
-  GpgmeError err = 0;
-  int i;
-  GpgmeKey key;
-
-  for (i = 0; (key = gpgme_signers_enum (ctx, i)); i++)
-    {
-      const char *s = gpgme_key_get_string_attr (key, GPGME_ATTR_KEYID,
-                                                NULL, 0);
-      if (s)
-       {
-         if (!err)
-           err = _gpgme_gpg_add_arg (gpg, "-u");
-         if (!err)
-           err = _gpgme_gpg_add_arg (gpg, s);
-       }
-      gpgme_key_unref (key);
-      if (err) break;
-    }
-  return err;
-}
-
-
 GpgmeError
 _gpgme_gpg_op_encrypt (GpgObject gpg, GpgmeRecipients recp,
                       GpgmeData plain, GpgmeData ciph, int use_armor)