View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0022486 | mantisbt | security | public | 2017-03-07 05:49 | 2017-03-21 20:06 |
Reporter | elandais | Assigned To | dregad | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.3.0-beta.3 | ||||
Target Version | 1.3.7 | Fixed in Version | 1.3.7 | ||
Summary | 0022486: CVE-2017-6797: XSS in bug_change_status_page.php | ||||
Description | Cross-Site Scripting Vulnerability in 'bug_change_status_page.php' page. The /bug_change_status_page.php page 'action_type' parameter in MantisBT is vulnerable to a cross-site scripting vulnerability when Javascript is supplied via GET or POST request. The exploitation example below uses the "alert()" JavaScript function to display "XSS" word. | ||||
Steps To Reproduce | REQUEST : GET /bug_change_status_page.php?bug_change_status_page_token=20170227zM42Eo2wSNnfOOCxNSwz4Gv0xbkGWu8E&new_status=90&change_type=closev2lx8%22%3E%3Cscript%3Ealert%28%22Test%20SLCC%22%29%3C%2fscript%3Eyrcxai8ja1g&id=3524 Parameters : | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Thanks for your bug report. We will look into this ASAP.
|
|
I confirm the vulnerability, but only when CSP is disabled. |
|
@elandais Please confirm that the attached patch fully addresses the issue you reported. 0001-Fix-XSS-in-bug_change_status_page.php.patch (897 bytes)
From 658f32443b4aaa1c3285eaf970d0416ca7a020a5 Mon Sep 17 00:00:00 2001 From: Damien Regad <dregad@mantisbt.org> Date: Tue, 7 Mar 2017 12:34:16 +0100 Subject: [PATCH] Fix XSS in bug_change_status_page.php The value of the change_type parameter was not encoded before being displayed as a hidden input. This vulnerability was reported by Etienne Landais. Fixes #22486 --- bug_change_status_page.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bug_change_status_page.php b/bug_change_status_page.php index 4e1c6fc..bc23ce6 100644 --- a/bug_change_status_page.php +++ b/bug_change_status_page.php @@ -385,7 +385,7 @@ layout_page_begin(); </tbody> </table> -<input type="hidden" name="action_type" value="<?php echo $f_change_type; ?>" /> +<input type="hidden" name="action_type" value="<?php echo string_attribute( $f_change_type ); ?>" /> </div> </div> -- 1.9.1 |
|
I confirm that the attached parch fully adresses the issue I reported. Thanks |
|
@elandais you did not answer my questions in 0022486:0055944 |
|
|
|
Security researchers and firms generally like to be credited with their findings in the CVE report, and often request a specific format for such credit, with an e-mail address, web site, reference ID, etc (see example) |
|
it's ok, we don't need to be credited for this discovery. Thanks |
|
Git blame indicates that this bug was introduced in MantisBT master f591e7b6 - 1.3.0-beta.3 so targeting 1.3.7 |
|
CVE Request 304568 for CVE ID Request sent |
|
MantisBT: master-1.3.x a2d90eca 2017-03-07 01:34 Details Diff |
Fix XSS in bug_change_status_page.php The value of the change_type parameter was not encoded before being displayed as a hidden input. This vulnerability was reported by Etienne Landais. Fixes 0022486 |
Affected Issues 0022486 |
|
mod - bug_change_status_page.php | Diff File | ||
MantisBT: master-2.2 c272c3f6 2017-03-07 01:34 Details Diff |
Fix XSS in bug_change_status_page.php The value of the change_type parameter was not encoded before being displayed as a hidden input. This vulnerability was reported by Etienne Landais. Fixes 0022486 |
Affected Issues 0022486 |
|
mod - bug_change_status_page.php | Diff File | ||
MantisBT: master-2.1 55b5b4f3 2017-03-07 01:34 Details Diff |
Fix XSS in bug_change_status_page.php The value of the change_type parameter was not encoded before being displayed as a hidden input. This vulnerability was reported by Etienne Landais. Fixes 0022486 |
Affected Issues 0022486 |
|
mod - bug_change_status_page.php | Diff File |