From: Howard Chu <hyc@openldap.org>
Date: Mon, 8 Oct 2007 05:41:44 +0000 (+0000)
Subject: ITS#4884 fix prev commit
X-Git-Tag: OPENLDAP_REL_ENG_2_4_9~20^2~540
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=f0468901c6d7fb5c14cfadb50edd22137cb36656;p=openldap

ITS#4884 fix prev commit
---

diff --git a/servers/slapd/bconfig.c b/servers/slapd/bconfig.c
index 7576a2c2f7..64894df73a 100644
--- a/servers/slapd/bconfig.c
+++ b/servers/slapd/bconfig.c
@@ -4187,11 +4187,26 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
 		rc = LDAP_SUCCESS;
 	}
 
+	colst = count_ocs( oc_at, &nocs );
+
 	/* Check whether the Add is allowed by its parent, and do
 	 * any necessary arg setup
 	 */
 	if ( last ) {
 		rc = config_add_oc( &coptr, last, e, ca );
+		if ( rc == LDAP_CONSTRAINT_VIOLATION ) {
+			for ( i = 0; i<nocs; i++ ) {
+				/* Already checked these */
+				if ( colst[i]->co_oc->soc_kind == LDAP_SCHEMA_STRUCTURAL )
+					continue;
+				if ( colst[i]->co_ldadd &&
+					( rc = colst[i]->co_ldadd( last, e, ca ))
+						!= LDAP_CONSTRAINT_VIOLATION ) {
+					coptr = colst[i];
+					break;
+				}
+			}
+		}
 		if ( rc == LDAP_CONSTRAINT_VIOLATION ) {
 			Debug( LDAP_DEBUG_TRACE, "%s: config_add_internal: "
 				"DN=\"%s\" no structural objectClass add function\n",
@@ -4200,8 +4215,6 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
 		}
 	}
 
-	colst = count_ocs( oc_at, &nocs );
-
 	/* Add the entry but don't parse it, we already have its contents */
 	if ( rc == LDAP_COMPARE_TRUE ) {
 		rc = LDAP_SUCCESS;