perhaps the assert can be removed; should work now

servers/slapd/back-bdb/tools.c

@@ -123,7 +123,7 @@ ID bdb_tool_entry_put(
 
 	assert( text );
 	assert( text->bv_val );
-	assert( text->bv_val[0] == '\0' );
+	assert( text->bv_val[0] == '\0' );	/* overconservative? */
 
 #ifdef NEW_LOGGING
 	LDAP_LOG ( TOOLS, ARGS, "=> bdb_tool_entry_put( %ld, \"%s\" )\n",

servers/slapd/back-ldbm/tools.c

@@ -179,7 +179,7 @@ ID ldbm_tool_entry_put(
 
 	assert( text );
 	assert( text->bv_val );
-	assert( text->bv_val[0] == '\0' );
+	assert( text->bv_val[0] == '\0' );	/* overconservative? */
 
 	if ( next_id_get( be, &id ) || id == NOID ) {
 		strncpy( text->bv_val, "unable to get nextid", text->bv_len );

servers/slapd/tools/slapadd.c

@@ -59,8 +59,12 @@ main( int argc, char **argv )
 		Entry *e = str2entry( buf );
 		struct berval bvtext;
 
+		/*
+		 * Initialize text buffer
+		 */
 		bvtext.bv_len = textlen;
 		bvtext.bv_val = textbuf;
+		bvtext.bv_val[0] = '\0';
 
 		if( e == NULL ) {
 			fprintf( stderr, "%s: could not parse entry (line=%d)\n",