Bug #7483

CVE-2014-3653 - Provisioning Templates Preview mode strips out text like <<FOO

Added by Aaron Stone about 2 years ago. Updated about 2 years ago.

Status:Closed
Priority:Normal
Assigned To:Aaron Stone
Category:Security
Target version:Sprint 29
Difficulty: Bugzilla link:
Found in release:1.6.0 Pull request:https://github.com/theforeman/foreman/pull/1778
Story points-
Velocity based estimate-
Release1.6.1Release relationshipAuto

Description

I have Foreman 1.5.1. I will try to test this against 1.5.2 and 1.6.0, but if someone else can test it first that would be grand.

Steps to reproduce:

In Provisioning Templates, click New Template.
Put this into the code box:

test <<FOO > bar

Hello World

FOO

click Preview
click Code

Now the contents are:
test < bar

Hello World

FOO

That's a pretty big problem for templates that want to use shell redirection!


Related issues

Related to Foreman - Bug #8133: template diffs don't get displayed anymore Closed 10/28/2014

Associated revisions

Revision cafa9477
Added by Aaron Stone about 2 years ago

Fixes #7483 - Use hidden input value to hold raw template contents (CVE-2014-3653)

History

#1 Updated by Aaron Stone about 2 years ago

  • Found in release set to 1.6.0

Tested, this does affect Foreman 1.5.2 and 1.6.0.

I posted screenshots of this bug in action here: https://github.com/sodabrew/foreman/issues/1

#2 Updated by Dominic Cleal about 2 years ago

  • Category set to Security
  • Status changed from New to Assigned
  • Assigned To set to Aaron Stone
  • Target version set to Sprint 29
  • Release set to 1.6.1

Thanks for the report. This has a security impact as it seems to be rendered as HTML, we're getting a CVE assigned. Please go ahead and submit your fix, we'll get it into 1.6.1.

#3 Updated by The Foreman Bot about 2 years ago

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

#4 Updated by Dominic Cleal about 2 years ago

  • Pull request https://github.com/theforeman/foreman/pull/1778 added

#5 Updated by Dominic Cleal about 2 years ago

  • Subject changed from Provisioning Templates Preview mode strips out text like <<FOO to CVE-2014-3653 - Provisioning Templates Preview mode strips out text like <<FOO

#6 Updated by Aaron Stone about 2 years ago

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

#7 Updated by Michael Moll almost 2 years ago

  • Related to Bug #8133: template diffs don't get displayed anymore added

Also available in: Atom PDF