PluggableAuth crashes if the entered password was wrong
Bug #1020928 reported by
Adrian Liechti
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
PHPDevShell |
Fix Committed
|
Low
|
Greg |
Bug Description
Fix ("/PluggableAut
/**
* @author Adrian Liechti
* If the Passwort does not have a length of 32 chars, it must be a Source.
*
* If the Password does have a length of 32 chars, the login has failed
* because it was checked before, it has to be a wrong Password.
*
* @important This means that a Source must not have a name with a lengt of 32 chars.
*/
if (strlen($source) !== 32) {
$plugin = $this->
if ($plugin-
return $user;
}
}
Changed in phpdevshell: | |
milestone: | none → 3.2.1-stable |
Changed in phpdevshell: | |
milestone: | 3.2.1-stable → 3.5 |
To post a comment you must log in.
In fact the problem was a typo: >lookupUser( array(' username' => $username, 'password' => $password)))) { >lookupUser( array(' username' => $username, 'password' => $password)))) {
if (is_a($auth, 'iAUTH_plugin') && ($plugin-
instead of
if (is_a($plugin, 'iAUTH_plugin') && ($plugin-
Fixed at rev 910.