Bug#650258: dhcpHost not removed if system is removed

[Mike Gabriel]

Package: gosa
Severity: important
Tags: patch

User:     debian-...@lists.debian.org
UserTags: debian-edu




https://forge.fusiondirectory.org/issues/553



The patch is also attached to this issue report.

Greets,
Mike


--

DAS-NETZWERKTEAM
mike gabriel, dorfstr. 27, 24245 barmissen
fon: +49 (4302) 281418, fax: +49 (4302) 281419

GnuPG Key ID 0xB588399B
mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de

freeBusy:
https://mail.das-netzwerkteam.de/freebusy/m.gabriel%40das-netzwerkteam.de.xfb

--- class_termDNS.inc	2011-10-03 21:38:13.000000000 +0200
+++ class_termDNS.inc.fixed	2011-11-28 10:43:21.000000000 +0100
@@ -467,9 +467,15 @@
 
   function remove_from_parent()
   {
+
+    $ldap = $this->config->get_ldap_link();
+
+    /************
+     * DNS Handling
+     ************/
+
     if($this->DNS_is_account){
 
-      $ldap = $this->config->get_ldap_link();
 
       $tmp = array();
       $this->dnsEntry['exists'] = false;
@@ -485,6 +491,19 @@
         }
       }
     }
+
+
+    /************
+     * DHCP Handling 
+     ************/
+
+    /* DHCP removed */
+    if($this->dhcp_is_Account){
+      $ldap->rmdir_recursive($this->dhcpHostEntry['dn']);
+      if (!$ldap->success()){
+        msg_dialog::display(_("LDAP error"), msgPool::ldaperror($ldap->get_error(), $this->dhcpHostEntry['dn'], LDAP_DEL, get_class()));
+      }
+    }
   }
 
 
@@ -782,7 +801,7 @@
           ($this->dhcpHostEntry['dhcpStatements'] == "" || count($this->dhcpHostEntry['dhcpStatements']) == 0) ){
         unset($this->dhcpHostEntry['dhcpStatements']);
       }
-  
+
       /* DHCP removed */
       if($this->initial_dhcp_is_Account && !$this->dhcp_is_Account){
         $ldap->rmdir_recursive($this->dhcpHostEntry['dn']);

If you remove a system from GOsa that had DHCP enabled then the dhcpHost entry is not removed from the dhcpService configuration. A more accurate description on how to produce this issue (and a patch) is given here: It would be great to have this patch applied as an update for GOsa 2.6 in Debian squeeze as it is relevant to Debian Edu squeeze. Thanks!

pgpJpOc3xHu86.pgp
Description: Digitale PGP-Unterschrift