Project

General

Profile

Bug #16024

Foreman form helpers do not escape JS when rendering label

Added by Marek Hulán about 2 years ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Security
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

The issue was introduced in Foreman 1.6. There's only one dynamic :label => in Foreman that uses MailNotification name which we don't allow users to modify so there's no vulnerable code in Foreman. But remote execution plugin that rely on this label to be escaped. Setting to 1.12.2, feel free to reset. For REX this is pretty important though.


Related issues

Related to Foreman Remote Execution - Bug #16019: CVE-2016-6319 - Persistent XSS in job invocation form triggered by unescaped user input nameResolved2016-08-09

Associated revisions

Revision 0f35fe14 (diff)
Added by Marek Hulán about 2 years ago

Fixes #16024 - escape labels of form fields

Revision 15a6c9b1 (diff)
Added by Marek Hulán about 2 years ago

Fixes #16024 - escape labels of form fields

(cherry picked from commit 0f35fe14acf0d0d3b55e9337bc5e2b9640ff2372)

History

#1 Updated by The Foreman Bot about 2 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3715 added

#2 Updated by Marek Hulán about 2 years ago

  • Category changed from Web Interface to Security

#3 Updated by Marek Hulán about 2 years ago

  • Related to Bug #16019: CVE-2016-6319 - Persistent XSS in job invocation form triggered by unescaped user input name added

#4 Updated by Marek Hulán about 2 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF