3 messages in net.sourceforge.lists.courier-usersRe: [courier-users] LDAP patch
FromSent OnAttachments
Michael RichardOct 28, 2005 10:52 am.patch
Mark BucciarelliOct 28, 2005 12:19 pm 
Michael RichardOct 28, 2005 1:10 pm 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:Re: [courier-users] LDAP patchActions...
From:Mark Bucciarelli (ma@gaiahost.coop)
Date:Oct 28, 2005 12:19:51 pm
List:net.sourceforge.lists.courier-users

On Fri, Oct 28, 2005 at 01:50:42PM -0400, Michael Richard wrote:

Hi,

I (finally) revisited my LDAP over SSL patch. Attached with this email is a new patch that will allow both the "old" LDAP_SERVER / LDAP_PORT and the new LDAP_URI settings.

I suspect there might be a memory leak caused by this patch. If someone better than I in C wants to double check, you're highly encouraged to do so ;)

I read the patch--a couple questions:

- does the routine that frees the ldap struct free the uri char*?

- are you positive there can be no buffer overflow with the sprintf's? for example, if somehow hostname was longer that 250 characters. there must be lots of other similar code in courier sources--does it use sprintnf?

hth,

m