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

Re: commit: ldap/servers/slapd/back-bdb init.c

Ralf Haferkamp wrote:
On Freitag, 12. September 2008, ralf@openldap.org wrote: 
Update of /repo/OpenLDAP/pkg/ldap/servers/slapd/back-bdb

Modified Files:
	init.c  1.293 ->  1.294

Log Message:
stop and remove checkpoint task in _db_destroy (ITS#5698)


Generally I'd think that this would better fit into the bdb_db_close()
function, but the _db_close hooks are not called from
backend_destroy_one() (which is what config_add_internal() calls when
it fails). Is there a special reason for that?

I think the assumption is that db_open failed, therefore db_close is not needed.

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