[xmlsec] Big patch to xmlsec in recent OpenOffice.org sources

Andrew Fan Xuelei.Fan at Sun.COM
Sun Feb 27 20:08:52 PST 2005


Aleksey Sanin wrote:

>>>>> 1) xmlsec/include/xmlsec/mscrypto/akmngr.c, 
>>>>> xmlsec/src/mscrypto/akmngr.c
>>>>> Why do you need "AppliedKeyManager"? How is it different from the
>>>>> "DefaultKeyManager" and do you think it would be easier to just
>>>>> merge the two?
>>>>>
>>>> The AppliedKeyManager enable user specify their preferred key store 
>>>> and certificate store. It would be a good idea just simply support 
>>>> both of the two manager.
>>>>
>>> I would really love to merge these two guys together. May be we can 
>>> just
>>> add functions to set prefered key/certs store to the DefaultKeysManager
>>> and provide reasonable defaults as we do now. I believe this way we can
>>> avoid un-necessary code duplication (see item 0) too).
>>>
>> It's a little harder, but definitely better. :-)
>
> It's not that hard and I already mostly done with it. The only
> question I have is why do you have the following code at the end
> of xmlSecMSCryptoX509StoreAdoptXXXX functions? Is this just for
> debugging or there is other reason for it?
>
>     {
>         PCCERT_CONTEXT ptCert ;
>
>         ptCert = NULL ;
>         while( 1 ) {
>             ptCert = CertEnumCertificatesInStore( ctx->trusted, ptCert 
> ) ;
>             if( ptCert == NULL )
>                 break ;
>         }
>     }
>
It looks quite strange to me either. It much more like a debugging. I 
think you can get rid of it completely if no guys declare it is pretty  
much important. :-P

-Andrew

> Thanks,
> Aleksey
> _______________________________________________
> xmlsec mailing list
> xmlsec at aleksey.com
> http://www.aleksey.com/mailman/listinfo/xmlsec





More information about the xmlsec mailing list