minor cleanup

[openldap] / servers / slapd / ava.c

diff --git a/servers/slapd/ava.c b/servers/slapd/ava.c
index ad0a42f98299beb386a55f4ab307bfead7c251cc..da6a2e3355ae37a296b8e9b24387ad04d394e690 100644 (file)
--- a/servers/slapd/ava.c
+++ b/servers/slapd/ava.c
@@ -14,7 +14,6 @@
 
 #include "slap.h"
 
-#ifdef SLAPD_SCHEMA_NOT_COMPAT
 
 void
 ava_free(
@@ -22,44 +21,61 @@ ava_free(
     int        freeit
 )
 {
-       ad_free( ava->aa_desc, 1 );
        ber_bvfree( ava->aa_value );
        if ( freeit ) {
                ch_free( (char *) ava );
        }
 }
 
-#else
-
-void
-ava_free(
-    Ava        *ava,
-    int        freeit
-)
-{
-       ch_free( (char *) ava->ava_type );
-       ch_free( (char *) ava->ava_value.bv_val );
-       if ( freeit ) {
-               ch_free( (char *) ava );
-       }
-}
-
 int
 get_ava(
     BerElement *ber,
-    Ava                *ava
+    AttributeAssertion **ava,
+       unsigned usage,
+       const char **text
 )
 {
-       if ( ber_scanf( ber, "{ao}", &ava->ava_type, &ava->ava_value )
-           == LBER_ERROR ) {
+       int rc;
+       struct berval type, value, *nvalue;
+       AttributeAssertion *aa;
+
+       rc = ber_scanf( ber, "{oo}", &type, &value );
+
+       if( rc == LBER_ERROR ) {
+#ifdef NEW_LOGGING
+               LDAP_LOG(( "filter", LDAP_LEVEL_ERR,
+                          "get_ava:  ber_scanf failure\n" ));
+#else
                Debug( LDAP_DEBUG_ANY, "  get_ava ber_scanf\n", 0, 0, 0 );
+#endif
+               *text = "Error decoding attribute value assertion";
                return SLAPD_DISCONNECT;
        }
 
-       attr_normalize( ava->ava_type );
-       value_normalize( ava->ava_value.bv_val, attr_syntax( ava->ava_type ) );
+       aa = ch_malloc( sizeof( AttributeAssertion ) );
+       aa->aa_desc = NULL;
+       aa->aa_value = NULL;
+
+       rc = slap_bv2ad( &type, &aa->aa_desc, text );
+       ch_free( type.bv_val );
+
+       if( rc != LDAP_SUCCESS ) {
+               ch_free( value.bv_val );
+               ch_free( aa );
+               return rc;
+       }
+
+       rc = value_normalize( aa->aa_desc, usage, &value, &nvalue, text );
+       ch_free( value.bv_val );
+
+       if( rc != LDAP_SUCCESS ) {
+               ch_free( aa );
+               return rc;
+       }
+
+       aa->aa_value = nvalue;
+       *ava = aa;
 
        return LDAP_SUCCESS;
 }
 
-#endif