krb5 commit: Only call SEC_PKCS12DecoderFinish() once

Greg Hudson ghudson at MIT.EDU
Mon May 13 02:00:05 EDT 2013


https://github.com/krb5/krb5/commit/b3efde67fc66818951d432d7e07fdc6d39b7034f
commit b3efde67fc66818951d432d7e07fdc6d39b7034f
Author: Nalin Dahyabhai <nalin at redhat.com>
Date:   Thu Dec 20 13:51:12 2012 -0500

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

 src/plugins/preauth/pkinit/pkinit_crypto_nss.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/src/plugins/preauth/pkinit/pkinit_crypto_nss.c b/src/plugins/preauth/pkinit/pkinit_crypto_nss.c
index 34457a6..2ef8ffd 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\": "


More information about the cvs-krb5 mailing list