[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: commit: ldap/servers/slapd bconfig.c

To: Ralf Haferkamp <rhafer@suse.de>
Subject: Re: commit: ldap/servers/slapd bconfig.c
From: Howard Chu <hyc@symas.com>
Date: Sun, 15 Nov 2009 12:37:21 -0800
Cc: OpenLDAP Commit <openldap-commit2devel@openldap.org>
In-reply-to: <200911061634.59420.rhafer@suse.de>
References: <200911051438.nA5Ec3ZD070034@cantor.openldap.org> <200911061634.59420.rhafer@suse.de>
User-agent: Mozilla/5.0 (X11; U; Linux i686; rv:1.9.1.5pre) Gecko/20091025 Lightning/1.0pre SeaMonkey/2.0a1pre Firefox/3.0.3

Ralf Haferkamp wrote:
> Am Donnerstag 05 November 2009 15:38:03 schrieb ralf@openldap.org:
>> Update of /repo/OpenLDAP/pkg/ldap/servers/slapd
>>
>> Modified Files:
>> 	bconfig.c  1.395 -> 1.396
>>
>> Log Message:
>> Initialize ConfigArgs before calling config_del_vals (ITS#6363)
> 
> Hm, after looking at this again I wonder if it would be better to move the 
> c->argv[] initialization into config_del_vals(). To avoid the duplicated code 
> that this patch created.
> 
Sounds good. Done.

-- 
  -- Howard Chu
  CTO, Symas Corp.           http://www.symas.com
  Director, Highland Sun     http://highlandsun.com/hyc/
  Chief Architect, OpenLDAP  http://www.openldap.org/project/

References:
- Re: commit: ldap/servers/slapd bconfig.c
  - From: Ralf Haferkamp <rhafer@suse.de>

Prev by Date: Re: TLS renegotiation
Next by Date: Please test RE24 for 5.0.20 candidate
Index(es):
- Chronological
- Thread