[Tickets #13247] Undefined index: writedn on line 529 of Horde/Group/Ldap.php

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/13247
------------------------------------------------------------------------------
  Ticket             | 13247
  Created By         | skhorde at smail.inf.fh-bonn-rhein-sieg.de
  Summary            | Undefined index: writedn on line 529 of
                     | Horde/Group/Ldap.php
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


skhorde at smail.inf.fh-bonn-rhein-sieg.de (2014-06-05 09:47) wrote:

If you setup Horde to use LDAP groups, but does not allow write  
access, then you try to remove user data via horde-remove-user-data,  
you get:

  HORDE: [horde] PHP ERROR: Undefined index: writedn [pid 31261 on  
line 529 of "/usr/share/php/Horde/Group/Ldap.php"]
  HORDE: [horde] PHP ERROR: Undefined index: writepw [pid 31261 on  
line 529 of "/usr/share/php/Horde/Group/Ldap.php"]
  HORDE: [horde] Could not delete attribute memberUid: Strong(er)  
authentication required [pid 31261 on line 456 of  
"/usr/share/php/Horde/Group/Ldap.php"]

Attached patch fixes the errors in the log by protecting removeUser() by
if ($this->readOnly()) {
    throw new Horde_Group_Exception('This group backend is read-only.');
}

as done by the other write functions in this PHP file.




skhorde at smail.inf.fh-bonn-rhein-sieg.de (2014-06-05 09:47) uploaded:  
h5_removeUser-no-ro-check.diff.bz2

http://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=h5_removeUser-no-ro-check.diff.bz2&ticket=13247&fn=%2Fh5_removeUser-no-ro-check.diff.bz2