From 2990b4fbb1acf74e98b55ce63fea3e2fe280d60e Mon Sep 17 00:00:00 2001
From: Matthieu Patou <mat@matws.net>
Date: Fri, 12 Nov 2010 19:58:09 +0300
Subject: samldb: relax groupType modification checks

Allow programs with the PROVISION control to bypass groupType checks.
This is needed by upgradeprovision for older alpha (11, 10 ...)
---
 source4/dsdb/samdb/ldb_modules/samldb.c | 59 ++++++++++++++++++---------------
 1 file changed, 32 insertions(+), 27 deletions(-)

(limited to 'source4/dsdb/samdb')

diff --git a/source4/dsdb/samdb/ldb_modules/samldb.c b/source4/dsdb/samdb/ldb_modules/samldb.c
index 4b8a303753..338b13110f 100644
--- a/source4/dsdb/samdb/ldb_modules/samldb.c
+++ b/source4/dsdb/samdb/ldb_modules/samldb.c
@@ -1281,35 +1281,40 @@ static int samldb_group_type_change(struct samldb_ctx *ac)
 	 * On each step also the group type itself
 	 * (security/distribution) is variable. */
 
-	switch (group_type) {
-	case GTYPE_SECURITY_GLOBAL_GROUP:
-	case GTYPE_DISTRIBUTION_GLOBAL_GROUP:
-		/* change to "universal" allowed */
-		if ((old_group_type == GTYPE_SECURITY_DOMAIN_LOCAL_GROUP) ||
-		    (old_group_type == GTYPE_DISTRIBUTION_DOMAIN_LOCAL_GROUP)) {
-			return LDB_ERR_UNWILLING_TO_PERFORM;
-		}
-	break;
-
-	case GTYPE_SECURITY_UNIVERSAL_GROUP:
-	case GTYPE_DISTRIBUTION_UNIVERSAL_GROUP:
-		/* each change allowed */
-	break;
-
-	case GTYPE_SECURITY_DOMAIN_LOCAL_GROUP:
-	case GTYPE_DISTRIBUTION_DOMAIN_LOCAL_GROUP:
-		/* change to "universal" allowed */
-		if ((old_group_type == GTYPE_SECURITY_GLOBAL_GROUP) ||
-		    (old_group_type == GTYPE_DISTRIBUTION_GLOBAL_GROUP)) {
+	if (ldb_request_get_control(ac->req, LDB_CONTROL_PROVISION_OID) == NULL) {
+		switch (group_type) {
+		case GTYPE_SECURITY_GLOBAL_GROUP:
+		case GTYPE_DISTRIBUTION_GLOBAL_GROUP:
+			/* change to "universal" allowed */
+			if ((old_group_type == GTYPE_SECURITY_DOMAIN_LOCAL_GROUP) ||
+			(old_group_type == GTYPE_DISTRIBUTION_DOMAIN_LOCAL_GROUP)) {
+				ldb_set_errstring(ldb,
+					"samldb: Change from security/distribution local group forbidden!");
+				return LDB_ERR_UNWILLING_TO_PERFORM;
+			}
+		break;
+
+		case GTYPE_SECURITY_UNIVERSAL_GROUP:
+		case GTYPE_DISTRIBUTION_UNIVERSAL_GROUP:
+			/* each change allowed */
+		break;
+		case GTYPE_SECURITY_DOMAIN_LOCAL_GROUP:
+		case GTYPE_DISTRIBUTION_DOMAIN_LOCAL_GROUP:
+			/* change to "universal" allowed */
+			if ((old_group_type == GTYPE_SECURITY_GLOBAL_GROUP) ||
+			(old_group_type == GTYPE_DISTRIBUTION_GLOBAL_GROUP)) {
+				ldb_set_errstring(ldb,
+					"samldb: Change from security/distribution global group forbidden!");
+				return LDB_ERR_UNWILLING_TO_PERFORM;
+			}
+		break;
+
+		case GTYPE_SECURITY_BUILTIN_LOCAL_GROUP:
+		default:
+			/* we don't allow this "groupType" values */
 			return LDB_ERR_UNWILLING_TO_PERFORM;
+		break;
 		}
-	break;
-
-	case GTYPE_SECURITY_BUILTIN_LOCAL_GROUP:
-	default:
-		/* we don't allow this "groupType" values */
-		return LDB_ERR_UNWILLING_TO_PERFORM;
-	break;
 	}
 
 	account_type =  ds_gtype2atype(group_type);
-- 
cgit