Security
Headlines
HeadlinesLatestCVEs

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.

ghsa
#xss#vulnerability#git#java#nginx

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

CVE-2023-33977: Improve upload validation to check for dangerous attributes · kiwitcms/Kiwi@d789f4b

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...