2005-09-12 Marcus Brinkmann <marcus@g10code.de>

	* keylist.c (release_op_data): Do not free opd->tmp_uid.

diff --git a/gpgme/ChangeLog b/gpgme/ChangeLog
index 9affd16659cb6cabb5bb2adef04ebc2538745cf0..53c707be7af730edb6d6bafc0aeafa9a1dfc32ad 100644 (file)
--- a/gpgme/ChangeLog
+++ b/gpgme/ChangeLog
@@ -1,3 +1,7 @@
+2005-09-12  Marcus Brinkmann  <marcus@g10code.de>
+
+       * keylist.c (release_op_data): Do not free opd->tmp_uid.
+
 2005-08-26  Marcus Brinkmann  <marcus@g10code.de>
 
        * rungpg.c (command_handler): Use _gpgme_io_write instead of write.

diff --git a/gpgme/keylist.c b/gpgme/keylist.c
index e26ddf096cf5a67fc032d26333674ff6aa24fe1e..8f9cf3750001f3bb69c21f27f74f7c4fac74a8cd 100644 (file)
--- a/gpgme/keylist.c
+++ b/gpgme/keylist.c
@@ -48,6 +48,7 @@ typedef struct
   struct _gpgme_op_keylist_result result;
 
   gpgme_key_t tmp_key;
+  /* This points to the last uid in tmp_key.  */
   gpgme_user_id_t tmp_uid;
   /* Something new is available.  */
   int key_cond;
@@ -63,8 +64,9 @@ release_op_data (void *hook)
 
   if (opd->tmp_key)
     gpgme_key_unref (opd->tmp_key);
-  if (opd->tmp_uid)
-    free (opd->tmp_uid);
+  /* opd->tmp_uid is actually part of opd->tmp_key, so we do not need
+     to release it here.  */
+
   while (key)
     {
       struct key_queue_item_s *next = key->next;