You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@santuario.apache.org by sc...@apache.org on 2017/09/01 23:32:26 UTC

svn commit: r1807016 - /santuario/xml-security-cpp/trunk/xsec/enc/WinCAPI/WinCAPICryptoProvider.cpp

Author: scantor
Date: Fri Sep  1 23:32:26 2017
New Revision: 1807016

URL: http://svn.apache.org/viewvc?rev=1807016&view=rev
Log:
SANTUARIO-461 - Key store should be deleted on any CryptoAcquireContext() error

Modified:
    santuario/xml-security-cpp/trunk/xsec/enc/WinCAPI/WinCAPICryptoProvider.cpp

Modified: santuario/xml-security-cpp/trunk/xsec/enc/WinCAPI/WinCAPICryptoProvider.cpp
URL: http://svn.apache.org/viewvc/santuario/xml-security-cpp/trunk/xsec/enc/WinCAPI/WinCAPICryptoProvider.cpp?rev=1807016&r1=1807015&r2=1807016&view=diff
==============================================================================
--- santuario/xml-security-cpp/trunk/xsec/enc/WinCAPI/WinCAPICryptoProvider.cpp (original)
+++ santuario/xml-security-cpp/trunk/xsec/enc/WinCAPI/WinCAPICryptoProvider.cpp Fri Sep  1 23:32:26 2017
@@ -113,16 +113,12 @@ WinCAPICryptoProvider::WinCAPICryptoProv
 		m_provRSAType,
 		dwFlags)) 
 	{
-		if (GetLastError() == NTE_BAD_KEYSET) {
-
-		      CryptAcquireContext(&m_provApacheKeyStore,
+		CryptAcquireContext(&m_provApacheKeyStore,
 			                    s_xsecKeyStoreName,
 			                    provRSAName,
 			                    m_provRSAType,
 			                    CRYPT_DELETEKEYSET);
 
-		}
-
 		// Try to create
 		if (!CryptAcquireContext(&m_provApacheKeyStore,
 			s_xsecKeyStoreName,