From: Nalin Dahyabhai <nalin@redhat.com>
Date: Thu, 20 Dec 2012 18:51:12 +0000 (-0500)
Subject: Only call SEC_PKCS12DecoderFinish() once
X-Git-Tag: krb5-1.12-alpha1~174
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b3efde67fc66818951d432d7e07fdc6d39b7034f;p=thirdparty%2Fkrb5.git

Only call SEC_PKCS12DecoderFinish() once

We already call SEC_PKCS12DecoderFinish() before entering the switch()
statement, so don't call it again.
---

diff --git a/src/plugins/preauth/pkinit/pkinit_crypto_nss.c b/src/plugins/preauth/pkinit/pkinit_crypto_nss.c
index 34457a6f68..2ef8ffdc03 100644
--- a/src/plugins/preauth/pkinit/pkinit_crypto_nss.c
+++ b/src/plugins/preauth/pkinit/pkinit_crypto_nss.c
@@ -2414,7 +2414,6 @@ crypto_load_pkcs12(krb5_context context,
                 }
                 break;
             default:
-                SEC_PKCS12DecoderFinish(ctx);
                 break;
             }
             pkiDebug("%s: skipping identity PKCS12 bundle \"%s\": "