-
Notifications
You must be signed in to change notification settings - Fork 0
Update search.php #47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
✅ No security or compliance issues detected. Reviewed everything up to 4d90086. Security Overview
Detected Code Changes
Reply to this PR with |
|
❌ Possible security or compliance issues detected. Reviewed everything up to 4d90086. Generated Fix Pull Requests Security Overview
Detected Code Changes
Reply to this PR with |
| include_once( PHPWG_ROOT_PATH.'include/common.inc.php' ); | ||
| include_once(PHPWG_ROOT_PATH.'include/functions_search.inc.php'); | ||
|
|
||
| echo $_GET['asdf']; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reflected XSS in search.php via 'asdf' Parameter (Severity: HIGH)
This reflected cross-site scripting (XSS) vulnerability allows an attacker to execute arbitrary JavaScript in a user's browser, potentially leading to session theft or other malicious actions. The issue occurs in search.php on lines 14-15, where the value of the asdf GET parameter is directly echoed into the HTTP response without proper sanitization. This allows an attacker to craft a malicious URL containing JavaScript code that will be executed when a victim visits the link.
| echo $_GET['asdf']; | |
| echo htmlspecialchars($_GET['asdf'], ENT_QUOTES | ENT_SUBSTITUTE, 'UTF-8'); |
|
❌ Possible security or compliance issues detected. Reviewed everything up to 4d90086. The following issues were found:
Security Overview
Detected Code Changes
Reply to this PR with |
|
❌ Possible security or compliance issues detected. Reviewed everything up to 4d90086. Security Overview
Detected Code Changes
Reply to this PR with |
No description provided.