From 0463e4b3fa4d1df594cf70077f5e29ed5c5d1d0e Mon Sep 17 00:00:00 2001
From: Mikhail Efremov <sem@altlinux.org>
Date: Feb 22 2017 22:10:31 +0000
Subject: certsave: Don't try to destroy destroyed object


Function DeleteTokenPrivateKey() always calls
SECKEY_DestroyPrivateKey(), it shouldn't be called second time.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>

---

diff --git a/src/certsave-n.c b/src/certsave-n.c
index ffaebed..67deb88 100644
--- a/src/certsave-n.c
+++ b/src/certsave-n.c
@@ -667,8 +667,8 @@ cm_certsave_n_main(int fd, struct cm_store_ca *ca, struct cm_store_entry *entry,
 								       "old key.\n");
 							}
 						}
+						SECKEY_DestroyPrivateKey(privkey);
 					}
-					SECKEY_DestroyPrivateKey(privkey);
 					if (oldcert != NULL) {
 						CERT_DestroyCertificate(oldcert);
 					}