Fixed aegypten bug 299
[gnupg.git] / sm / delete.c
index 53eff86..199e53e 100644 (file)
@@ -15,7 +15,8 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
+ * USA.
  */
 
 #include <config.h>
 
 /* Delete a certificate or an secret key from a key database. */
 static int
-delete_one (CTRL ctrl, const char *username)
+delete_one (ctrl_t ctrl, const char *username)
 {
   int rc = 0;
   KEYDB_SEARCH_DESC desc;
   KEYDB_HANDLE kh = NULL;
-  KsbaCert cert = NULL;
+  ksba_cert_t cert = NULL;
   int duplicates = 0;
   
   rc = keydb_classify_name (username, &desc);
@@ -67,7 +68,7 @@ delete_one (CTRL ctrl, const char *username)
     rc = keydb_get_cert (kh, &cert);
   if (!rc)
     {
-      char fpr[20];
+      unsigned char fpr[20];
 
       gpgsm_get_fingerprint (cert, 0, fpr, NULL);
 
@@ -77,8 +78,8 @@ delete_one (CTRL ctrl, const char *username)
         rc = 0;
       else if (!rc)
         {
-          KsbaCert cert2 = NULL;
-          char fpr2[20];
+          ksba_cert_t cert2 = NULL;
+          unsigned char fpr2[20];
 
           /* We ignore all duplicated certificates which might have
              been inserted due to program bugs. */
@@ -105,7 +106,14 @@ delete_one (CTRL ctrl, const char *username)
       goto leave;
     }
 
-  /* we need to search again to get back to the right position. */
+  /* We need to search again to get back to the right position. */
+  rc = keydb_lock (kh);
+  if (rc)
+    {
+      log_error (_("error locking keybox: %s\n"), gpg_strerror (rc));
+      goto leave;
+    }
+                   
   do 
     {
       keydb_search_reset (kh);
@@ -117,7 +125,7 @@ delete_one (CTRL ctrl, const char *username)
           goto leave;
         }
       
-      rc = keydb_delete (kh);
+      rc = keydb_delete (kh, duplicates ? 0 : 1);
       if (rc) 
         goto leave;
       if (opt.verbose)
@@ -140,7 +148,7 @@ delete_one (CTRL ctrl, const char *username)
 
 /* Delete the certificates specified by NAMES. */
 int
-gpgsm_delete (CTRL ctrl, STRLIST names)
+gpgsm_delete (ctrl_t ctrl, strlist_t names)
 {
   int rc;