diff options
author | Aleksander Machniak <alec@alec.pl> | 2014-04-30 09:07:29 +0200 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2014-04-30 09:08:28 +0200 |
commit | 34e6526c958f3a0143a2cf5c8f1b9be29ebd65ef (patch) | |
tree | 638fbeaa29b28a59d310ff8e3710ecd35a8aa846 /program/include | |
parent | 50fee948fd6dda6774c3e95469023d15add0d9ea (diff) |
Fix invalid host validation on login (#1489841)
Diffstat (limited to 'program/include')
-rw-r--r-- | program/include/rcmail.php | 32 |
1 files changed, 10 insertions, 22 deletions
diff --git a/program/include/rcmail.php b/program/include/rcmail.php index bf2291310..f0afc6b63 100644 --- a/program/include/rcmail.php +++ b/program/include/rcmail.php @@ -497,30 +497,18 @@ class rcmail extends rcube $username_domain = $this->config->get('username_domain'); $login_lc = $this->config->get('login_lc', 2); - if (!$host) { - $host = $default_host; - } - - // Validate that selected host is in the list of configured hosts - if (is_array($default_host)) { - $allowed = false; - - foreach ($default_host as $key => $host_allowed) { - if (!is_numeric($key)) { - $host_allowed = $key; - } - if ($host == $host_allowed) { - $allowed = true; - break; - } + // host is validated in rcmail::autoselect_host(), so here + // we'll only handle unset host (if possible) + if (!$host && !empty($default_host)) { + if (is_array($default_host)) { + list($key, $val) = each($default_host); + $host = is_numeric($key) ? $val : $key; } - - if (!$allowed) { - $host = null; + else { + $host = $default_host; } - } - else if (!empty($default_host) && $host != rcube_utils::parse_host($default_host)) { - $host = null; + + $host = rcube_utils::parse_host($host); } if (!$host) { |