In LdapCommon checkPassword/changePassword only get the 'dn' attribute as an optimization as no other attributes are necessary.

Thanks to drslump reported at http://status.net/open-source/issues/2955
This commit is contained in:
Craig Andrews 2010-12-17 17:22:52 -05:00
parent 0e7a283883
commit 786250e3d9

View File

@ -140,7 +140,7 @@ class LdapCommon
function checkPassword($username, $password) function checkPassword($username, $password)
{ {
$entry = $this->get_user($username); $entry = $this->get_user($username,array('dn' => 'dn'));
if(!$entry){ if(!$entry){
return false; return false;
}else{ }else{
@ -168,7 +168,7 @@ class LdapCommon
//throw new Exception(_('Sorry, changing LDAP passwords is not supported at this time')); //throw new Exception(_('Sorry, changing LDAP passwords is not supported at this time'));
return false; return false;
} }
$entry = $this->get_user($username); $entry = $this->get_user($username,array('dn' => 'dn'));
if(!$entry){ if(!$entry){
return false; return false;
}else{ }else{