AUTH_ldap lookupUers needs to close the ldap connection

Bug #1032840 reported by Bill Gosse
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PHPDevShell
Confirmed
Low
Greg

Bug Description

AUTH_ldap class's lookupUers function should close the lpad connection once its done authenticating/binding the user.

        if (!@ldap_bind($connect, $username, $password)) {
            ldap_close($connect);
            return false; // if we can't bind it's likely the user is unknown or the password is wrong
        }
        $this->log('Bound');

        ldap_close($connect);
        return true;

Greg (gregfr)
Changed in phpdevshell:
status: New → Confirmed
assignee: nobody → Greg (gregfr)
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.