ghsa-8727-m6gj-mc37
Vulnerability from github
There is a strong parameters bypass vector in ActionPack.
Versions Affected: rails <= 6.0.3 Not affected: rails < 5.0.0 Fixed Versions: rails >= 5.2.4.3, rails >= 6.0.3.1
Impact
In some cases user supplied information can be inadvertently leaked from
Strong Parameters. Specifically the return value of each
, or each_value
,
or each_pair
will return the underlying "untrusted" hash of data that was
read from the parameters. Applications that use this return value may be
inadvertently use untrusted user input.
Impacted code will look something like this:
``
def update
# Attacker has included the parameter:
{ is_admin: true }`
User.update(clean_up_params)
end
def clean_up_params params.each { |k, v| SomeModel.check(v) if k == :name } end ```
Note the mistaken use of each
in the clean_up_params
method in the above
example.
Workarounds
Do not use the return values of each
, each_value
, or each_pair
in your
application.
{ "affected": [ { "database_specific": { "last_known_affected_version_range": "\u003c= 5.2.4.2" }, "package": { "ecosystem": "RubyGems", "name": "actionpack" }, "ranges": [ { "events": [ { "introduced": "5.0.0" }, { "fixed": "5.2.4.3" } ], "type": "ECOSYSTEM" } ] }, { "database_specific": { "last_known_affected_version_range": "\u003c= 6.0.3" }, "package": { "ecosystem": "RubyGems", "name": "actionpack" }, "ranges": [ { "events": [ { "introduced": "6.0.0" }, { "fixed": "6.0.3.1" } ], "type": "ECOSYSTEM" } ] } ], "aliases": [ "CVE-2020-8164" ], "database_specific": { "cwe_ids": [ "CWE-502" ], "github_reviewed": true, "github_reviewed_at": "2020-05-26T15:06:53Z", "nvd_published_at": "2020-06-19T17:15:00Z", "severity": "HIGH" }, "details": "There is a strong parameters bypass vector in ActionPack.\n\nVersions Affected: rails \u003c= 6.0.3\nNot affected: rails \u003c 5.0.0\nFixed Versions: rails \u003e= 5.2.4.3, rails \u003e= 6.0.3.1\n\nImpact\n------\nIn some cases user supplied information can be inadvertently leaked from\nStrong Parameters. Specifically the return value of `each`, or `each_value`,\nor `each_pair` will return the underlying \"untrusted\" hash of data that was\nread from the parameters. Applications that use this return value may be\ninadvertently use untrusted user input.\n\nImpacted code will look something like this:\n\n```\ndef update\n # Attacker has included the parameter: `{ is_admin: true }`\n User.update(clean_up_params)\nend\n\ndef clean_up_params\n params.each { |k, v| SomeModel.check(v) if k == :name }\nend\n```\n\nNote the mistaken use of `each` in the `clean_up_params` method in the above\nexample.\n\nWorkarounds\n-----------\nDo not use the return values of `each`, `each_value`, or `each_pair` in your\napplication.", "id": "GHSA-8727-m6gj-mc37", "modified": "2023-09-25T16:55:14Z", "published": "2020-05-26T15:09:16Z", "references": [ { "type": "ADVISORY", "url": "https://nvd.nist.gov/vuln/detail/CVE-2020-8164" }, { "type": "WEB", "url": "https://hackerone.com/reports/292797" }, { "type": "PACKAGE", "url": "https://github.com/rails/rails" }, { "type": "WEB", "url": "https://github.com/rubysec/ruby-advisory-db/blob/master/gems/actionpack/CVE-2020-8164.yml" }, { "type": "WEB", "url": "https://groups.google.com/forum/#!topic/rubyonrails-security/f6ioe4sdpbY" }, { "type": "WEB", "url": "https://groups.google.com/g/rubyonrails-security/c/f6ioe4sdpbY" }, { "type": "WEB", "url": "https://lists.debian.org/debian-lts-announce/2020/06/msg00022.html" }, { "type": "WEB", "url": "https://lists.debian.org/debian-lts-announce/2020/07/msg00013.html" }, { "type": "WEB", "url": "https://www.debian.org/security/2020/dsa-4766" }, { "type": "WEB", "url": "http://lists.opensuse.org/opensuse-security-announce/2020-09/msg00089.html" }, { "type": "WEB", "url": "http://lists.opensuse.org/opensuse-security-announce/2020-09/msg00093.html" }, { "type": "WEB", "url": "http://lists.opensuse.org/opensuse-security-announce/2020-09/msg00107.html" } ], "schema_version": "1.4.0", "severity": [ { "score": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:N/A:N", "type": "CVSS_V3" } ], "summary": "Possible Strong Parameters Bypass in ActionPack" }
- Seen: The vulnerability was mentioned, discussed, or seen somewhere by the user.
- Confirmed: The vulnerability is confirmed from an analyst perspective.
- Exploited: This vulnerability was exploited and seen by the user reporting the sighting.
- Patched: This vulnerability was successfully patched by the user reporting the sighting.
- Not exploited: This vulnerability was not exploited or seen by the user reporting the sighting.
- Not confirmed: The user expresses doubt about the veracity of the vulnerability.
- Not patched: This vulnerability was not successfully patched by the user reporting the sighting.