[jira] Commented: (SANTUARIO-58) WINCAPI RSA Signatures Destructor ~WinCAPICryptoKeyRSA does not destroy key

["steve cullum (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SANTUARIO-58?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12968898#action_12968898 ] 

steve cullum commented on SANTUARIO-58:
---------------------------------------

Scott

Sorry to bother you, but I have just received this email from Jira and am wondering if I need to do anything.  AFAIK this was fixed a while ago so not sure what if anything I need to do.  I would love to be involved again with this project but I was under the impression it was no longer being actively developed.


Steve






> WINCAPI RSA Signatures Destructor ~WinCAPICryptoKeyRSA does not destroy key
> ---------------------------------------------------------------------------
>
>                 Key: SANTUARIO-58
>                 URL: https://issues.apache.org/jira/browse/SANTUARIO-58
>             Project: Santuario
>          Issue Type: Bug
>          Components: C++
>    Affects Versions: C++ 1.2.0
>         Environment: Operating System: All
> Platform: All
>            Reporter: Steve Cullum
>             Fix For: C++ 1.5.1
>
>
> WINCAPI RSA Signatures memory leak in destructor
> src\enc\WinCAPI\WinCAPICryptoKeyRSA.cpp
> WinCAPICryptoKeyRSA::~WinCAPICryptoKeyRSA() 
> {
>   // If we have a RSA, delete it
>   if (m_key == 0)
>     CryptDestroyKey(m_key);
>   
>   if (mp_exponent)
>     delete[] mp_exponent;
> 	
>   if (mp_modulus)
>     delete[] mp_modulus;
> };
> Due to a typo, CryptDestroyKey() never gets called :- it should read 
>   if(m_key != 0)
>     CryptDestroyKey(m_key);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.