XSS Vulnerability
|
4 posts
|
Hi Philippe.
|
|
8 posts
|
I could not reproduce it in version 0.95 RC2 |
|
4 posts
|
Great, will upgrade from 0.94 to 0.95 then. |
|
4 posts
|
Nope, 0.95 RC2 still the same:
|
|
1493 posts
|
Hmmm… I’m puzzled too. I can’t reproduce this on either 0.9.4 or 0.9.5 with or without mod_rewrite enabled. I get “access forbidden” messages or 404’s. |
|
4 posts
|
Are you using IE7 too? |
|
4 posts
|
Look I am not making it up (madebyfrog.com):
|
|
651 posts
|
When I try to replicate this with madebyfrog.com on FireFox I get:
I think you are correct in your conclusion that Frog is vulnerable to XSS. It would seem that Frog is not cleaning up the URL before using it. The reason why people can’t replicate the problem on FF is because it (quite correctly) encodes the HTML characters. Internet Explorer should also be encoding the characters, so I consider that a bug in IE. However, Frog should not assume that HTTP requests for URLs come only through browsers. Hackers will not be using FireFox to make their attacks. Issue added. I’ll see about fixing this in 0.9.5 final. |
|
1493 posts
|
It seems to vary not only by browser, but also by host. At a rough estimate, about 50% of the Frog sites I tried this on echoed the scripted message. It also varied with browser (as you note), with Opera being very quick to comply :) on some occasions. But Martijn is on the case, I see. ;) |
|
31 posts
|
yep i am able to reproduce on my site aswell |
|
541 posts
|
haha interesting !! and funny thanks for that |
|
257 posts
|
An interesting little article that just popped up in my RSS regarding how common this problem is and how to combat it… |
|
21 posts
|
I did some automated testing of my own site to see if there are anymore vulnerabilities. Below are the findings: Vulnerable to URL Cross-Site Scripting:
Vulnerable to Cross-Site Scripting: URL: http://www.frogdomain.com:80/admin/login/forgot Request:
Response:
The forgot password script allows the user to modify the input and process the additional “><sCrIpT>alert(54231)</sCrIpT>” />” |
|
7 posts
|
We need to filter all inputs by remove any characters, except letter and or number. Sample filtering with htmlentities:
Frog developer should fix this problem and release new version that free from XSS Vulnerability. |
|
18 posts
|
Are security problems fixed yet? |
|
4 posts
|
I’d be interested as well to know if it is fixed yet or not… |


