Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 1 | <?php if ( ! defined('BASEPATH')) exit('No direct script access allowed'); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 2 | /** |
| 3 | * CodeIgniter |
| 4 | * |
Greg Aker | 741de1c | 2010-11-10 14:52:57 -0600 | [diff] [blame] | 5 | * An open source application development framework for PHP 5.1.6 or newer |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 6 | * |
Derek Jones | f4a4bd8 | 2011-10-20 12:18:42 -0500 | [diff] [blame] | 7 | * NOTICE OF LICENSE |
| 8 | * |
| 9 | * Licensed under the Open Software License version 3.0 |
| 10 | * |
| 11 | * This source file is subject to the Open Software License (OSL 3.0) that is |
| 12 | * bundled with this package in the files license.txt / license.rst. It is |
| 13 | * also available through the world wide web at this URL: |
| 14 | * http://opensource.org/licenses/OSL-3.0 |
| 15 | * If you did not receive a copy of the license and are unable to obtain it |
| 16 | * through the world wide web, please send an email to |
| 17 | * licensing@ellislab.com so we can send you a copy immediately. |
| 18 | * |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 19 | * @package CodeIgniter |
Derek Jones | f4a4bd8 | 2011-10-20 12:18:42 -0500 | [diff] [blame] | 20 | * @author EllisLab Dev Team |
| 21 | * @copyright Copyright (c) 2008 - 2011, EllisLab, Inc. (http://ellislab.com/) |
| 22 | * @license http://opensource.org/licenses/OSL-3.0 Open Software License (OSL 3.0) |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 23 | * @link http://codeigniter.com |
| 24 | * @since Version 1.0 |
| 25 | * @filesource |
| 26 | */ |
| 27 | |
| 28 | // ------------------------------------------------------------------------ |
| 29 | |
| 30 | /** |
| 31 | * Security Class |
| 32 | * |
| 33 | * @package CodeIgniter |
| 34 | * @subpackage Libraries |
| 35 | * @category Security |
Derek Jones | f4a4bd8 | 2011-10-20 12:18:42 -0500 | [diff] [blame] | 36 | * @author EllisLab Dev Team |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 37 | * @link http://codeigniter.com/user_guide/libraries/security.html |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 38 | */ |
| 39 | class CI_Security { |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 40 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 41 | /** |
| 42 | * Random Hash for protecting URLs |
| 43 | * |
| 44 | * @var string |
| 45 | * @access protected |
| 46 | */ |
| 47 | protected $_xss_hash = ''; |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 48 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 49 | /** |
| 50 | * Random Hash for Cross Site Request Forgery Protection Cookie |
| 51 | * |
| 52 | * @var string |
| 53 | * @access protected |
| 54 | */ |
| 55 | protected $_csrf_hash = ''; |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 56 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 57 | /** |
| 58 | * Expiration time for Cross Site Request Forgery Protection Cookie |
| 59 | * Defaults to two hours (in seconds) |
| 60 | * |
| 61 | * @var int |
| 62 | * @access protected |
| 63 | */ |
| 64 | protected $_csrf_expire = 7200; |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 65 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 66 | /** |
| 67 | * Token name for Cross Site Request Forgery Protection Cookie |
| 68 | * |
| 69 | * @var string |
| 70 | * @access protected |
| 71 | */ |
| 72 | protected $_csrf_token_name = 'ci_csrf_token'; |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 73 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 74 | /** |
| 75 | * Cookie name for Cross Site Request Forgery Protection Cookie |
| 76 | * |
| 77 | * @var string |
| 78 | * @access protected |
| 79 | */ |
| 80 | protected $_csrf_cookie_name = 'ci_csrf_token'; |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 81 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 82 | /** |
| 83 | * List of never allowed strings |
| 84 | * |
| 85 | * @var array |
| 86 | * @access protected |
| 87 | */ |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 88 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 89 | protected $_never_allowed_str = array( |
| 90 | 'document.cookie' => '[removed]', |
| 91 | 'document.write' => '[removed]', |
| 92 | '.parentNode' => '[removed]', |
| 93 | '.innerHTML' => '[removed]', |
| 94 | 'window.location' => '[removed]', |
| 95 | '-moz-binding' => '[removed]', |
| 96 | '<!--' => '<!--', |
| 97 | '-->' => '-->', |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 98 | '<![CDATA[' => '<![CDATA[', |
| 99 | '<comment>' => '<comment>' |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 100 | ); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 101 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 102 | /** |
| 103 | * List of never allowed regex replacement |
| 104 | * |
| 105 | * @var array |
| 106 | * @access protected |
| 107 | */ |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 108 | protected $_never_allowed_regex = array( |
| 109 | "javascript\s*:" => '[removed]', |
| 110 | "expression\s*(\(|&\#40;)" => '[removed]', // CSS and IE |
| 111 | "vbscript\s*:" => '[removed]', // IE, surprise! |
| 112 | "Redirect\s+302" => '[removed]' |
| 113 | ); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 114 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 115 | /** |
| 116 | * Constructor |
| 117 | */ |
Greg Aker | a926328 | 2010-11-10 15:26:43 -0600 | [diff] [blame] | 118 | public function __construct() |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 119 | { |
patwork | ef1a55a | 2011-04-09 13:04:06 +0200 | [diff] [blame] | 120 | // CSRF config |
| 121 | foreach(array('csrf_expire', 'csrf_token_name', 'csrf_cookie_name') as $key) |
| 122 | { |
| 123 | if (FALSE !== ($val = config_item($key))) |
| 124 | { |
| 125 | $this->{'_'.$key} = $val; |
| 126 | } |
| 127 | } |
| 128 | |
patwork | 9e26798 | 2011-04-11 13:02:32 +0200 | [diff] [blame] | 129 | // Append application specific cookie prefix |
Greg Aker | b3e614d | 2011-04-19 20:19:17 -0500 | [diff] [blame] | 130 | if (config_item('cookie_prefix')) |
| 131 | { |
patwork | 9e26798 | 2011-04-11 13:02:32 +0200 | [diff] [blame] | 132 | $this->_csrf_cookie_name = config_item('cookie_prefix').$this->_csrf_cookie_name; |
| 133 | } |
Derek Jones | b3f10a2 | 2010-07-25 19:11:26 -0500 | [diff] [blame] | 134 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 135 | // Set the CSRF hash |
| 136 | $this->_csrf_set_hash(); |
Derek Allard | 958543a | 2010-07-22 14:10:26 -0400 | [diff] [blame] | 137 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 138 | log_message('debug', "Security Class Initialized"); |
| 139 | } |
| 140 | |
| 141 | // -------------------------------------------------------------------- |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 142 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 143 | /** |
| 144 | * Verify Cross Site Request Forgery Protection |
| 145 | * |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 146 | * @return object |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 147 | */ |
Eric Barnes | 9805ecc | 2011-01-16 23:35:16 -0500 | [diff] [blame] | 148 | public function csrf_verify() |
Derek Allard | 958543a | 2010-07-22 14:10:26 -0400 | [diff] [blame] | 149 | { |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 150 | // If no POST data exists we will set the CSRF cookie |
| 151 | if (count($_POST) == 0) |
| 152 | { |
| 153 | return $this->csrf_set_cookie(); |
| 154 | } |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 155 | |
Alex Bilbie | aeb2c3e | 2011-08-21 16:14:54 +0100 | [diff] [blame] | 156 | // Check if URI has been whitelisted from CSRF checks |
| 157 | if ($exclude_uris = config_item('csrf_exclude_uris')) |
| 158 | { |
| 159 | $uri = load_class('URI', 'core'); |
| 160 | if (in_array($uri->uri_string(), $exclude_uris)) |
| 161 | { |
| 162 | return $this; |
| 163 | } |
| 164 | } |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 165 | |
| 166 | // Do the tokens exist in both the _POST and _COOKIE arrays? |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 167 | if ( ! isset($_POST[$this->_csrf_token_name]) OR |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 168 | ! isset($_COOKIE[$this->_csrf_cookie_name])) |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 169 | { |
| 170 | $this->csrf_show_error(); |
| 171 | } |
| 172 | |
| 173 | // Do the tokens match? |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 174 | if ($_POST[$this->_csrf_token_name] != $_COOKIE[$this->_csrf_cookie_name]) |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 175 | { |
| 176 | $this->csrf_show_error(); |
| 177 | } |
| 178 | |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 179 | // We kill this since we're done and we don't want to |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 180 | // polute the _POST array |
| 181 | unset($_POST[$this->_csrf_token_name]); |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 182 | |
RS71 | 2be25a6 | 2011-12-31 16:02:04 -0200 | [diff] [blame^] | 183 | // Regenerate on every submission? |
| 184 | if (config_item('csrf_regenerate')) |
| 185 | { |
| 186 | // Nothing should last forever |
| 187 | unset($_COOKIE[$this->_csrf_cookie_name]); |
| 188 | $this->_csrf_hash = ''; |
| 189 | } |
| 190 | |
Derek Jones | b3f10a2 | 2010-07-25 19:11:26 -0500 | [diff] [blame] | 191 | $this->_csrf_set_hash(); |
| 192 | $this->csrf_set_cookie(); |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 193 | |
Alex Bilbie | aeb2c3e | 2011-08-21 16:14:54 +0100 | [diff] [blame] | 194 | log_message('debug', "CSRF token verified"); |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 195 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 196 | return $this; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 197 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 198 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 199 | // -------------------------------------------------------------------- |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 200 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 201 | /** |
| 202 | * Set Cross Site Request Forgery Protection Cookie |
| 203 | * |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 204 | * @return object |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 205 | */ |
Eric Barnes | 9805ecc | 2011-01-16 23:35:16 -0500 | [diff] [blame] | 206 | public function csrf_set_cookie() |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 207 | { |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 208 | $expire = time() + $this->_csrf_expire; |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 209 | $secure_cookie = (bool) config_item('cookie_secure'); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 210 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 211 | if ($secure_cookie) |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 212 | { |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 213 | $req = isset($_SERVER['HTTPS']) ? $_SERVER['HTTPS'] : FALSE; |
| 214 | |
| 215 | if ( ! $req OR $req == 'off') |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 216 | { |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 217 | return FALSE; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 218 | } |
| 219 | } |
Derek Allard | 958543a | 2010-07-22 14:10:26 -0400 | [diff] [blame] | 220 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 221 | setcookie($this->_csrf_cookie_name, $this->_csrf_hash, $expire, config_item('cookie_path'), config_item('cookie_domain'), $secure_cookie); |
| 222 | |
| 223 | log_message('debug', "CRSF cookie Set"); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 224 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 225 | return $this; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 226 | } |
| 227 | |
| 228 | // -------------------------------------------------------------------- |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 229 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 230 | /** |
| 231 | * Show CSRF Error |
| 232 | * |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 233 | * @return void |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 234 | */ |
Eric Barnes | 9805ecc | 2011-01-16 23:35:16 -0500 | [diff] [blame] | 235 | public function csrf_show_error() |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 236 | { |
| 237 | show_error('The action you have requested is not allowed.'); |
| 238 | } |
| 239 | |
| 240 | // -------------------------------------------------------------------- |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 241 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 242 | /** |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 243 | * Get CSRF Hash |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 244 | * |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 245 | * Getter Method |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 246 | * |
| 247 | * @return string self::_csrf_hash |
| 248 | */ |
| 249 | public function get_csrf_hash() |
| 250 | { |
| 251 | return $this->_csrf_hash; |
| 252 | } |
| 253 | |
| 254 | // -------------------------------------------------------------------- |
| 255 | |
| 256 | /** |
| 257 | * Get CSRF Token Name |
| 258 | * |
| 259 | * Getter Method |
| 260 | * |
| 261 | * @return string self::csrf_token_name |
| 262 | */ |
| 263 | public function get_csrf_token_name() |
| 264 | { |
| 265 | return $this->_csrf_token_name; |
| 266 | } |
| 267 | |
| 268 | // -------------------------------------------------------------------- |
| 269 | |
| 270 | /** |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 271 | * XSS Clean |
| 272 | * |
| 273 | * Sanitizes data so that Cross Site Scripting Hacks can be |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 274 | * prevented. This function does a fair amount of work but |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 275 | * it is extremely thorough, designed to prevent even the |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 276 | * most obscure XSS attempts. Nothing is ever 100% foolproof, |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 277 | * of course, but I haven't been able to get anything passed |
| 278 | * the filter. |
| 279 | * |
| 280 | * Note: This function should only be used to deal with data |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 281 | * upon submission. It's not something that should |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 282 | * be used for general runtime processing. |
| 283 | * |
| 284 | * This function was based in part on some code and ideas I |
| 285 | * got from Bitflux: http://channel.bitflux.ch/wiki/XSS_Prevention |
| 286 | * |
| 287 | * To help develop this script I used this great list of |
| 288 | * vulnerabilities along with a few other hacks I've |
| 289 | * harvested from examining vulnerabilities in other programs: |
| 290 | * http://ha.ckers.org/xss.html |
| 291 | * |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 292 | * @param mixed string or array |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 293 | * @param bool |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 294 | * @return string |
| 295 | */ |
Eric Barnes | 9805ecc | 2011-01-16 23:35:16 -0500 | [diff] [blame] | 296 | public function xss_clean($str, $is_image = FALSE) |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 297 | { |
| 298 | /* |
| 299 | * Is the string an array? |
| 300 | * |
| 301 | */ |
| 302 | if (is_array($str)) |
| 303 | { |
| 304 | while (list($key) = each($str)) |
| 305 | { |
| 306 | $str[$key] = $this->xss_clean($str[$key]); |
| 307 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 308 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 309 | return $str; |
| 310 | } |
| 311 | |
| 312 | /* |
| 313 | * Remove Invisible Characters |
| 314 | */ |
Greg Aker | 757dda6 | 2010-04-14 19:06:19 -0500 | [diff] [blame] | 315 | $str = remove_invisible_characters($str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 316 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 317 | // Validate Entities in URLs |
| 318 | $str = $this->_validate_entities($str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 319 | |
| 320 | /* |
| 321 | * URL Decode |
| 322 | * |
| 323 | * Just in case stuff like this is submitted: |
| 324 | * |
| 325 | * <a href="http://%77%77%77%2E%67%6F%6F%67%6C%65%2E%63%6F%6D">Google</a> |
| 326 | * |
| 327 | * Note: Use rawurldecode() so it does not remove plus signs |
| 328 | * |
| 329 | */ |
| 330 | $str = rawurldecode($str); |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 331 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 332 | /* |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 333 | * Convert character entities to ASCII |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 334 | * |
| 335 | * This permits our tests below to work reliably. |
| 336 | * We only convert entities that are within tags since |
| 337 | * these are the ones that will pose security problems. |
| 338 | * |
| 339 | */ |
| 340 | |
| 341 | $str = preg_replace_callback("/[a-z]+=([\'\"]).*?\\1/si", array($this, '_convert_attribute'), $str); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 342 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 343 | $str = preg_replace_callback("/<\w+.*?(?=>|<|$)/si", array($this, '_decode_entity'), $str); |
| 344 | |
| 345 | /* |
| 346 | * Remove Invisible Characters Again! |
| 347 | */ |
Greg Aker | 757dda6 | 2010-04-14 19:06:19 -0500 | [diff] [blame] | 348 | $str = remove_invisible_characters($str); |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 349 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 350 | /* |
| 351 | * Convert all tabs to spaces |
| 352 | * |
| 353 | * This prevents strings like this: ja vascript |
| 354 | * NOTE: we deal with spaces between characters later. |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 355 | * NOTE: preg_replace was found to be amazingly slow here on |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 356 | * large blocks of data, so we use str_replace. |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 357 | */ |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 358 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 359 | if (strpos($str, "\t") !== FALSE) |
| 360 | { |
| 361 | $str = str_replace("\t", ' ', $str); |
| 362 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 363 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 364 | /* |
| 365 | * Capture converted string for later comparison |
| 366 | */ |
| 367 | $converted_string = $str; |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 368 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 369 | // Remove Strings that are never allowed |
| 370 | $str = $this->_do_never_allowed($str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 371 | |
| 372 | /* |
| 373 | * Makes PHP tags safe |
| 374 | * |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 375 | * Note: XML tags are inadvertently replaced too: |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 376 | * |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 377 | * <?xml |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 378 | * |
| 379 | * But it doesn't seem to pose a problem. |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 380 | */ |
| 381 | if ($is_image === TRUE) |
| 382 | { |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 383 | // Images have a tendency to have the PHP short opening and |
| 384 | // closing tags every so often so we skip those and only |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 385 | // do the long opening tags. |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 386 | $str = preg_replace('/<\?(php)/i', "<?\\1", $str); |
| 387 | } |
| 388 | else |
| 389 | { |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 390 | $str = str_replace(array('<?', '?'.'>'), array('<?', '?>'), $str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 391 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 392 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 393 | /* |
| 394 | * Compact any exploded words |
| 395 | * |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 396 | * This corrects words like: j a v a s c r i p t |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 397 | * These words are compacted back to their correct state. |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 398 | */ |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 399 | $words = array( |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 400 | 'javascript', 'expression', 'vbscript', 'script', |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 401 | 'applet', 'alert', 'document', 'write', 'cookie', 'window' |
| 402 | ); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 403 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 404 | foreach ($words as $word) |
| 405 | { |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 406 | $word = implode("\s*", str_split($word)) . "\s*"; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 407 | |
| 408 | // We only want to do this when it is followed by a non-word character |
| 409 | // That way valid stuff like "dealer to" does not become "dealerto" |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 410 | $str = preg_replace_callback('#('.substr($word, 0, -3).')(\W)#is', array($this, '_compact_exploded_words'), $str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 411 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 412 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 413 | /* |
| 414 | * Remove disallowed Javascript in links or img tags |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 415 | * We used to do some version comparisons and use of stripos for PHP5, |
| 416 | * but it is dog slow compared to these simplified non-capturing |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 417 | * preg_match(), especially if the pattern exists in the string |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 418 | */ |
| 419 | do |
| 420 | { |
| 421 | $original = $str; |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 422 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 423 | if (preg_match("/<a/i", $str)) |
| 424 | { |
| 425 | $str = preg_replace_callback("#<a\s+([^>]*?)(>|$)#si", array($this, '_js_link_removal'), $str); |
| 426 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 427 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 428 | if (preg_match("/<img/i", $str)) |
| 429 | { |
| 430 | $str = preg_replace_callback("#<img\s+([^>]*?)(\s?/?>|$)#si", array($this, '_js_img_removal'), $str); |
| 431 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 432 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 433 | if (preg_match("/script/i", $str) OR preg_match("/xss/i", $str)) |
| 434 | { |
| 435 | $str = preg_replace("#<(/*)(script|xss)(.*?)\>#si", '[removed]', $str); |
| 436 | } |
| 437 | } |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 438 | while($original != $str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 439 | |
| 440 | unset($original); |
| 441 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 442 | // Remove evil attributes such as style, onclick and xmlns |
| 443 | $str = $this->_remove_evil_attributes($str, $is_image); |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 444 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 445 | /* |
| 446 | * Sanitize naughty HTML elements |
| 447 | * |
| 448 | * If a tag containing any of the words in the list |
| 449 | * below is found, the tag gets converted to entities. |
| 450 | * |
| 451 | * So this: <blink> |
| 452 | * Becomes: <blink> |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 453 | */ |
| 454 | $naughty = 'alert|applet|audio|basefont|base|behavior|bgsound|blink|body|embed|expression|form|frameset|frame|head|html|ilayer|iframe|input|isindex|layer|link|meta|object|plaintext|style|script|textarea|title|video|xml|xss'; |
| 455 | $str = preg_replace_callback('#<(/*\s*)('.$naughty.')([^><]*)([><]*)#is', array($this, '_sanitize_naughty_html'), $str); |
| 456 | |
| 457 | /* |
| 458 | * Sanitize naughty scripting elements |
| 459 | * |
| 460 | * Similar to above, only instead of looking for |
| 461 | * tags it looks for PHP and JavaScript commands |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 462 | * that are disallowed. Rather than removing the |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 463 | * code, it simply converts the parenthesis to entities |
| 464 | * rendering the code un-executable. |
| 465 | * |
| 466 | * For example: eval('some code') |
| 467 | * Becomes: eval('some code') |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 468 | */ |
| 469 | $str = preg_replace('#(alert|cmd|passthru|eval|exec|expression|system|fopen|fsockopen|file|file_get_contents|readfile|unlink)(\s*)\((.*?)\)#si', "\\1\\2(\\3)", $str); |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 470 | |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 471 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 472 | // Final clean up |
| 473 | // This adds a bit of extra precaution in case |
| 474 | // something got through the above filters |
| 475 | $str = $this->_do_never_allowed($str); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 476 | |
| 477 | /* |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 478 | * Images are Handled in a Special Way |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 479 | * - Essentially, we want to know that after all of the character |
| 480 | * conversion is done whether any unwanted, likely XSS, code was found. |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 481 | * If not, we return TRUE, as the image is clean. |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 482 | * However, if the string post-conversion does not matched the |
| 483 | * string post-removal of XSS, then it fails, as there was unwanted XSS |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 484 | * code found and removed/changed during processing. |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 485 | */ |
| 486 | |
| 487 | if ($is_image === TRUE) |
| 488 | { |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 489 | return ($str === $converted_string) ? TRUE : FALSE; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 490 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 491 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 492 | log_message('debug', "XSS Filtering completed"); |
| 493 | return $str; |
| 494 | } |
| 495 | |
| 496 | // -------------------------------------------------------------------- |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 497 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 498 | /** |
| 499 | * Random Hash for protecting URLs |
| 500 | * |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 501 | * @return string |
| 502 | */ |
Eric Barnes | 9805ecc | 2011-01-16 23:35:16 -0500 | [diff] [blame] | 503 | public function xss_hash() |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 504 | { |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 505 | if ($this->_xss_hash == '') |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 506 | { |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 507 | mt_srand(); |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 508 | $this->_xss_hash = md5(time() + mt_rand(0, 1999999999)); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 509 | } |
| 510 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 511 | return $this->_xss_hash; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 512 | } |
| 513 | |
| 514 | // -------------------------------------------------------------------- |
| 515 | |
| 516 | /** |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 517 | * HTML Entities Decode |
| 518 | * |
| 519 | * This function is a replacement for html_entity_decode() |
| 520 | * |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 521 | * The reason we are not using html_entity_decode() by itself is because |
| 522 | * while it is not technically correct to leave out the semicolon |
| 523 | * at the end of an entity most browsers will still interpret the entity |
| 524 | * correctly. html_entity_decode() does not convert entities without |
| 525 | * semicolons, so we are left with our own little solution here. Bummer. |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 526 | * |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 527 | * @param string |
| 528 | * @param string |
| 529 | * @return string |
| 530 | */ |
freewil | 8cc0cfe | 2011-08-27 21:53:00 -0400 | [diff] [blame] | 531 | public function entity_decode($str, $charset = NULL) |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 532 | { |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 533 | if (strpos($str, '&') === FALSE) |
freewil | 5c9b0d1 | 2011-08-28 12:15:23 -0400 | [diff] [blame] | 534 | { |
| 535 | return $str; |
| 536 | } |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 537 | |
freewil | 5c9b0d1 | 2011-08-28 12:15:23 -0400 | [diff] [blame] | 538 | if (empty($charset)) |
| 539 | { |
| 540 | $charset = config_item('charset'); |
| 541 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 542 | |
Andrey Andreev | 3d113bd | 2011-10-05 00:03:20 +0300 | [diff] [blame] | 543 | $str = html_entity_decode($str, ENT_COMPAT, $charset); |
| 544 | $str = preg_replace('~&#x(0*[0-9a-f]{2,5})~ei', 'chr(hexdec("\\1"))', $str); |
| 545 | return preg_replace('~&#([0-9]{2,4})~e', 'chr(\\1)', $str); |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 546 | } |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 547 | |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 548 | // -------------------------------------------------------------------- |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 549 | |
Derek Jones | a091147 | 2010-03-30 10:33:09 -0500 | [diff] [blame] | 550 | /** |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 551 | * Filename Security |
| 552 | * |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 553 | * @param string |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 554 | * @param bool |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 555 | * @return string |
| 556 | */ |
Eric Barnes | 9805ecc | 2011-01-16 23:35:16 -0500 | [diff] [blame] | 557 | public function sanitize_filename($str, $relative_path = FALSE) |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 558 | { |
| 559 | $bad = array( |
| 560 | "../", |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 561 | "<!--", |
| 562 | "-->", |
| 563 | "<", |
| 564 | ">", |
| 565 | "'", |
| 566 | '"', |
| 567 | '&', |
| 568 | '$', |
| 569 | '#', |
| 570 | '{', |
| 571 | '}', |
| 572 | '[', |
| 573 | ']', |
| 574 | '=', |
| 575 | ';', |
| 576 | '?', |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 577 | "%20", |
| 578 | "%22", |
| 579 | "%3c", // < |
Barry Mieny | dd67197 | 2010-10-04 16:33:58 +0200 | [diff] [blame] | 580 | "%253c", // < |
| 581 | "%3e", // > |
| 582 | "%0e", // > |
| 583 | "%28", // ( |
| 584 | "%29", // ) |
| 585 | "%2528", // ( |
| 586 | "%26", // & |
| 587 | "%24", // $ |
| 588 | "%3f", // ? |
| 589 | "%3b", // ; |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 590 | "%3d" // = |
| 591 | ); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 592 | |
Derek Jones | 2ef3759 | 2010-10-06 17:51:59 -0500 | [diff] [blame] | 593 | if ( ! $relative_path) |
| 594 | { |
| 595 | $bad[] = './'; |
| 596 | $bad[] = '/'; |
| 597 | } |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 598 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 599 | $str = remove_invisible_characters($str, FALSE); |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 600 | return stripslashes(str_replace($bad, '', $str)); |
| 601 | } |
| 602 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 603 | // ---------------------------------------------------------------- |
| 604 | |
| 605 | /** |
| 606 | * Compact Exploded Words |
| 607 | * |
| 608 | * Callback function for xss_clean() to remove whitespace from |
| 609 | * things like j a v a s c r i p t |
| 610 | * |
| 611 | * @param type |
| 612 | * @return type |
| 613 | */ |
| 614 | protected function _compact_exploded_words($matches) |
| 615 | { |
| 616 | return preg_replace('/\s+/s', '', $matches[1]).$matches[2]; |
| 617 | } |
| 618 | |
| 619 | // -------------------------------------------------------------------- |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 620 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 621 | /* |
| 622 | * Remove Evil HTML Attributes (like evenhandlers and style) |
| 623 | * |
| 624 | * It removes the evil attribute and either: |
| 625 | * - Everything up until a space |
| 626 | * For example, everything between the pipes: |
| 627 | * <a |style=document.write('hello');alert('world');| class=link> |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 628 | * - Everything inside the quotes |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 629 | * For example, everything between the pipes: |
| 630 | * <a |style="document.write('hello'); alert('world');"| class="link"> |
| 631 | * |
| 632 | * @param string $str The string to check |
| 633 | * @param boolean $is_image TRUE if this is an image |
| 634 | * @return string The string with the evil attributes removed |
| 635 | */ |
| 636 | protected function _remove_evil_attributes($str, $is_image) |
| 637 | { |
| 638 | // All javascript event handlers (e.g. onload, onclick, onmouseover), style, and xmlns |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 639 | $evil_attributes = array('on\w*', 'style', 'xmlns', 'formaction'); |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 640 | |
| 641 | if ($is_image === TRUE) |
| 642 | { |
| 643 | /* |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 644 | * Adobe Photoshop puts XML metadata into JFIF images, |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 645 | * including namespacing, so we have to allow this for images. |
| 646 | */ |
| 647 | unset($evil_attributes[array_search('xmlns', $evil_attributes)]); |
| 648 | } |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 649 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 650 | do { |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 651 | $count = 0; |
| 652 | $attribs = array(); |
| 653 | |
| 654 | // find occurrences of illegal attribute strings without quotes |
| 655 | preg_match_all("/(".implode('|', $evil_attributes).")\s*=\s*([^\s]*)/is", $str, $matches, PREG_SET_ORDER); |
| 656 | |
| 657 | foreach ($matches as $attr) |
| 658 | { |
| 659 | $attribs[] = preg_quote($attr[0], '/'); |
| 660 | } |
| 661 | |
| 662 | // find occurrences of illegal attribute strings with quotes (042 and 047 are octal quotes) |
| 663 | preg_match_all("/(".implode('|', $evil_attributes).")\s*=\s*(\042|\047)([^\\2]*?)(\\2)/is", $str, $matches, PREG_SET_ORDER); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 664 | |
Pascal Kriete | c38e3b6 | 2011-11-14 13:55:00 -0500 | [diff] [blame] | 665 | foreach ($matches as $attr) |
| 666 | { |
| 667 | $attribs[] = preg_quote($attr[0], '/'); |
| 668 | } |
| 669 | |
| 670 | // replace illegal attribute strings that are inside an html tag |
| 671 | if (count($attribs) > 0) |
| 672 | { |
| 673 | $str = preg_replace("/<(\/?[^><]+?)([^A-Za-z\-])(".implode('|', $attribs).")([\s><])([><]*)/i", '<$1$2$4$5', $str, -1, $count); |
| 674 | } |
| 675 | |
| 676 | } while ($count); |
| 677 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 678 | return $str; |
| 679 | } |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 680 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 681 | // -------------------------------------------------------------------- |
| 682 | |
| 683 | /** |
| 684 | * Sanitize Naughty HTML |
| 685 | * |
| 686 | * Callback function for xss_clean() to remove naughty HTML elements |
| 687 | * |
| 688 | * @param array |
| 689 | * @return string |
| 690 | */ |
| 691 | protected function _sanitize_naughty_html($matches) |
| 692 | { |
| 693 | // encode opening brace |
| 694 | $str = '<'.$matches[1].$matches[2].$matches[3]; |
| 695 | |
| 696 | // encode captured opening or closing brace to prevent recursive vectors |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 697 | $str .= str_replace(array('>', '<'), array('>', '<'), |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 698 | $matches[4]); |
| 699 | |
| 700 | return $str; |
| 701 | } |
| 702 | |
| 703 | // -------------------------------------------------------------------- |
| 704 | |
| 705 | /** |
| 706 | * JS Link Removal |
| 707 | * |
| 708 | * Callback function for xss_clean() to sanitize links |
| 709 | * This limits the PCRE backtracks, making it more performance friendly |
| 710 | * and prevents PREG_BACKTRACK_LIMIT_ERROR from being triggered in |
| 711 | * PHP 5.2+ on link-heavy strings |
| 712 | * |
| 713 | * @param array |
| 714 | * @return string |
| 715 | */ |
| 716 | protected function _js_link_removal($match) |
| 717 | { |
| 718 | $attributes = $this->_filter_attributes(str_replace(array('<', '>'), '', $match[1])); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 719 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 720 | return str_replace($match[1], preg_replace("#href=.*?(alert\(|alert&\#40;|javascript\:|livescript\:|mocha\:|charset\=|window\.|document\.|\.cookie|<script|<xss|base64\s*,)#si", "", $attributes), $match[0]); |
| 721 | } |
| 722 | |
| 723 | // -------------------------------------------------------------------- |
| 724 | |
| 725 | /** |
| 726 | * JS Image Removal |
| 727 | * |
| 728 | * Callback function for xss_clean() to sanitize image tags |
| 729 | * This limits the PCRE backtracks, making it more performance friendly |
| 730 | * and prevents PREG_BACKTRACK_LIMIT_ERROR from being triggered in |
| 731 | * PHP 5.2+ on image tag heavy strings |
| 732 | * |
| 733 | * @param array |
| 734 | * @return string |
| 735 | */ |
| 736 | protected function _js_img_removal($match) |
| 737 | { |
| 738 | $attributes = $this->_filter_attributes(str_replace(array('<', '>'), '', $match[1])); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 739 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 740 | return str_replace($match[1], preg_replace("#src=.*?(alert\(|alert&\#40;|javascript\:|livescript\:|mocha\:|charset\=|window\.|document\.|\.cookie|<script|<xss|base64\s*,)#si", "", $attributes), $match[0]); |
| 741 | } |
| 742 | |
| 743 | // -------------------------------------------------------------------- |
| 744 | |
| 745 | /** |
| 746 | * Attribute Conversion |
| 747 | * |
| 748 | * Used as a callback for XSS Clean |
| 749 | * |
| 750 | * @param array |
| 751 | * @return string |
| 752 | */ |
| 753 | protected function _convert_attribute($match) |
| 754 | { |
| 755 | return str_replace(array('>', '<', '\\'), array('>', '<', '\\\\'), $match[0]); |
| 756 | } |
| 757 | |
| 758 | // -------------------------------------------------------------------- |
| 759 | |
| 760 | /** |
| 761 | * Filter Attributes |
| 762 | * |
| 763 | * Filters tag attributes for consistency and safety |
| 764 | * |
| 765 | * @param string |
| 766 | * @return string |
| 767 | */ |
| 768 | protected function _filter_attributes($str) |
| 769 | { |
| 770 | $out = ''; |
| 771 | |
| 772 | if (preg_match_all('#\s*[a-z\-]+\s*=\s*(\042|\047)([^\\1]*?)\\1#is', $str, $matches)) |
| 773 | { |
| 774 | foreach ($matches[0] as $match) |
| 775 | { |
| 776 | $out .= preg_replace("#/\*.*?\*/#s", '', $match); |
| 777 | } |
| 778 | } |
| 779 | |
| 780 | return $out; |
| 781 | } |
| 782 | |
| 783 | // -------------------------------------------------------------------- |
| 784 | |
| 785 | /** |
| 786 | * HTML Entity Decode Callback |
| 787 | * |
| 788 | * Used as a callback for XSS Clean |
| 789 | * |
| 790 | * @param array |
| 791 | * @return string |
| 792 | */ |
| 793 | protected function _decode_entity($match) |
| 794 | { |
| 795 | return $this->entity_decode($match[0], strtoupper(config_item('charset'))); |
| 796 | } |
| 797 | |
| 798 | // -------------------------------------------------------------------- |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 799 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 800 | /** |
| 801 | * Validate URL entities |
| 802 | * |
| 803 | * Called by xss_clean() |
| 804 | * |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 805 | * @param string |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 806 | * @return string |
| 807 | */ |
| 808 | protected function _validate_entities($str) |
| 809 | { |
| 810 | /* |
| 811 | * Protect GET variables in URLs |
| 812 | */ |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 813 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 814 | // 901119URL5918AMP18930PROTECT8198 |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 815 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 816 | $str = preg_replace('|\&([a-z\_0-9\-]+)\=([a-z\_0-9\-]+)|i', $this->xss_hash()."\\1=\\2", $str); |
| 817 | |
| 818 | /* |
| 819 | * Validate standard character entities |
| 820 | * |
Derek Jones | 37f4b9c | 2011-07-01 17:56:50 -0500 | [diff] [blame] | 821 | * Add a semicolon if missing. We do this to enable |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 822 | * the conversion of entities to ASCII later. |
| 823 | * |
| 824 | */ |
| 825 | $str = preg_replace('#(&\#?[0-9a-z]{2,})([\x00-\x20])*;?#i', "\\1;\\2", $str); |
| 826 | |
| 827 | /* |
| 828 | * Validate UTF16 two byte encoding (x00) |
| 829 | * |
| 830 | * Just as above, adds a semicolon if missing. |
| 831 | * |
| 832 | */ |
| 833 | $str = preg_replace('#(&\#x?)([0-9A-F]+);?#i',"\\1\\2;",$str); |
| 834 | |
| 835 | /* |
| 836 | * Un-Protect GET variables in URLs |
| 837 | */ |
| 838 | $str = str_replace($this->xss_hash(), '&', $str); |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 839 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 840 | return $str; |
| 841 | } |
| 842 | |
| 843 | // ---------------------------------------------------------------------- |
| 844 | |
| 845 | /** |
| 846 | * Do Never Allowed |
| 847 | * |
| 848 | * A utility function for xss_clean() |
| 849 | * |
| 850 | * @param string |
| 851 | * @return string |
| 852 | */ |
| 853 | protected function _do_never_allowed($str) |
| 854 | { |
| 855 | foreach ($this->_never_allowed_str as $key => $val) |
| 856 | { |
| 857 | $str = str_replace($key, $val, $str); |
| 858 | } |
| 859 | |
| 860 | foreach ($this->_never_allowed_regex as $key => $val) |
| 861 | { |
| 862 | $str = preg_replace("#".$key."#i", $val, $str); |
| 863 | } |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 864 | |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 865 | return $str; |
| 866 | } |
| 867 | |
| 868 | // -------------------------------------------------------------------- |
| 869 | |
| 870 | /** |
| 871 | * Set Cross Site Request Forgery Protection Cookie |
| 872 | * |
| 873 | * @return string |
| 874 | */ |
| 875 | protected function _csrf_set_hash() |
| 876 | { |
| 877 | if ($this->_csrf_hash == '') |
| 878 | { |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 879 | // If the cookie exists we will use it's value. |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 880 | // We don't necessarily want to regenerate it with |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 881 | // each page load since a page could contain embedded |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 882 | // sub-pages causing this feature to fail |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 883 | if (isset($_COOKIE[$this->_csrf_cookie_name]) && |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 884 | $_COOKIE[$this->_csrf_cookie_name] != '') |
| 885 | { |
| 886 | return $this->_csrf_hash = $_COOKIE[$this->_csrf_cookie_name]; |
| 887 | } |
David Behler | 07b5342 | 2011-08-15 00:25:06 +0200 | [diff] [blame] | 888 | |
Chris Berthe | d93e6f3 | 2011-09-25 10:33:25 -0400 | [diff] [blame] | 889 | $this->_csrf_hash = md5(uniqid(rand(), TRUE)); |
| 890 | $this->csrf_set_cookie(); |
Pascal Kriete | c9c045a | 2011-04-05 14:50:41 -0400 | [diff] [blame] | 891 | } |
| 892 | |
| 893 | return $this->_csrf_hash; |
| 894 | } |
| 895 | |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 896 | } |
Derek Jones | e701d76 | 2010-03-02 18:17:01 -0600 | [diff] [blame] | 897 | |
| 898 | /* End of file Security.php */ |
Phil Sturgeon | c00a5a0 | 2011-11-22 15:25:32 +0000 | [diff] [blame] | 899 | /* Location: ./system/core/Security.php */ |