X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=functions%2Fauth.php;h=5304458e6120d0f36bbf3856a235cae950ce6ba7;hb=6b76cffa6afb885fc54c385527ea18e1e7d46dcb;hp=11a484fed5b6e99faf95734dcb5b3a161cd26323;hpb=de219480fb4613b3d9f31c15fb7af266f456e1e7;p=squirrelmail.git diff --git a/functions/auth.php b/functions/auth.php index 11a484fe..5304458e 100644 --- a/functions/auth.php +++ b/functions/auth.php @@ -9,7 +9,7 @@ * functions/global.php * functions/strings.php. * - * @copyright © 1999-2007 The SquirrelMail Project Team + * @copyright 1999-2010 The SquirrelMail Project Team * @license http://opensource.org/licenses/gpl-license.php GNU Public License * @version $Id$ * @package squirrelmail @@ -24,29 +24,65 @@ * and PAGE_NAME in session and returns false. POST information is saved in * 'session_expired_post' variable, PAGE_NAME is saved in 'session_expired_location'. * + * This function optionally checks the referrer of this page request. If the + * administrator wants to impose a check that the referrer of this page request + * is another page on the same domain (otherwise, the page request is likely + * the result of a XSS or phishing attack), then they need to specify the + * acceptable referrer domain in a variable named $check_referrer in + * config/config.php (or the configuration tool) for which the value is + * usually the same as the $domain setting (for example: + * $check_referrer = 'example.com'; + * However, in some cases (where proxy servers are in use, etc.), the + * acceptable referrer might be different. If $check_referrer is set to + * "###DOMAIN###", then the current value of $domain is used (useful in + * situations where $domain might change at runtime (when using the Login + * Manager plugin to host multiple domains with one SquirrelMail installation, + * for example)): + * $check_referrer = '###DOMAIN###'; + * NOTE HOWEVER, that referrer checks are not foolproof - they can be spoofed + * by browsers, and some browsers intentionally don't send them, in which + * case SquirrelMail silently ignores referrer checks. + * * Script that uses this function instead of is_logged_in() function, must handle user * level messages. * @return boolean * @since 1.5.1 */ function sqauth_is_logged_in() { - if ( sqsession_is_registered('user_is_logged_in') ) { + + global $check_referrer, $domain; + if (!sqgetGlobalVar('HTTP_REFERER', $referrer, SQ_SERVER)) $referrer = ''; + if ($check_referrer == '###DOMAIN###') $check_referrer = $domain; + if (!empty($check_referrer)) { + $ssl_check_referrer = 'https://' . $check_referrer; + $plain_check_referrer = 'http://' . $check_referrer; + } + if (sqsession_is_registered('user_is_logged_in') + && (!$check_referrer || empty($referrer) + || ($check_referrer && !empty($referrer) + && (strpos(strtolower($referrer), strtolower($plain_check_referrer)) === 0 + || strpos(strtolower($referrer), strtolower($ssl_check_referrer)) === 0)))) { return true; } - // First we store some information in the new session to prevent - // information-loss. - $session_expired_post = $_POST; - $session_expired_location = PAGE_NAME; - if (!sqsession_is_registered('session_expired_post')) { - sqsession_register($session_expired_post,'session_expired_post'); - } - if (!sqsession_is_registered('session_expired_location')) { - sqsession_register($session_expired_location,'session_expired_location'); - } - session_write_close(); + // First we store some information in the new session to prevent + // information-loss. + $session_expired_post = $_POST; + if (defined('PAGE_NAME')) + $session_expired_location = PAGE_NAME; + else + $session_expired_location = ''; + + if (!sqsession_is_registered('session_expired_post')) { + sqsession_register($session_expired_post,'session_expired_post'); + } + if (!sqsession_is_registered('session_expired_location')) { + sqsession_register($session_expired_location,'session_expired_location'); + } + + session_write_close(); - return false; + return false; } /** @@ -66,19 +102,26 @@ function sqauth_read_password() { /** * Saves or updates user password information * - * This function is used to update password information that SquirrelMail - * stores during existing web session. It does not modify password stored - * in authentication system used by IMAP server. + * This function is used to update the password information that + * SquirrelMail stores in the existing PHP session. It does NOT + * modify the password stored in the authentication system used + * by the IMAP server. + * + * This function must be called before any html output is started. + * Direct access to password information is deprecated. The saved + * password information is available only to the SquirrelMail script + * that is called/executed AFTER the current one. If your script + * needs access to the saved password after a sqauth_save_password() + * call, use the returned OTP encrypted key. * - * Function must be called before any html output started. Direct access - * to password information is deprecated. Saved password information is - * available only to next executed SquirrelMail script. If your script needs - * access to saved password after sqauth_save_password() call, use returned - * OTP encrypted key. * @param string $pass password - * @return string password encrypted with OTP. In case script wants to access - * password information before reloading page. + * + * @return string Password encrypted with OTP. In case the script + * wants to access the password information before + * the end of its execution. + * * @since 1.5.1 + * */ function sqauth_save_password($pass) { sqgetGlobalVar('base_uri', $base_uri, SQ_SESSION); @@ -126,6 +169,7 @@ function cram_md5_response ($username,$password,$challenge) { */ function digest_md5_response ($username,$password,$challenge,$service,$host,$authz='') { $result=digest_md5_parse_challenge($challenge); + //FIXME we should check that $result contains the expected values that we use below // verify server supports qop=auth // $qop = explode(",",$result['qop']); @@ -187,7 +231,8 @@ function digest_md5_response ($username,$password,$challenge,$service,$host,$aut */ function digest_md5_parse_challenge($challenge) { $challenge=base64_decode($challenge); - while (isset($challenge)) { + $parsed = array(); + while (!empty($challenge)) { if ($challenge{0} == ',') { // First char is a comma, must not be 1st time through loop $challenge=substr($challenge,1); } @@ -260,7 +305,7 @@ function hmac_md5($data, $key='') { * * @param string $user Reference to SMTP username * @param string $pass Reference to SMTP password (unencrypted) - * @since 1.5.0 + * @since 1.4.11 */ function get_smtp_user(&$user, &$pass) { global $username, $smtp_auth_mech, @@ -277,4 +322,11 @@ function get_smtp_user(&$user, &$pass) { $user = $username; $pass = sqauth_read_password(); } + + // plugin authors note: override $user or $pass by + // directly changing the arguments array contents + // in your plugin e.g., $args[0] = 'new_username'; + // + $temp = array(&$user, &$pass); + do_hook('smtp_auth', $temp); }