Add LDAP_VALID and other asserts.

libraries/libldap/addentry.c

@@ -27,6 +27,9 @@ ldap_delete_result_entry( LDAPMessage **list, LDAPMessage *e )
 {
 	LDAPMessage	*tmp, *prev = NULL;
 
+	assert( list != NULL );
+	assert( e != NULL );
+
 	for ( tmp = *list; tmp != NULL && tmp != e; tmp = tmp->lm_chain )
 		prev = tmp;
 
@@ -45,6 +48,9 @@ ldap_delete_result_entry( LDAPMessage **list, LDAPMessage *e )
 void
 ldap_add_result_entry( LDAPMessage **list, LDAPMessage *e )
 {
+	assert( list != NULL );
+	assert( e != NULL );
+
 	e->lm_chain = *list;
 	*list = e;
 }

libraries/libldap/getattr.c

@@ -46,7 +46,7 @@ ldap_first_attribute( LDAP *ld, LDAPMessage *entry, BerElement **ber )
 	 * positioned right before the next attribute type/value sequence.
 	 */
 
-	if ( ber_scanf( *ber, "{x{{ax}", &attr )
+	if ( ber_scanf( *ber, "{x{{ax}" /*}}*/, &attr )
 	    == LBER_ERROR ) {
 		ld->ld_errno = LDAP_DECODING_ERROR;
 		ber_free( *ber, 0 );

libraries/libldap/getentry.c

@@ -27,6 +27,7 @@ ldap_first_entry( LDAP *ld, LDAPMessage *chain )
 {
 	assert( ld != NULL );
 	assert( LDAP_VALID( ld ) );
+	assert( chain != NULL );
 
 	if( ld == NULL || chain == NULL ) {
 		return NULL;
@@ -42,6 +43,7 @@ ldap_next_entry( LDAP *ld, LDAPMessage *entry )
 {
 	assert( ld != NULL );
 	assert( LDAP_VALID( ld ) );
+	assert( entry != NULL );
 
 	if ( ld == NULL || entry == NULL ) {
 		return NULL;

libraries/libldap/getvalues.c

@@ -30,6 +30,11 @@ ldap_get_values( LDAP *ld, LDAPMessage *entry, LDAP_CONST char *target )
 	int		found = 0;
 	char		**vals;
 
+	assert( ld != NULL );
+	assert( LDAP_VALID( ld ) );
+	assert( entry != NULL );
+	assert( target != NULL );
+
 	Debug( LDAP_DEBUG_TRACE, "ldap_get_values\n", 0, 0, 0 );
 
 	ber = *entry->lm_ber;
@@ -82,6 +87,11 @@ ldap_get_values_len( LDAP *ld, LDAPMessage *entry, LDAP_CONST char *target )
 	int		found = 0;
 	struct berval	**vals;
 
+	assert( ld != NULL );
+	assert( LDAP_VALID( ld ) );
+	assert( entry != NULL );
+	assert( target != NULL );
+
 	Debug( LDAP_DEBUG_TRACE, "ldap_get_values_len\n", 0, 0, 0 );
 
 	ber = *entry->lm_ber;

libraries/libldap/references.c

@@ -19,11 +19,11 @@
 
 #include "ldap-int.h"
 
-/* ARGSUSED */
 LDAPMessage *
 ldap_first_reference( LDAP *ld, LDAPMessage *chain )
 {
 	assert( ld != NULL );
+	assert( LDAP_VALID( ld ) );
 	assert( chain !=  NULL );
 
 	if ( ld == NULL || chain == NULL ) {
@@ -39,6 +39,7 @@ LDAPMessage *
 ldap_next_reference( LDAP *ld, LDAPMessage *ref )
 {
 	assert( ld != NULL );
+	assert( LDAP_VALID( ld ) );
 	assert( ref !=  NULL );
 
 	if ( ld == NULL || ref == NULL ) {
@@ -64,12 +65,14 @@ ldap_count_references( LDAP *ld, LDAPMessage *chain )
 	int	i;
 
 	assert( ld != NULL );
+	assert( LDAP_VALID( ld ) );
 	assert( chain !=  NULL );
 
 	if ( ld == NULL ) {
 		return -1;
 	}
 
+	
 	for ( i = 0; chain != NULL; chain = chain->lm_chain ) {
 		if( chain->lm_msgtype == LDAP_RES_SEARCH_REFERENCE ) {
 			i++;
@@ -92,6 +95,7 @@ ldap_parse_reference(
 	int rc;
 
 	assert( ld != NULL );
+	assert( LDAP_VALID( ld ) );
 	assert( ref !=  NULL );
 
 	if( ld == NULL || ref == NULL ||