[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: (ITS#6432) PATCH: MozNSS crypto (tls_m.c) - support InitContext, improved PEM support
- To: openldap-its@OpenLDAP.org
- Subject: Re: (ITS#6432) PATCH: MozNSS crypto (tls_m.c) - support InitContext, improved PEM support
- From: hyc@symas.com
- Date: Sat, 19 Dec 2009 01:07:25 GMT
- Auto-submitted: auto-generated (OpenLDAP-ITS)
rmeggins@redhat.com wrote:
> You are right, Howard. It works as you described. I'm not sure what
> happened in my testing. Mea culpa.
>
> This is a new patch with the SetURL changes backed out.
>
> ftp://ftp.openldap.org/incoming/openldap-2.4.20-tls_m_c-InitContext-PEM-20091218-3.patch
OK. Another question - you're only using NSS_InitContext() for client
initialization. Any particular reason not to always use it?
Also in tlsm_ctx_free() is it safe to pass a NULL to NSS_ShutdownContext()? I
think these two lines should be enclosed in "if (c->tc_initctx) {}"
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/