Skip to content

Commit 725f041

Browse files
committed
Reverted commit FusionGen@40d0ae7 - $Key should not be disabled as this is a helper function used to sanitize context. With this disabled then potential security risks become reality. We need to figure out how to fix issue FusionGen#22 in a different way than this.
1 parent ef0886a commit 725f041

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

system/core/Input.php

+8-8
Original file line numberDiff line numberDiff line change
@@ -610,14 +610,14 @@ function _sanitize_globals()
610610
}
611611
}
612612

613-
// Clean $_POST Data
614-
// if (is_array($_POST) AND count($_POST) > 0)
615-
// {
616-
// foreach ($_POST as $key => $val)
617-
// {
618-
// $_POST[$this->_clean_input_keys($key)] = $this->_clean_input_data($val);
619-
// }
620-
// }
613+
// Clean $_POST Data - This should under no circumstances be disabled as it can potentially cause a major security risk.
614+
if (is_array($_POST) AND count($_POST) > 0)
615+
{
616+
foreach ($_POST as $key => $val)
617+
{
618+
$_POST[$this->_clean_input_keys($key)] = $this->_clean_input_data($val);
619+
}
620+
}
621621

622622
// Clean $_COOKIE Data
623623
if (is_array($_COOKIE) AND count($_COOKIE) > 0)

0 commit comments

Comments
 (0)