Code

Updated ACL handling in groups
[gosa.git] / gosa-core / include / class_ldap.inc
index 96a7e56ee679444c3902766fca45e608bd821bf2..a1b6a87022d1cf05efebaac63ecfcb7ae7340ce0 100644 (file)
@@ -1,12 +1,26 @@
 <?php
-/*****************************************************************************
-  newldap.inc - version 1.0
-  Copyright (C) 2003 Alejandro Escanero Blanco <aescanero@chaosdimension.org>
-  Copyright (C) 2004-2006 Cajus Pollmeier <pollmeier@gonicus.de>
-
-  Based in code of ldap.inc of
-  Copyright (C) 1998  Eric Kilfoil <eric@ipass.net>
- *****************************************************************************/
+/*
+ * This code is part of GOsa (http://www.gosa-project.org)
+ * Copyright (C) 2003-2008 GONICUS GmbH
+ * Copyright (C) 2003 Alejandro Escanero Blanco <aescanero@chaosdimension.org>
+ * Copyright (C) 1998  Eric Kilfoil <eric@ipass.net>
+ *
+ * ID: $$Id$$
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ */
 
 define("ALREADY_EXISTING_ENTRY",-10001);
 define("UNKNOWN_TOKEN_IN_LDIF_FILE",-10002);
