Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.
Bug 1064140 - (CVE-2014-0057) CVE-2014-0057 CFME: Dangerous send in ServiceController
CVE-2014-0057 CFME: Dangerous send in ServiceController
Status: CLOSED ERRATA
Product: Security Response
Classification: Other
Component: vulnerability (Show other bugs)
unspecified
All Linux
high Severity high
: ---
: ---
Assigned To: Red Hat Product Security
impact=important,public=20140311,repo...
: Security
Depends On: 1064143 1064144
Blocks: 1064142
  Show dependency treegraph
 
Reported: 2014-02-12 00:40 EST by Kurt Seifried
Modified: 2014-08-25 10:12 EDT (History)
11 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2014-03-17 15:58:50 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2014:0215 normal SHIPPED_LIVE Critical: cfme security, bug fix, and enhancement update 2014-03-11 16:56:48 EDT

  None (edit)
Description Kurt Seifried 2014-02-12 00:40:49 EST
Jan Rusnacko of the Red Hat Product Security Team reports:

CFME contains an unsafe invocation of send method on user-supplied
argument. This issue is reported by Brakeman as problem in ServiceController
method x_button:

vmdb/app/controllers/service_controller.rb

16 def x_button
17 @explorer = true
18 model, action = pressed2model_action(params[:pressed])
19 @sb[:action] = action
20 if ["ownership","tag"].include?(action)
21 self.send(params[:pressed],"Service")
22 else
23 self.send(params[:pressed])
24 end

Assuming pressed2model_action sanitizes user input, this look like
false-positive. However, pressed2model_action does not do sanitization, and only
splits string on underscore:

def pressed2model_action(pressed)
pressed =~ /^(vm_vdi|miq_template)_(.*)$/ ? [$1, $2] : pressed.split('_', 2)
end

If action parsed from user-supplied input is neither "ownership", nor "tag",
else branch is executed and client can execute arbitrary method on controller,
including private.
Comment 3 Martin Prpič 2014-02-24 13:58:03 EST
Acknowledgements:

This issue was discovered by Jan Rusnacko of the Red Hat Product Security Team.
Comment 4 errata-xmlrpc 2014-03-11 12:59:13 EDT
This issue has been addressed in following products:

  CloudForms Management Engine 5.x

Via RHSA-2014:0215 https://rhn.redhat.com/errata/RHSA-2014-0215.html

Note You need to log in before you can comment on or make changes to this bug.