Headline
GHSA-2fqm-m4r2-fh98: kiwitcms vulnerable to stored cross-site scripting via unrestricted file upload
Impact
Kiwi TCMS allows users to upload attachments to test plans, test cases, etc. Earlier versions of Kiwi TCMS had introduced upload validators in order to prevent potentially dangerous files from being uploaded and Content-Security-Policy definition to prevent cross-site-scripting attacks.
The upload validation checks were not 100% robust which left the possibility to circumvent them and upload a potentially dangerous file which allows execution of arbitrary JavaScript in the browser. Additionally we’ve discovered that Nginx’s proxy_pass
directive will strip some headers negating protections built into Kiwi TCMS when served behind a reverse proxy.
Patches
- Improved file upload validation code
- Updated Nginx reverse proxy configuration for *.tenant.kiwitcms.org
Workarounds
If serving Kiwi TCMS behind a reverse proxy make sure that additional header values are still passed to the client browser. If they aren’t redefine them inside the proxy configuration. See etc/nginx.conf#L66-L68 and etc/nginx.conf#L87
References
Disclosed by M Nadeem Qazi.
kiwitcms vulnerable to stored cross-site scripting via unrestricted file upload
High severity GitHub Reviewed Published Jun 6, 2023 in kiwitcms/Kiwi • Updated Jun 6, 2023
Package
pip kiwitcms (pip)
Affected versions
<= 12.3
Patched versions
12.4
Description
Impact
Kiwi TCMS allows users to upload attachments to test plans, test cases, etc. Earlier versions of Kiwi TCMS had introduced upload validators in order to prevent potentially dangerous files from being uploaded and Content-Security-Policy definition to prevent cross-site-scripting attacks.
The upload validation checks were not 100% robust which left the possibility to circumvent them and upload a potentially dangerous file which allows execution of arbitrary JavaScript in the browser. Additionally we’ve discovered that Nginx’s proxy_pass directive will strip some headers negating protections built into Kiwi TCMS when served behind a reverse proxy.
Patches
- Improved file upload validation code
- Updated Nginx reverse proxy configuration for *.tenant.kiwitcms.org
Workarounds
If serving Kiwi TCMS behind a reverse proxy make sure that additional header values are still passed to the client browser. If they aren’t redefine them inside the proxy configuration. See etc/nginx.conf#L66-L68 and etc/nginx.conf#L87
References
Disclosed by M Nadeem Qazi.
References
- GHSA-2fqm-m4r2-fh98
- kiwitcms/Kiwi@d789f4b
- https://github.com/kiwitcms/Kiwi/blob/master/etc/nginx.conf#L66-L68
- https://github.com/kiwitcms/Kiwi/blob/master/etc/nginx.conf#L87
- https://huntr.dev/bounties/6aea9a26-e29a-467b-aa5a-f767f0c2ec96/
- https://kiwitcms.org/blog/kiwi-tcms-team/2023/06/06/kiwi-tcms-124/
atodorov published to kiwitcms/Kiwi
Jun 6, 2023
Published to the GitHub Advisory Database
Jun 6, 2023
Reviewed
Jun 6, 2023
Last updated
Jun 6, 2023
Severity
High
8.1
/ 10
CVSS base metrics
Attack vector
Network
Attack complexity
Low
Privileges required
None
User interaction
Required
Scope
Unchanged
Confidentiality
High
Integrity
High
Availability
None
CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:U/C:H/I:H/A:N
Weaknesses
CWE-79 CWE-434
CVE ID
CVE-2023-33977
GHSA ID
GHSA-2fqm-m4r2-fh98
Source code
kiwitcms/Kiwi
Credits
- mnqazi Reporter
Checking history
See something to contribute? Suggest improvements for this vulnerability.
Related news
Kiwi TCMS is an open source test management system for both manual and automated testing. Kiwi TCMS allows users to upload attachments to test plans, test cases, etc. Earlier versions of Kiwi TCMS had introduced upload validators in order to prevent potentially dangerous files from being uploaded and Content-Security-Policy definition to prevent cross-site-scripting attacks. The upload validation checks were not 100% robust which left the possibility to circumvent them and upload a potentially dangerous file which allows execution of arbitrary JavaScript in the browser. Additionally we've discovered that Nginx's `proxy_pass` directive will strip some headers negating protections built into Kiwi TCMS when served behind a reverse proxy. This issue has been addressed in version 12.4. Users are advised to upgrade. Users unable to upgrade who are serving Kiwi TCMS behind a reverse proxy should make sure that additional header values are still passed to the client browser. If they aren't red...