Skip to content

Commit

Permalink
When cleaning input data, set url_encode to be false when removing in…
Browse files Browse the repository at this point in the history
…visible characters. The $_POST, $_GET, and $_COOKIE variables are not urlencoded, but remove_invisible_characters assume that it is and will remove characters such as "%00" unless the url_encoded parameter is set to false.
  • Loading branch information
veenix committed Mar 30, 2012
1 parent 3396d53 commit 9df2447
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion system/core/Input.php
Expand Up @@ -530,7 +530,7 @@ protected function _clean_input_data($str)
}

// Remove control characters
$str = remove_invisible_characters($str);
$str = remove_invisible_characters($str, false);

// Should we filter the input data?
if ($this->_enable_xss === TRUE)
Expand Down

1 comment on commit 9df2447

@cwillenbrock
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just spent the past 5 hours trying to figure out what was wrong with my feed, and found this post.

THANK YOU!!!

Please sign in to comment.