Cross-site Scripting (XSS) - Reflected in bigprof-software/online-invoicing-system

Valid

Reported on

Jul 1st 2021


✍️ Description

/app/admin/pageTransferOwnership.php with sourceMemberID parameter is vulnerable to Reflected XSS. Line 216 of pageTransferOwnership.php sends unvalidated data to a web browser, which can result in the browser executing malicious code. In this case the data is sent at builtin_echo() in pageTransferOwnership.php at line 216.

🕵️‍♂️ Proof of Concept

Visit /ois/app/admin/pageTransferOwnership.php?csrf_token=097053d51fa978278e9faa090cd02bd5&sourceGroupID=2&sourceMemberID="><script>alert(1)</script> and XSS will be poped up.

POC: https://drive.google.com/file/d/1NnzkhgDzcFhEcvQXAEBjEcax0arMVAcr/view?usp=sharing

💥 Impact

The impact of an exploited XSS vulnerability on a web application varies a lot. It ranges from user's Session Hijacking, and if used in conjunction with a social engineering attack it can also lead to disclosure of sensitive data, CSRF attacks and other security vulnerabilities. By exploiting a cross-site scripting vulnerability an attacker can impersonate the victim and take over the account. If the victim has administrative rights it might even lead to code execution on the server, depending on the application and the privileges of the account. Read about the apache.org jira incident for more information on how a XSS vulnerability was used in a successful attack which also led to code execution.

References

We have contacted a member of the bigprof-software/online-invoicing-system team and are waiting to hear back a year ago
Akshay Jain
a year ago

Researcher


Just Verified, after further analysis.. it is a stored XSS as it is being stored at /app/admin/pageViewRecords.php anyhow.. :)

BigProf
a year ago

Thanks for reporting this issue. It's valid and I'll mark it now as so. However, what's the likeliness that an admin user follows a link containing reflected XSS attack to his app that also includes a valid CSRF token?

BigProf Software validated this vulnerability a year ago
Akshay Jain has been awarded the disclosure bounty
The fix bounty is now up for grabs
BigProf Software marked this as fixed with commit 41dda7 a year ago
BigProf Software has been awarded the fix bounty
This vulnerability will not receive a CVE
Akshay Jain
a year ago

Researcher


If admin panel is same for 2 admin user, then Admin-1 can steal the session cookies for Admin-2. However, I'll also test the integrity of CSRF token today if it is bypassable to make this more critical :)

BigProf
a year ago

OK, would be nice to check this :) Thanks!

to join this conversation