GHSA-7vw7-qx38-37vr: Propel/propel SQL Injection Vulnerability

CRITICAL
CVSS 9.8
CWE-89
Published
Advisory Summary

GHSA-7vw7-qx38-37vr is a CRITICAL (CVSS 9.8) sql injection vulnerability (CWE-89: Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection')) affecting Propel/propel libraries in the Packagist ecosystem. The issue involves propel2 sql injection possible with limit() on mysql. Remediation is available by upgrading affected packages to version 2.0.0-alpha8.

GHSA-7vw7-qx38-37vr at a Glance

CVE Identifier N/A
GHSA Identifier GHSA-7vw7-qx38-37vr
Severity CRITICAL
CVSS v3.1 Base Score 9.8
CWE Classification CWE-89: Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection')
Affected Product Propel/propel
Package Ecosystem Packagist
Published Date
Last Updated

Security Impact

CVSS: 9.8

The security impact metrics below are parsed directly from the CVSS v3.1 vector string (CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H).

Attack Vector Network (AV:N)
Attack Complexity Low (AC:L)
Privileges Required None (PR:N)
User Interaction None (UI:N)
Scope Unchanged (S:U)
Confidentiality High (C:H)
Integrity High (I:H)
Availability High (A:H)

How to Fix GHSA-7vw7-qx38-37vr

To remediate this vulnerability, upgrade all affected Propel/propel dependencies to a patched release. Both direct and transitive dependencies should be checked, as frameworks or security libraries may pull in vulnerable versions transitively.

propel/propel fixed: 2.0.0-alpha8
Upgrade propel/propel to 2.0.0-alpha8
Verify Direct and Transitive Dependencies

Run the following command in your project directory to verify that all resolved dependencies use the patched release:

inspect dependencies for propel/propel

How to Check if You Are Affected

To determine whether your application is affected by GHSA-7vw7-qx38-37vr, inspect your project dependencies for Propel/propel:

inspect dependencies for propel/propel

Analyze the terminal output to locate any version of Propel/propel in your dependency tree. If any resolved version matches the vulnerable ranges listed in the table below (for example, versions earlier than 2.0.0-alpha8), your application is affected.

Note: Verify declared build manifests and deployed runtime container environments.

Technical Details

Vulnerability Mechanism & Technical Analysis

This vulnerability is classified under CWE-89: Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection'), where the product constructs all or part of an sql command using externally-influenced input from an upstream component, but it does not neutralize or incorrectly neutralizes special elements that could modify the intended sql command when it is sent to a downstream component. without sufficient removal or quoting of sql syntax in user-controllable inputs, the generated sql query can cause those inputs to be interpreted as sql instead of ordinary user data. In Propel/propel, software state validation or input boundaries are circumvented during processing. Review the official source advisory statement below for specific affected components, classes, and reproduction details.

Source Advisory Statement

The limit() query method is susceptible to catastrophic SQL injection with MySQL. For example, given a model User for a table users: ``` UserQuery::create()->limit('1;DROP TABLE users')->find(); ``` This will drop the users table! The cause appears to be a lack of integer casting of the limit input in either Propel\Runtime\ActiveQuery\Criteria::setLimit() or in Propel\Runtime\Adapter\Pdo\MysqlAdapter::applyLimit(). The code comments there seem to imply that casting was avoided due to overflow issues with 32-bit integers. This is surprising behavior since one of the primary purposes of an ORM is to prevent basic SQL injection. This affects all versions of Propel: 1.x, 2.x, and 3.

GHSA-7vw7-qx38-37vr Affected Propel/propel Versions

1 component(s)
Fixed Versions:
2.0.0-alpha8
Product / Package Ecosystem Affected Versions Fixed Version
propel/propel
Packagist
≥ 2.0.0-alpha1
< 2.0.0-alpha8
2.0.0-alpha8

References

Frequently Asked Questions

What is GHSA-7vw7-qx38-37vr?
GHSA-7vw7-qx38-37vr is a security vulnerability in Propel/propel involving Propel2 SQL injection possible with limit() on MySQL under CWE-89: Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection').
Which Propel/propel versions are affected?
Affected versions include releases prior to 2.0.0-alpha8 across affected packages: propel/propel.
What version fixes GHSA-7vw7-qx38-37vr?
Upgrading to 2.0.0-alpha8 (or the latest patched release for your respective branch) resolves the vulnerability.
Is propel/propel affected?
Yes, propel/propel is affected in releases prior to 2.0.0-alpha8. Upgrade your dependency to 2.0.0-alpha8 or later.
What is the CVSS score of GHSA-7vw7-qx38-37vr?
GHSA-7vw7-qx38-37vr has a CVSS v3.1 base score of 9.8 (CRITICAL) with vector string CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H.
What CWE is associated with GHSA-7vw7-qx38-37vr?
GHSA-7vw7-qx38-37vr is classified under CWE-89: Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') (The product constructs all or part of an SQL command using externally-influenced input from an upstream component, but it does not neutralize or incorrectly neutralizes special elements that could modify the intended SQL command when it is sent to a downstream component. Without sufficient removal or quoting of SQL syntax in user-controllable inputs, the generated SQL query can cause those inputs to be interpreted as SQL instead of ordinary user data.).
How can I check Packagist dependencies for GHSA-7vw7-qx38-37vr?
Run the dependency tree command: 'inspect dependencies for propel/propel'. This command lists direct and transitive dependencies in your build configuration so you can verify if a vulnerable version is resolved.
Machine-readable OSV JSON OSV v1.5 Schema
Normalized schema output
{
  "schema_version" : "1.4.0",
  "id" : "GHSA-7vw7-qx38-37vr",
  "modified" : "2024-05-20T17:34:33Z",
  "published" : "2024-05-20T17:34:33Z",
  "aliases" : [ ],
  "summary" : "Propel2 SQL injection possible with limit() on MySQL",
  "details" : "The limit() query method is susceptible to catastrophic SQL injection with MySQL.\n\nFor example, given a model User for a table users:\n```\nUserQuery::create()->limit('1;DROP TABLE users')->find();\n```\nThis will drop the users table!\n\nThe cause appears to be a lack of integer casting of the limit input in either Propel\\Runtime\\ActiveQuery\\Criteria::setLimit() or in Propel\\Runtime\\Adapter\\Pdo\\MysqlAdapter::applyLimit(). The code comments there seem to imply that casting was avoided due to overflow issues with 32-bit integers.\n\nThis is surprising behavior since one of the primary purposes of an ORM is to prevent basic SQL injection.\n\nThis affects all versions of Propel: 1.x, 2.x, and 3.",
  "severity" : [ {
    "type" : "CVSS_V3",
    "score" : "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H"
  } ],
  "affected" : [ {
    "package" : {
      "ecosystem" : "Packagist",
      "name" : "propel/propel"
    },
    "ranges" : [ {
      "type" : "ECOSYSTEM",
      "events" : [ {
        "introduced" : "2.0.0-alpha1"
      }, {
        "fixed" : "2.0.0-alpha8"
      } ]
    } ],
    "database_specific" : {
      "last_known_affected_version_range" : "<= 2.0.0-alpha7"
    }
  } ],
  "references" : [ {
    "type" : "WEB",
    "url" : "https://github.com/propelorm/Propel2/issues/1463"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/propelorm/Propel2/pull/1464"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/propelorm/Propel2/commit/cd23d7384a15cfe203e23b3a835c8ab1d81d9246"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/FriendsOfPHP/security-advisories/blob/master/propel/propel/2018-02-14.yaml"
  }, {
    "type" : "PACKAGE",
    "url" : "https://github.com/propelorm/Propel2"
  } ],
  "database_specific" : {
    "cwe_ids" : [ "CWE-89" ],
    "severity" : "CRITICAL",
    "github_reviewed" : true,
    "github_reviewed_at" : "2024-05-20T17:34:33Z",
    "nvd_published_at" : null
  }
}