Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

We've moved the forum!

Please use forum.silverstripe.org for any new questions (announcement).
The forum archive will stick around, but will be read only.

You can also use our Slack channel or StackOverflow to ask for help.
Check out our community overview for more options to contribute.

Archive /

Our old forums are still available as a read-only archive.

Moderators: martimiz, Sean, Ed, biapar, Willr, Ingo

XSS in search module.


Go to End


2 Posts   3130 Views

Avatar
mateusz.ujma

Google Summer of Code Hacker, 6 Posts

2 April 2007 at 7:42am

Avatar
Sigurd

Forum Moderator, 628 Posts

4 April 2007 at 12:46pm

Edited: 04/04/2007 5:23pm

Thanks very much for pointing this out mateusz, its immensely helpful for people to be notifying us of security issues like this! Sean has looked into this and fixed it yesterday, so it is available for download in our daily builds. It has been escalated to be included in our 2.0.1 release too, hence we just built 2.0.1rc4 ...

patch:

search/SearchForm.php (revision 33165)
public function getSearchQuery() {
- return $_REQUEST['Search'];
+ return Convert::raw2xml($_REQUEST['Search']);
}