LibreHealthIO / lh-ehr

LibreHealth EHR - Free Open Source Electronic Health Records
Other
241 stars 264 forks source link

[security vulnerability] Cross-Site Scripting (XSS) vulnerabilities #1693

Open GatekeeperBuster opened 1 year ago

GatekeeperBuster commented 1 year ago

Recently, our team found a reflected cross-site scripting (XSS) vulnerability The vulnerability logic is present in the file: https://github.com/LibreHealthIO/lh-ehr/blob/master/interface/billing/get_claim_file.php#L25

image

The echo directly outputs the parameter $_GET['key'] without any sanitization. This makes it susceptible to Cross-Site Scripting (XSS) attacks. As a result, attackers can exploit this vulnerability by injecting malicious html code with $_GET['key']

To fix this vulnerability, we recommend that developers implement properly sanitize (e.g., htmlspecialchars()) for user input before displaying it on the webpage.