Author: the.real.singesang

Description:
$openid = this->getUserUrl($user); assigns an array. strlen of an empty array also returns something greater 0. so it should check, if the array is empty then

possible solution:

Index: SpecialOpenIDServer.body.php

• SpecialOpenIDServer.body.php (revision 52244)

+++ SpecialOpenIDServer.body.php (working copy)
@@ -235,7 +235,7 @@

$openid = $this->getUserUrl( $user );

• if ( isset( $openid ) && strlen( $openid ) > 0 ) {

+ if ( isset( $openid ) && strlen( $openid[0] ) > 0 ) {

        wfDebug( "OpenID: Not one of our users; logs in with OpenID.\n" );
        return $request->answer( false, $this->serverUrl() );
}

Version: unspecified
Severity: minor