@@ -17,13 +31,16 @@ define("SPECIALS_OVERRIDE", TRUE);
 class LDAP{
 
   var $hascon   =false;
-  var $hasres   =false;
   var $reconnect=false;
   var $tls      = false;
-  var $basedn   ="";
   var $cid;
+  var $hasres   = array();
+  var $sr       = array();
+  var $re       = array();
+  var $basedn   ="";
+  var $start    = array(); // 0 if we are fetching the first entry, otherwise 1
   var $error    = ""; // Any error messages to be returned can be put here
-  var $start    = 0; // 0 if we are fetching the first entry, otherwise 1
+  var $srp      = 0;
   var $objectClasses = array(); // Information read from slapd.oc.conf
   var $binddn   = "";
   var $bindpw   = "";
@@ -43,8 +60,8 @@ class LDAP{
     $this->hostname=$hostname;
 
     /* Check if MAX_LDAP_QUERY_TIME is defined */ 
-    if(isset($config->data['MAIN']['MAX_LDAP_QUERY_TIME'])){
-      $str = $config->data['MAIN']['MAX_LDAP_QUERY_TIME'];
+    if(is_object($config) && $config->get_cfg_value("ldapMaxQueryTime") != ""){
+      $str = $config->get_cfg_value("ldapMaxQueryTime");
       $this->max_ldap_query_time = (float)($str);
     }
 
@@ -52,22 +69,32 @@ class LDAP{
   }
 
 
+  function getSearchResource()
+  {
+    $this->sr[$this->srp]= NULL;
+    $this->start[$this->srp]= 0;
+    $this->hasres[$this->srp]= false;
+    return $this->srp++;
+  }
+
+
   /* Function to replace all problematic characters inside a DN by \001XX, where
      \001 is decoded to chr(1) [ctrl+a]. It is not impossible, but very unlikely
      that this character is inside a DN.
-     
+
      Currently used codes:
-      ,   => CO
-      \2C => CO
-      (   => OB
-      )   => CB
-      /   => SL                                                                  */
+     ,   => CO
+     \2C => CO
+     (   => OB
+     )   => CB
+     /   => SL                                                                  
+     \22 => DQ                                                                  */
   static function convert($dn)
   {
     if (SPECIALS_OVERRIDE == TRUE){
-      $tmp= preg_replace(array("/\\\\,/", "/\\\\2C/", "/\(/", "/\)/", "/\//"),
-                           array("\001CO", "\001CO", "\001OB", "\001CB", "\001SL"),
-                           $dn);
+      $tmp= preg_replace(array("/\\\\,/", "/\\\\2C/", "/\(/", "/\)/", "/\//", "/\\\\22/", '/\\\\"/'),
+          array("\001CO", "\001CO", "\001OB", "\001CB", "\001SL", "\001DQ", "\001DQ"),
+          $dn);
       return (preg_replace('/,\s+/', ',', $tmp));
     } else {
       return ($dn);
@@ -82,14 +109,22 @@ class LDAP{
   static function fix($dn)
   {
     if (SPECIALS_OVERRIDE == TRUE){
-      return (preg_replace(array("/\001CO/", "/\001OB/", "/\001CB/", "/\001SL/"),
-                           array("\,", "(", ")", "/"),
-                           $dn));
+      return (preg_replace(array("/\001CO/", "/\001OB/", "/\001CB/", "/\001SL/", "/\001DQ/"),
+            array("\,", "(", ")", "/", '\"'),
+            $dn));
     } else {
       return ($dn);
     }
   }
 
+  /* Function to fix problematic characters in DN's that are used for search
+     requests. I.e. member=....                                               */
+  static function prepare4filter($dn)
+  {
+    $fixed= normalizeLdap(str_replace('\\\\', '\\\\\\', LDAP::fix($dn)));
+    return str_replace('\\,', '\\\\,', $fixed);
+  }
+
 
   function connect()
   {
@@ -126,13 +161,13 @@ class LDAP{
   function rebind($ldap, $referral)
   {
     $credentials= $this->get_credentials($referral);
-    if (@ldap_bind($ldap, LDAP::fix($credentials['ADMIN']), $credentials['PASSWORD'])) {
+    if (@ldap_bind($ldap, LDAP::fix($credentials['ADMINDN']), $credentials['ADMINPASSWORD'])) {
       $this->error = "Success";
       $this->hascon=true;
       $this->reconnect= true;
       return (0);
     } else {
-      $this->error = "Could not bind to " . $credentials['ADMIN'];
+      $this->error = "Could not bind to " . $credentials['ADMINDN'];
       return NULL;
     }
   }
@@ -161,100 +196,116 @@ class LDAP{
 
   function cd($dir)
   {
-    if ($dir == "..")
+    if ($dir == ".."){
       $this->basedn = $this->getParentDir();
-    else
+    } else {
       $this->basedn = LDAP::convert($dir);
+    }
   }
 
   function getParentDir($basedn = "")
   {
-    if ($basedn=="")
+    if ($basedn==""){
       $basedn = $this->basedn;
-    else
-      $basedn = LDAP::convert($this->basedn);
-    return(ereg_replace("[^,]*[,]*[ ]*(.*)", "\\1", $basedn));
+    } else {
+      $basedn = LDAP::convert($basedn);
+    }
+    return(preg_replace("/[^,]*[,]*[ ]*(.*)/", "$1", $basedn));
   }
 
-  function search($filter, $attrs= array())
+  
+  function search($srp, $filter, $attrs= array())
   {
     if($this->hascon){
       if ($this->reconnect) $this->connect();
 
-      $start = microtime();
-   
-      $this->clearResult();
-      $this->sr = @ldap_search($this->cid, LDAP::fix($this->basedn), $filter, $attrs);
+      $start = microtime(true);
+      $this->clearResult($srp);
+      $this->sr[$srp] = @ldap_search($this->cid, LDAP::fix($this->basedn), $filter, $attrs);
       $this->error = @ldap_error($this->cid);
-      $this->resetResult();
-      $this->hasres=true;
+      $this->resetResult($srp);
+      $this->hasres[$srp]=true;
    
       /* Check if query took longer as specified in max_ldap_query_time */
       if($this->max_ldap_query_time){
-        $diff = get_MicroTimeDiff($start,microtime());
+        $diff = microtime(true) - $start;
         if($diff > $this->max_ldap_query_time){
           msg_dialog::display(_("Performance warning"), sprintf(_("LDAP performance is poor: last query took about %.2fs!"), $diff), WARNING_DIALOG);
         }
       }
 
-      $this->log("LDAP operation: time=".get_MicroTimeDiff($start,microtime())." operation=search('".LDAP::fix($this->basedn)."', '$filter')");
-      return($this->sr);
+      $this->log("LDAP operation: time=".(microtime(true)-$start)." operation=search('".LDAP::fix($this->basedn)."', '$filter')");
+      return($this->sr[$srp]);
     }else{
       $this->error = "Could not connect to LDAP server";
       return("");
     }
   }
 
-  function ls($filter = "(objectclass=*)", $basedn = "",$attrs = array("*"))
+  function ls($srp, $filter = "(objectclass=*)", $basedn = "",$attrs = array("*"))
   {
     if($this->hascon){
       if ($this->reconnect) $this->connect();
-      $this->clearResult();
+
+      $this->clearResult($srp);
       if ($basedn == "")
         $basedn = $this->basedn;
       else
         $basedn= LDAP::convert($basedn);
   
-      $start = microtime();
-      $this->sr = @ldap_list($this->cid, LDAP::fix($basedn), $filter,$attrs);
+      $start = microtime(true);
+      $this->sr[$srp] = @ldap_list($this->cid, LDAP::fix($basedn), $filter,$attrs);
       $this->error = @ldap_error($this->cid);
-      $this->resetResult();
-      $this->hasres=true;
+      $this->resetResult($srp);
+      $this->hasres[$srp]=true;
 
        /* Check if query took longer as specified in max_ldap_query_time */
       if($this->max_ldap_query_time){
-        $diff = get_MicroTimeDiff($start,microtime());
+        $diff = microtime(true) - $start;
         if($diff > $this->max_ldap_query_time){
           msg_dialog::display(_("Performance warning"), sprintf(_("LDAP performance is poor: last query took about %.2fs!"), $diff), WARNING_DIALOG);
         }
       }
 
-      $this->log("LDAP operation: time=".get_MicroTimeDiff($start,microtime())." operation=ls('".LDAP::fix($basedn)."', '$filter')");
+      $this->log("LDAP operation: time=".(microtime(true) - $start)." operation=ls('".LDAP::fix($basedn)."', '$filter')");
 
-      return($this->sr);
+      return($this->sr[$srp]);
     }else{
       $this->error = "Could not connect to LDAP server";
       return("");
     }
   }
 
-  function cat($dn,$attrs= array("*"))
+  function cat($srp, $dn,$attrs= array("*"), $filter = "(objectclass=*)")
   {
     if($this->hascon){
       if ($this->reconnect) $this->connect();
-      $this->clearResult();
-      $filter = "(objectclass=*)";
-      $this->sr = @ldap_read($this->cid, LDAP::fix($dn), $filter,$attrs);
+
+      $this->clearResult($srp);
+      $this->sr[$srp] = @ldap_read($this->cid, LDAP::fix($dn), $filter,$attrs);
       $this->error = @ldap_error($this->cid);
-      $this->resetResult();
-      $this->hasres=true;
-      return($this->sr);
+      $this->resetResult($srp);
+      $this->hasres[$srp]=true;
+      return($this->sr[$srp]);
     }else{
       $this->error = "Could not connect to LDAP server";
       return("");
     }
   }
 
+  function object_match_filter($dn,$filter)
+  {
+    if($this->hascon){
+      if ($this->reconnect) $this->connect();
+      $res =  @ldap_read($this->cid, LDAP::fix($dn), $filter, array("objectClass"));
+      $rv =   @ldap_count_entries($this->cid, $res);
+      return($rv);
+    }else{
+      $this->error = "Could not connect to LDAP server";
+      return(FALSE);
+    }
+  }
+
   function set_size_limit($size)
   {
     /* Ignore zero settings */
@@ -268,26 +319,26 @@ class LDAP{
     }
   }
 
-  function fetch()
+  function fetch($srp)
   {
     $att= array();
     if($this->hascon){
-      if($this->hasres){
-        if ($this->start == 0)
+      if($this->hasres[$srp]){
+        if ($this->start[$srp] == 0)
         {
-          if ($this->sr){
-            $this->start = 1;
-            $this->re= @ldap_first_entry($this->cid, $this->sr);
+          if ($this->sr[$srp]){
+            $this->start[$srp] = 1;
+            $this->re[$srp]= @ldap_first_entry($this->cid, $this->sr[$srp]);
           } else {
             return array();
           }
         } else {
-          $this->re= @ldap_next_entry($this->cid, $this->re);
+          $this->re[$srp]= @ldap_next_entry($this->cid, $this->re[$srp]);
         }
-        if ($this->re)
+        if ($this->re[$srp])
         {
-          $att= @ldap_get_attributes($this->cid, $this->re);
-          $att['dn']= trim(LDAP::convert(@ldap_get_dn($this->cid, $this->re)));
+          $att= @ldap_get_attributes($this->cid, $this->re[$srp]);
+          $att['dn']= trim(LDAP::convert(@ldap_get_dn($this->cid, $this->re[$srp])));
         }
         $this->error = @ldap_error($this->cid);
         if (!isset($att)){
@@ -295,7 +346,7 @@ class LDAP{
         }
         return($att);
       }else{
-        $this->error = "Perform a Fetch with no Search";
+        $this->error = "Perform a fetch with no search";
         return("");
       }
     }else{
@@ -304,31 +355,31 @@ class LDAP{
     }
   }
 
-  function resetResult()
+  function resetResult($srp)
   {
-    $this->start = 0;
+    $this->start[$srp] = 0;
   }
 
-  function clearResult()
+  function clearResult($srp)
   {
-    if($this->hasres){
-      $this->hasres = false;
-      @ldap_free_result($this->sr);
+    if($this->hasres[$srp]){
+      $this->hasres[$srp] = false;
+      @ldap_free_result($this->sr[$srp]);
     }
   }
 
-  function getDN()
+  function getDN($srp)
   {
     if($this->hascon){
-      if($this->hasres){
+      if($this->hasres[$srp]){
 
-        if(!$this->re)
+        if(!$this->re[$srp])
           {
           $this->error = "Perform a Fetch with no valid Result";
           }
           else
           {
-          $rv = @ldap_get_dn($this->cid, $this->re);
+          $rv = @ldap_get_dn($this->cid, $this->re[$srp]);
         
           $this->error = @ldap_error($this->cid);
           return(trim(LDAP::convert($rv)));
@@ -343,11 +394,11 @@ class LDAP{
     }
   }
 
-  function count()
+  function count($srp)
   {
     if($this->hascon){
-      if($this->hasres){
-        $rv = @ldap_count_entries($this->cid, $this->sr);
+      if($this->hasres[$srp]){
+        $rv = @ldap_count_entries($this->cid, $this->sr[$srp]);
         $this->error = @ldap_error($this->cid);
         return($rv);
       }else{
@@ -367,7 +418,23 @@ class LDAP{
       if ($dn == "")
         $dn = $this->basedn;
 
-      $r = @ldap_mod_del($this->cid, LDAP::fix($dn), $attrs);
+      $r = ldap_mod_del($this->cid, LDAP::fix($dn), $attrs);
+      $this->error = @ldap_error($this->cid);
+      return($r);
+    }else{
+      $this->error = "Could not connect to LDAP server";
+      return("");
+    }
+  }
+
+  function mod_add($attrs = "", $dn = "")
+  {
+    if($this->hascon){
+      if ($this->reconnect) $this->connect();
+      if ($dn == "")
+        $dn = $this->basedn;
+
+      $r = @ldap_mod_add($this->cid, LDAP::fix($dn), $attrs);
       $this->error = @ldap_error($this->cid);
       return($r);
     }else{
@@ -405,6 +472,52 @@ class LDAP{
     }
   }
 
+
+  /*! \brief Move the given Ldap entry from $source to $dest
+      @param  String  $source The source dn.
+      @param  String  $dest   The destination dn.
+      @return Boolean TRUE on success else FALSE.
+   */
+  function rename_dn($source,$dest)
+  {
+    /* Check if source and destination are the same entry */
+    if(strtolower($source) == strtolower($dest)){
+      trigger_error("Source and destination can't be the same entry.");
+      $this->error = "Source and destination can't be the same entry.";
+      return(FALSE);
+    }
+
+    /* Check if destination entry exists */    
+    if($this->dn_exists($dest)){
+      trigger_error("Destination '$dest' already exists.");
+      $this->error = "Destination '$dest' already exists.";
+      return(FALSE);
+    }
+
+    /* Extract the name and the parent part out ouf source dn.
+        e.g.  cn=herbert,ou=department,dc=... 
+         parent   =>  ou=department,dc=...
+         dest_rdn =>  cn=herbert
+     */
+    $parent   = preg_replace("/^[^,]+,/","", $dest);
+    $dest_rdn = preg_replace("/,.*$/","",$dest);
+
+    if($this->hascon){
+      if ($this->reconnect) $this->connect();
+      $r= ldap_rename($this->cid,@LDAP::fix($source), @LDAP::fix($dest_rdn),@LDAP::fix($parent),TRUE); 
+      $this->error = ldap_error($this->cid);
+
+      /* Check if destination dn exists, if not the 
+          server may not support this operation */
+      $r &= is_resource($this->dn_exists($dest));
+      return($r);
+    }else{
+      $this->error = "Could not connect to LDAP server";
+      return(FALSE);
+    }
+  }
+
+
   /**
   *  Function rmdir_recursive
   *
@@ -413,17 +526,16 @@ class LDAP{
   *  GiveBack:    True on sucessfull , 0 in error, and "" when we don't get a ldap conection
   *
   */
-
-  function rmdir_recursive($deletedn)
+  function rmdir_recursive($srp, $deletedn)
   {
     if($this->hascon){
       if ($this->reconnect) $this->connect();
       $delarray= array();
         
       /* Get sorted list of dn's to delete */
-      $this->ls ("(objectClass=*)",$deletedn);
-      while ($this->fetch()){
-        $deldn= $this->getDN();
+      $this->ls ($srp, "(objectClass=*)",$deletedn);
+      while ($this->fetch($srp)){
+        $deldn= $this->getDN($srp);
         $delarray[$deldn]= strlen($deldn);
       }
       arsort ($delarray);
@@ -431,7 +543,7 @@ class LDAP{
 
       /* Really Delete ALL dn's in subtree */
       foreach ($delarray as $key => $value){
-        $this->rmdir_recursive($key);
+        $this->rmdir_recursive($srp, $key);
       }
       
       /* Finally Delete own Node */
@@ -444,174 +556,26 @@ class LDAP{
     }
   }
 
-  /* Copy given attributes and sub-dns with attributes to destination dn 
-  */
-  function copy_FAI_resource_recursive($sourcedn,$destinationdn,$destinationName,$type="branch",$is_first = true,$depth=0)
-  {
-    error_reporting(E_ALL | E_STRICT);
-    
-    if($is_first){
-      echo "<h2>".sprintf(_("Creating copy of %s"),"<i>".LDAP::fix($sourcedn)."</i>")."</h2>";
-    }else{
-      if(preg_match("/^ou=/",$sourcedn)){
-        echo "<h3>"._("Processing")." <i>".LDAP::fix($destinationdn)."</i></h3>";
-      }else{
-        $tmp = split(",",$sourcedn);
-        
-        echo "&nbsp;<b>"._("Object").":</b> ";
+  function makeReadableErrors($error,$attrs)
+  { 
+    global $config;
 
-        $deststr = LDAP::fix($destinationdn);
-        if(strlen($deststr) > 96){
-          $deststr = substr($deststr,0,96)."...";
-        }
+    if($this->success()) return("");
 
-        echo $deststr."<br>";
+    $str = "";
+    if(preg_match("/^objectClass: value #([0-9]*) invalid per syntax$/", $this->get_additional_error())){
+      $oc = preg_replace("/^objectClass: value #([0-9]*) invalid per syntax$/","\\1", $this->get_additional_error());
+      if(isset($attrs['objectClass'][$oc])){
+        $str.= " - <b>objectClass: ".$attrs['objectClass'][$oc]."</b>";
       }
     }
+    if($error == "Undefined attribute type"){
+      $str = " - <b>attribute: ".preg_replace("/:.*$/","",$this->get_additional_error())."</b>";
+    } 
 
-    flush();
-    
-    if($this->hascon){
-      if ($this->reconnect) $this->connect();
-
-      /* Save base dn */
-      $basedn= $this->basedn;
-      $delarray= array();
-     
-      /* Check if destination entry already exists */
-      $this->cat($destinationdn);
-
-      if($this->count()){
-        return;
-      }else{
-        
-        $this->clearResult();
-
-        /* Get source entry */
-        $this->cd($basedn);
-        $this->cat($sourcedn);
-        $attr = $this->fetch();
-
-        /* Error while fetching object / attribute abort*/
-        if((!$attr) || (count($attr)) ==0) {
-          echo _("Error while fetching source dn - aborted!");
-          return;
-        }
-  
-        /* check if this is a department */
-        if(in_array("organizationalUnit",$attr['objectClass'])){
-          $attr['dn'] = LDAP::convert($destinationdn);
-          $this->cd($basedn);
-          $this->create_missing_trees($destinationdn);
-          $this->cd($destinationdn);
-
-          /* If is first entry, append FAIbranch to department entry */
-          if($is_first){
-            $this->cat($destinationdn);
-            $attr= $this->fetch();
-
-            /* Filter unneeded informations */
-            foreach($attr as $key => $value){
-              if(is_numeric($key)) unset($attr[$key]);
-              if(isset($attr[$key]['count'])){
-                if(is_array($attr[$key])){
-                  unset($attr[$key]['count']);
-                }
-              }
-            }
-            
-            unset($attr['count']);
-            unset($attr['dn']);
-
-            /* Add marking attribute */
-            $attr['objectClass'][] = "FAIbranch";
-            
-            /* Add this entry */
-            $this->modify($attr);
-          }
-        }else{
-
-          /* If this is no department */
-          foreach($attr as $key => $value){
-            if(in_array($key ,array("FAItemplateFile","FAIscript", "gotoLogonScript", "gosaApplicationIcon","gotoMimeIcon"))){
-              $sr= ldap_read($this->cid, LDAP::fix($sourcedn), "$key=*", array($key));
-              $ei= ldap_first_entry($this->cid, $sr);
-              if ($tmp= @ldap_get_values_len($this->cid, $ei,$key)){
-                $attr[$key] = $tmp;
-              }
-            }
-
-            if(is_numeric($key)) unset($attr[$key]);
-            if(isset($attr[$key]['count'])){
-              if(is_array($attr[$key])){
-                unset($attr[$key]['count']);
-              }
-            }
-          }
-          unset($attr['count']);
-          unset($attr['dn']);
-
-          if((!in_array("gosaApplication" , $attr['objectClass'])) && (!in_array("gotoMimeType", $attr['objectClass']))){
-            $attr['FAIdebianRelease'] = $destinationName;
-            if($type=="branch"){
-              $attr['FAIstate'] ="branch";
-            }elseif($type=="freeze"){
-              $attr['FAIstate'] ="freeze";
-            }else{
-              msg_dialog::display(_("Internal error"), sprintf(_("FAIstate '%s' is unknown!"),$type), ERROR_DIALOG);
-            }
-          }
-
-          /* Replace FAIdebianRelease with new release name */
-          if(in_array("FAIpackageList" , $attr['objectClass'])){
-            $attr['FAIdebianRelease'] = $destinationName; 
-          }
-
-          /* Add entry */
-          $this->cd($destinationdn);
-          $this->cat($destinationdn);
-          $a = $this->fetch();
-          if(!count($a)){
-            $this->add($attr);
-          }
-
-          if($this->error != "Success"){
-            /* Some error occurred */
-            print "---------------------------------------------";
-            print $this->get_error()."<br>";
-            print $sourcedn."<br>";
-            print $destinationdn."<br>";
-            print_a( $attr);
-            exit();
-          }          
-        }
-      }
-
-      echo "<script language=\"javascript\" type=\"text/javascript\">scrollDown2();</script>" ;
-
-      $this->ls ("(objectClass=*)",$sourcedn);
-      while ($this->fetch()){
-        $deldn= $this->getDN();
-        $delarray[$deldn]= strlen($deldn);
-      }
-      asort ($delarray);
-      reset ($delarray);
-
-       $depth ++;
-      foreach($delarray as $dn => $bla){
-        if($dn != $destinationdn){
-          $this->cd($basedn);
-          $item = $this->fetch($this->cat($dn));
-          if(!in_array("FAIbranch",$item['objectClass'])){
-            $this->copy_FAI_resource_recursive($dn,str_replace($sourcedn,$destinationdn,$dn),$destinationName,$type,false,$depth);
-          } 
-        }
-      }
-    }
-    if($is_first){
-      echo "<p class='seperator'>&nbsp;</p>";
-    }
+    @DEBUG(DEBUG_LDAP,__LINE__,__FUNCTION__,__FILE__,$attrs,"Erroneous data");
 
+    return($str);
   }
 
   function modify($attrs)
@@ -623,6 +587,9 @@ class LDAP{
       if ($this->reconnect) $this->connect();
       $r = @ldap_modify($this->cid, LDAP::fix($this->basedn), $attrs);
       $this->error = @ldap_error($this->cid);
+      if(!$this->success()){
+        $this->error.= $this->makeReadableErrors($this->error,$attrs);
+      }
       return($r ? $r : 0);
     }else{
       $this->error = "Could not connect to LDAP server";
@@ -636,6 +603,9 @@ class LDAP{
       if ($this->reconnect) $this->connect();
       $r = @ldap_add($this->cid, LDAP::fix($this->basedn), $attrs);
       $this->error = @ldap_error($this->cid);
+      if(!$this->success()){
+        $this->error.= $this->makeReadableErrors($this->error,$attrs);
+      }
       return($r ? $r : 0);
     }else{
       $this->error = "Could not connect to LDAP server";
@@ -643,7 +613,7 @@ class LDAP{
     }
   }
 
-  function create_missing_trees($target)
+  function create_missing_trees($srp, $target)
   {
     global $config;
 
@@ -669,7 +639,7 @@ class LDAP{
       /* Ignore referrals */
       $found= false;
       foreach($this->referrals as $ref){
-        $base= preg_replace('!^[^:]+://[^/]+/([^?]+).*$!', '\\1', $ref['URL']);
+        $base= preg_replace('!^[^:]+://[^/]+/([^?]+).*$!', '\\1', $ref['URI']);
         if ($base == $cdn){
           $found= true;
           break;
@@ -679,8 +649,8 @@ class LDAP{
         continue;
       }
 
-      $this->cat ($cdn);
-      $attrs= $this->fetch();
+      $this->cat ($srp, $cdn);
+      $attrs= $this->fetch($srp);
 
       /* Create missing entry? */
       if (count ($attrs)){
@@ -692,7 +662,8 @@ class LDAP{
 
       } else {
         $type= preg_replace('/^([^=]+)=.*$/', '\\1', $cdn);
-        $param= preg_replace('/^[^=]+=([^,]+),.*$/', '\\1', $cdn);
+        $param= LDAP::fix(preg_replace('/^[^=]+=([^,]+).*$/', '\\1', $cdn));
+        $param=preg_replace(array('/\\\\,/','/\\\\"/'),array(',','"'),$param);
 
         $na= array();
 
@@ -702,7 +673,7 @@ class LDAP{
           /* Get name of first matching objectClass */
           $ocname= "";
           foreach($classes as $class){
-            if (isset($class['MUST']) && $class['MUST'] == "$type"){
+            if (isset($class['MUST']) && in_array($type, $class['MUST'])){
 
               /* Look for first classes that is structural... */
               if (isset($class['STRUCTURAL'])){
@@ -719,8 +690,8 @@ class LDAP{
 
           /* Bail out, if we've nothing to do... */
           if ($ocname == ""){
-            msg_dialog::display(_("Internal error"), sprintf(_("Cannot automatically create subtrees with RDN '%s': no object class found"),$type), ERROR_DIALOG);
-            display_error_page();
+            msg_dialog::display(_("Internal error"), sprintf(_("Cannot automatically create subtrees with RDN '%s': no object class found!"),$type), FATAL_ERROR_DIALOG);
+            exit();
           }
 
           /* Assemble_entry */
@@ -767,16 +738,19 @@ class LDAP{
               $na["dc"]= $param;
               break;
             default:
-              msg_dialog::display(_("Internal error"), sprintf(_("Cannot automatically create subtrees with RDN '%s': not supported"),$type), ERROR_DIALOG);
-              display_error_page();
+              msg_dialog::display(_("Internal error"), sprintf(_("Cannot automatically create subtrees with RDN '%s': not supported"),$type), FATAL_ERROR_DIALOG);
+              exit();
           }
 
         }
         $this->cd($cdn);
         $this->add($na);
     
-        show_ldap_error($this->get_error(), sprintf(_("Creating subtree '%s' failed."),$cdn));
-        if (!preg_match('/success/i', $this->error)){
+        if (!$this->success()){
+
+          print_a(array($cdn,$na));
+
+          msg_dialog::display(_("LDAP error"), msgPool::ldaperror($this->get_error(), $cdn, LDAP_ADD, get_class()));
           return FALSE;
         }
       }
@@ -786,14 +760,14 @@ class LDAP{
   }
 
 
-  function recursive_remove()
+  function recursive_remove($srp)
   {
     $delarray= array();
 
     /* Get sorted list of dn's to delete */
-    $this->search ("(objectClass=*)");
-    while ($this->fetch()){
-      $deldn= $this->getDN();
+    $this->search ($srp, "(objectClass=*)");
+    while ($this->fetch($srp)){
+      $deldn= $this->getDN($srp);
       $delarray[$deldn]= strlen($deldn);
     }
     arsort ($delarray);
@@ -805,6 +779,7 @@ class LDAP{
     }
   }
 
+
   function get_attribute($dn, $name,$r_array=0)
   {
     $data= "";
@@ -820,12 +795,11 @@ class LDAP{
         }
       }
     }
-    if($r_array==0)
-    return ($data);
-    else
-    return ($info);
-  
-  
+    if($r_array==0) {
+      return ($data);
+    } else {
+      return ($info);
+    }
   }
  
 
@@ -837,6 +811,13 @@ class LDAP{
     return ($error);
   }
 
+
+  function success()
+  {
+    return (preg_match('/Success/i', $this->error));
+  }
+
+
   function get_error()
   {
     if ($this->error == 'Success'){
@@ -865,30 +846,30 @@ class LDAP{
     if (isset($referrals[$server])){
       return ($referrals[$server]);
     } else {
-      $ret['ADMIN']= LDAP::fix($this->binddn);
-      $ret['PASSWORD']= $this->bindpw;
+      $ret['ADMINDN']= LDAP::fix($this->binddn);
+      $ret['ADMINPASSWORD']= $this->bindpw;
     }
 
     return ($ret);
   }
 
 
-  function gen_ldif ($dn, $filter= "(objectClass=*)", $attributes= array('*'), $recursive= TRUE)
+  function gen_ldif ($srp, $dn, $filter= "(objectClass=*)", $attributes= array('*'), $recursive= TRUE)
   {
     $display= "";
 
     if ($recursive){
       $this->cd($dn);
-      $this->ls($filter,$dn, array('dn','objectClass'));
+      $this->ls($srp, $filter,$dn, array('dn','objectClass'));
       $deps = array();
 
       $display .= $this->gen_one_entry($dn)."\n";
 
-      while ($attrs= $this->fetch()){
+      while ($attrs= $this->fetch($srp)){
         $deps[] = $attrs['dn'];
       }
       foreach($deps as $dn){
-        $display .= $this->gen_ldif($dn, $filter,$attributes,$recursive);
+        $display .= $this->gen_ldif($srp, $dn, $filter,$attributes,$recursive);
       }
     } else {
       $display.= $this->gen_one_entry($dn);
@@ -897,15 +878,15 @@ class LDAP{
   }
 
 
-  function gen_xls ($dn, $filter= "(objectClass=*)", $attributes= array('*'), $recursive= TRUE,$r_array=0)
+  function gen_xls ($srp, $dn, $filter= "(objectClass=*)", $attributes= array('*'), $recursive= TRUE,$r_array=0)
   {
     $display= array();
 
       $this->cd($dn);
-      $this->search("$filter");
+      $this->search($srp, "$filter");
 
       $i=0;
-      while ($attrs= $this->fetch()){
+      while ($attrs= $this->fetch($srp)){
         $j=0;
 
         foreach ($attributes as $at){
@@ -985,13 +966,13 @@ class LDAP{
       if JustMofify id false the destination dn will be overwritten by the new ldif. 
     */
 
-  function import_complete_ldif($str_attr,&$error,$JustModify,$DeleteOldEntries)
+  function import_complete_ldif($srp, $str_attr,$error,$JustModify,$DeleteOldEntries)
   {
     if($this->reconnect) $this->connect();
 
-    /* First we have to splitt the string ito detect empty lines
+    /* First we have to split the string into empty lines.
        An empty line indicates an new Entry */
-    $entries = split("\n",$str_attr);
+    $entries = preg_split("/\n/",$str_attr);
 
     $data = "";
     $cnt = 0; 
@@ -1032,9 +1013,9 @@ class LDAP{
         /* Append lines ... */
         if(!empty($tmp2)) {
           /* check if we need base64_decode for this line */
-          if(ereg("::",$tmp2))
+          if(strstr($tmp2, "::") !== false)
           {
-            $encoded = split("::",$entry);
+            $encoded = explode("::",$entry);
             $attr  = trim($encoded[0]);
             $value = base64_decode(trim($encoded[1]));
             /* Add linenumber */
@@ -1057,11 +1038,11 @@ class LDAP{
        
        Now we check every insertblock and try to insert */
     foreach ( $all as $single) {
-      $lineone = split("\n",$single);  
-      $ndn = split("#", $lineone[0]);
+      $lineone = preg_split("/\n/",$single);  
+      $ndn = explode("#", $lineone[0]);
       $line = base64_decode($ndn[1]);
 
-      $dnn = split (":",$line,2);
+      $dnn = explode (":",$line,2);
       $current_line = $ndn[0];
       $dn    = $dnn[0];
       $value = $dnn[1];
@@ -1096,7 +1077,7 @@ class LDAP{
       }
      
       /* If we can't Import, return with a file error */
-      if(!$this->import_single_entry($single,$usemodify,$usermdir) ) {
+      if(!$this->import_single_entry($srp, $single,$usemodify,$usermdir) ) {
         $error= sprintf(_("Error while importing dn: '%s', please check your LDIF from line %s on!"), $line,
                         $current_line);
         return UNKNOWN_TOKEN_IN_LDIF_FILE;      }
@@ -1111,7 +1092,7 @@ class LDAP{
       if $modify is true;  All variables that are not touched by the new ldif will be kept.
       if $modify is false; The new ldif overwrites the old entry, and all untouched attributes get lost.
   */
-  function import_single_entry($str_attr,$modify,$delete)
+  function import_single_entry($srp, $str_attr,$modify,$delete)
   {
     global $config;
 
@@ -1123,7 +1104,7 @@ class LDAP{
     if($this->reconnect) $this->connect();
 
     $ret = false;
-    $rows= split("\n",$str_attr);
+    $rows= preg_split("/\n/",$str_attr);
     $data= false;
 
     foreach($rows as $row) {
@@ -1135,13 +1116,13 @@ class LDAP{
 
           /* We are using line numbers 
              Because there is a # before a : */
-          $tmp1= split("#",$row);
+          $tmp1= explode("#",$row);
           $current_line= $tmp1[0];
           $row= base64_decode($tmp1[1]);
         }
 
         /* Split the line into  attribute  and value */
-        $attr   = split(":", $row,2);
+        $attr   = explode(":", $row,2);
         $attr[0]= trim($attr[0]);  /* attribute */
         $attr[1]= $attr[1];  /* value */
 
@@ -1192,13 +1173,13 @@ class LDAP{
 
       /* Delete existing entry */
       if($delete){
-        $this->rmdir_recursive($data['dn']);
+        $this->rmdir_recursive($srp, $data['dn']);
       }
      
       /* Create missing trees */
       $this->cd ($this->basedn);
       $this->cd($config->current['BASE']);
-      $this->create_missing_trees(preg_replace("/^[^,]+,/","",$data['dn']));
+      $this->create_missing_trees($srp, preg_replace("/^[^,]+,/","",$data['dn']));
       $this->cd($data['dn']);
 
       $dn = $data['dn'];
@@ -1206,11 +1187,11 @@ class LDAP{
       
       if(!$modify){
 
-        $this->cat($dn);
-        if($this->count()){
+        $this->cat($srp, $dn);
+        if($this->count($srp)){
         
           /* The destination entry exists, overwrite it with the new entry */
-          $attrs = $this->fetch();
+          $attrs = $this->fetch($srp);
           foreach($attrs as $name => $value ){
             if(!is_numeric($name)){
               if(in_array($name,array("dn","count"))) continue;
@@ -1233,14 +1214,18 @@ class LDAP{
         $ret = $this->modify($data);
       }
     }
-    show_ldap_error($this->get_error(), sprintf(_("Ldap import with dn '%s' failed."),$dn));
+
+    if (!$this->success()){
+      msg_dialog::display(_("LDAP error"), msgPool::ldaperror($this->get_error(), $dn, "", get_class()));
+    }
+
     return($ret);
   }
 
   
   function importcsv($str)
   {
-    $lines = split("\n",$str);
+    $lines = preg_split("/\n/",$str);
     foreach($lines as $line)
     {
       /* continue if theres a comment */
@@ -1252,32 +1237,34 @@ class LDAP{
       $line= str_replace ("\t"  ,"," ,$line);
       echo $line;
 
-      $cells = split(",",$line )  ;
+      $cells = explode(",",$line )  ;
       $linet= str_replace ("\t\t",",",$line);
-      $cells = split("\t",$line);
+      $cells = preg_split("/\t/",$line);
       $count = count($cells);  
     }
 
   }
   
-  function get_objectclasses()
+  function get_objectclasses( $force_reload = FALSE)
   {
     $objectclasses = array();
     global $config;
 
     /* Only read schema if it is allowed */
     if(isset($config) && preg_match("/config/i",get_class($config))){
-      if(!isset($config->data['MAIN']['SCHEMA_CHECK']) || !preg_match("/true/i",$config->data['MAIN']['SCHEMA_CHECK'])){
+      if ($config->get_cfg_value("schemaCheck") != "true"){
         return($objectclasses);
       } 
     }
+
+    /* Return the cached results. */
+    if(class_available('session') && session::global_is_set("LDAP_CACHE::get_objectclasses") && !$force_reload){
+      $objectclasses = session::global_get("LDAP_CACHE::get_objectclasses");
+      return($objectclasses);
+    }
        
          # Get base to look for schema 
-         $sr = @ldap_read ($this->cid, NULL, "objectClass=*", array("subschemaSubentry"));
-    if(!$sr){
-           $sr = @ldap_read ($this->cid, "", "objectClass=*", array("subschemaSubentry"));
-    }
-
+         $sr = @ldap_read ($this->cid, "", "objectClass=*", array("subschemaSubentry"));
          $attr = @ldap_get_entries($this->cid,$sr);
          if (!isset($attr[0]['subschemasubentry'][0])){
            return array();
@@ -1296,7 +1283,7 @@ class LDAP{
         continue;
       }
       $name= "OID";
-      $pattern= split(' ', $val);
+      $pattern= explode(' ', $val);
       $ocname= preg_replace("/^.* NAME\s+\(*\s*'([^']+)'\s*\)*.*$/", '\\1', $val);
       $objectclasses[$ocname]= array();
 
@@ -1308,7 +1295,11 @@ class LDAP{
                     break;
 
           case ')': if ($name != ""){
-                      $objectclasses[$ocname][$name]= $this->value2container($value);
+                      $v = $this->value2container($value);
+                      if(in_array($name, array('MUST', 'MAY')) && !is_array($v)){
+                        $v = array($v);
+                      }
+                      $objectclasses[$ocname][$name]= $v;
                     }
                     $name= "";
                     $value= "";
@@ -1323,7 +1314,11 @@ class LDAP{
           case 'MUST':
           case 'MAY':
                     if ($name != ""){
-                      $objectclasses[$ocname][$name]= $this->value2container($value);
+                      $v = $this->value2container($value);
+                      if(in_array($name, array('MUST', 'MAY')) && !is_array($v)){
+                        $v = array($v);
+                      }
+                      $objectclasses[$ocname][$name]= $v;
                     }
                     $name= $chunk;
                     $value= "";
@@ -1334,6 +1329,10 @@ class LDAP{
       }
 
          }
+    if(class_available("session")){
+      session::global_set("LDAP_CACHE::get_objectclasses",$objectclasses);
+    }
+
          return $objectclasses;
   }
 
@@ -1362,8 +1361,8 @@ class LDAP{
 
   function log($string)
   {
-    if (session::is_set('config')){
-      $cfg = session::get('config');
+    if (session::global_is_set('config')){
+      $cfg = session::global_get('config');
       if (isset($cfg->current['LDAPSTATS']) && preg_match('/true/i', $cfg->current['LDAPSTATS'])){
         syslog (LOG_INFO, $string);
       }
@@ -1372,10 +1371,10 @@ class LDAP{
 
   /* added by Guido Serra aka Zeph <zeph@purotesto.it> */
   function getCn($dn){
-    $simple= split(",", $dn);
+    $simple= explode(",", $dn);
 
     foreach($simple as $piece) {
-      $partial= split("=", $piece);
+      $partial= explode("=", $piece);
 
       if($partial[0] == "cn"){
         return $partial[1];
@@ -1431,8 +1430,6 @@ class LDAP{
     return ($result);
   }
 
-
 }
-
 // vim:tabstop=2:expandtab:shiftwidth=2:filetype=php:syntax:ruler:
 ?>