slaptest core dump (ITS#3328)

[Dale.Moore@cs.cmu.edu]

Full_Name: Dale Moore
Version: 2.2.15
OS: Solaris 5.9
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (128.2.203.2)


slaptest with bdb appears to dump core after
reporting 'config file testing succeeded'.
The problem is that we are freeing a lock that
is never allocated.  We call XLOCK_ID_FREE without
ever calling XLOCK_ID
 diff -c servers/slapd/back-bdb/init.c{~,}
*** servers/slapd/back-bdb/init.c~      Fri Jul 16 15:51:43 2004
--- servers/slapd/back-bdb/init.c       Mon Sep 13 14:42:39 2004
***************
*** 490,496 ****
                ldap_pvt_thread_rdwr_wunlock ( &bdb->bi_idl_tree_rwlock );
        }
  
!       XLOCK_ID_FREE(bdb->bi_dbenv, bdb->bi_cache.c_locker);
  
        return 0;
  }
--- 490,498 ----
                ldap_pvt_thread_rdwr_wunlock ( &bdb->bi_idl_tree_rwlock );
        }
  
!       if (bdb->bi_dbenv) {
!               XLOCK_ID_FREE(bdb->bi_dbenv, bdb->bi_cache.c_locker);
!       }
  
        return 0;
  }