From 8ddcc29855df15961f64f594683862f3df455577 Mon Sep 17 00:00:00 2001
From: Roland Gruber <post@rolandgruber.de>
Date: Mon, 3 Nov 2003 17:48:15 +0000
Subject: [PATCH] fixed bug with identical missing suffixes, create missing
 suffixes recursivly

---
 lam/templates/initsuff.php | 84 ++++++++++++++++++++++++++++++++------
 lam/templates/main.php     |  8 ++--
 2 files changed, 75 insertions(+), 17 deletions(-)

diff --git a/lam/templates/initsuff.php b/lam/templates/initsuff.php
index 1f66a743..5495f470 100644
--- a/lam/templates/initsuff.php
+++ b/lam/templates/initsuff.php
@@ -36,32 +36,90 @@ setlanguage();
 // check if user already pressed button
 if ($_POST['add_suff'] || $_POST['cancel']) {
 	if ($_POST['add_suff']) {
-	$fail = array();
-	$errors = array();
-	$new_suff = $_POST['new_suff'];
-	$new_suff = str_replace("\\'", "", $new_suff);
-	$new_suff = explode(";", $new_suff);
-		// add entry
+		$fail = array();
+		$errors = array();
+		$new_suff = $_POST['new_suff'];
+		$new_suff = str_replace("\\'", "", $new_suff);
+		$new_suff = explode(";", $new_suff);
+		// add entries
 		for ($i = 0; $i < sizeof($new_suff); $i++) {
+			// check if entry is already present
+			$info = @ldap_search($_SESSION['ldap']->server, $new_suff[$i], "", array());
+			$res = @ldap_get_entries($_SESSION['ldap']->server, $info);
+			if ($res) continue;
 			$suff = $new_suff[$i];
 			// generate DN and attributes
 			$tmp = explode(",", $suff);
 			$name = explode("=", $tmp[0]);
 			array_shift($tmp);
 			$end = implode(",", $tmp);
-			if ($name[0] != "ou") {
-				$fail[] = $suff;
-				continue;
+			if ($name[0] != "ou") {  // add root entry
+				$attr = array();
+				$attr[$name[0]] = $name[1];
+				$attr['objectClass'] = 'organization';
+				$dn = $suff;
+				if (!@ldap_add($_SESSION['ldap']->server(), $dn, $attr)) {
+					$fail[] = $suff;
+					continue;
+				}
 			}
-			else {
+			else {  // add organizational unit
 				$name = $name[1];
 				$attr = array();
 				$attr['objectClass'] = "organizationalunit";
 				$attr['ou'] = $name;
-				$dn = "ou=" . $name . "," . $end;
+				$dn = $suff;
 				if (!@ldap_add($_SESSION['ldap']->server(), $dn, $attr)) {
-					$fail[] = $suff;
-					$error[] = ldap_error($_SESSION['ldap']->server());
+					// check if we have to add parent entries
+					if (ldap_errno($_SESSION['ldap']->server()) == 32) {
+						$temp = explode(",", $suff);
+						$subsuffs = array();
+						// make list of subsuffixes
+						for ($k = 0; $k < sizeof($temp); $k++) {
+							$part = explode("=", $temp[$k]);
+							if ($part[0] == "ou") $subsuffs[] = implode(",", array_slice($temp, $k));
+							else {
+								$subsuffs[] = implode(",", array_slice($temp, $k));
+								break;
+							}
+						}
+						// create missing entries
+						for ($k = sizeof($subsuffs) - 1; $k >= 0; $k--) {
+							// check if subsuffix is present
+							$info = @ldap_search($_SESSION['ldap']->server, $subsuffs[$k], "", array());
+							$res = @ldap_get_entries($_SESSION['ldap']->server, $info);
+							if (!$res) {
+								$suffarray = explode(",", $subsuffs[$k]);
+								$headarray = explode("=", $suffarray[0]);
+								if ($headarray[0] == "ou") {  // add ou entry
+									$attr = array();
+									$attr['objectClass'] = 'organizationalunit';
+									$attr['ou'] = $headarray[1];
+									$dn = $subsuffs[$k];
+									if (!@ldap_add($_SESSION['ldap']->server(), $dn, $attr)) {
+										$fail[] = $suff;
+										$error[] = ldap_error($_SESSION['ldap']->server());
+										break;
+									}
+								}
+								else {  // add root entry
+									$attr = array();
+									$attr['objectClass'] = 'organization';
+									$attr[$headarray[0]] = $headarray[1];
+									$dn = $subsuffs[$k];
+									if (!@ldap_add($_SESSION['ldap']->server(), $dn, $attr)) {
+										$fail[] = $suff;
+										$error[] = ldap_error($_SESSION['ldap']->server());
+										break;
+									}
+								}
+							}
+						}
+					}
+					else {
+						$fail[] = $suff;
+						$error[] = ldap_error($_SESSION['ldap']->server());
+					}
 				}
 			}
 		}
diff --git a/lam/templates/main.php b/lam/templates/main.php
index e817378f..f85eeacb 100644
--- a/lam/templates/main.php
+++ b/lam/templates/main.php
@@ -29,22 +29,22 @@ $new_suffs = array();
 if ($conf->get_UserSuffix() && ($conf->get_UserSuffix() != "")) {
 	$info = @ldap_search($_SESSION['ldap']->server, $conf->get_UserSuffix(), "", array());
 	$res = @ldap_get_entries($_SESSION['ldap']->server, $info);
-	if (!$res) $new_suffs[] = $conf->get_UserSuffix();
+	if (!$res && !in_array($conf->get_UserSuffix(), $new_suffs)) $new_suffs[] = $conf->get_UserSuffix();
 }
 if ($conf->get_GroupSuffix() && ($conf->get_GroupSuffix() != "")) {
 	$info = @ldap_search($_SESSION['ldap']->server, $conf->get_GroupSuffix(), "", array());
 	$res = @ldap_get_entries($_SESSION['ldap']->server, $info);
-	if (!$res) $new_suffs[] = $conf->get_GroupSuffix();
+	if (!$res && !in_array($conf->get_GroupSuffix(), $new_suffs)) $new_suffs[] = $conf->get_GroupSuffix();
 }
 if ($conf->get_HostSuffix() && ($conf->get_HostSuffix() != "")) {
 	$info = @ldap_search($_SESSION['ldap']->server, $conf->get_HostSuffix(), "", array());
 	$res = @ldap_get_entries($_SESSION['ldap']->server, $info);
-	if (!$res) $new_suffs[] = $conf->get_HostSuffix();
+	if (!$res && !in_array($conf->get_HostSuffix(), $new_suffs)) $new_suffs[] = $conf->get_HostSuffix();
 }
 if ($conf->get_DomainSuffix() && ($conf->get_DomainSuffix() != "")) {
 	$info = @ldap_search($_SESSION['ldap']->server, $conf->get_DomainSuffix(), "", array());
 	$res = @ldap_get_entries($_SESSION['ldap']->server, $info);
-	if (!$res) $new_suffs[] = $conf->get_DomainSuffix();
+	if (!$res && !in_array($conf->get_DomainSuffix(), $new_suffs)) $new_suffs[] = $conf->get_DomainSuffix();
 }
 
 if ($_SESSION['config']->is_samba3()) {