{"id":"GHSA-7vw7-qx38-37vr","aliases":[],"url":"https://o3.security/vulnerability/GHSA-7vw7-qx38-37vr","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.","published":"2024-05-20T17:34:33Z","modified":"2024-12-05T05:32:18.600809Z","cvss":{"score":9.8,"severity":"CRITICAL","vector":"CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H"},"epss":null,"cisaKev":null,"exploitsKnown":0,"affectedPackages":[{"ecosystem":"Packagist","name":"propel/propel","fixedVersion":"2.0.0-alpha8"}],"fix":{"url":"https://github.com/propelorm/Propel2/pull/1464","label":"propelorm/Propel2#1464"},"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"}],"provenance":{"sources":["OSV.dev","FIRST.org (EPSS)"],"lastVerified":"2024-12-05T05:32:18.600809Z"}}