{"id":"GHSA-7vw7-qx38-37vr","aliases":[],"status":"published","title":"Propel2 SQL injection possible with limit() on MySQL","description":"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()-\u003elimit('1;DROP TABLE users')-\u003efind();\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":{"version":"3.1","score":9.8,"vector":"CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H","level":"critical","from":"ghsa"},"facts":{"in_kev":false,"known_exploited":false,"public_exploit":false,"fix_available":true},"cwes":["CWE-89"],"published":"2024-05-20T17:34:33Z","modified":"2024-12-05T05:32:18Z","fixes":[{"ecosystem":"Packagist","package":"propel/propel","fixed_in":["2.0.0-alpha8"],"sources":["ghsa","osv"]}],"affected":[{"source":"ghsa","ecosystem":"Packagist","package":"propel/propel","ranges":[{"type":"ECOSYSTEM","events":[{"introduced":"2.0.0-alpha1"},{"last_affected":"2.0.0-alpha7"},{"fixed":"2.0.0-alpha8"}]}]},{"source":"osv","ecosystem":"Packagist","package":"propel/propel","ranges":[{"type":"ECOSYSTEM","events":[{"introduced":"2.0.0-alpha1"},{"fixed":"2.0.0-alpha8"}]}],"versions":["2.0.0-alpha1","2.0.0-alpha2","2.0.0-alpha3","2.0.0-alpha4","2.0.0-alpha5","2.0.0-alpha6","2.0.0-alpha7"]}],"references":[{"url":"https://github.com/propelorm/Propel2/issues/1463","tags":["web"]},{"url":"https://github.com/propelorm/Propel2/pull/1464","tags":["web"]},{"url":"https://github.com/propelorm/Propel2/commit/cd23d7384a15cfe203e23b3a835c8ab1d81d9246","tags":["web"]},{"url":"https://github.com/FriendsOfPHP/security-advisories/blob/master/propel/propel/2018-02-14.yaml","tags":["web"]},{"url":"https://github.com/advisories/GHSA-7vw7-qx38-37vr"},{"url":"https://github.com/propelorm/Propel2","tags":["package"]}],"sources":{"ghsa":[{"id":"GHSA-7vw7-qx38-37vr","title":"Propel2 SQL injection possible with limit() on MySQL","description":"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()-\u003elimit('1;DROP TABLE users')-\u003efind();\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.","status":"published","published":"2024-05-20T17:34:33Z","modified":"2024-05-20T17:34:34Z","cvss":[{"version":"3.1","score":9.8,"vector":"CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H","level":"critical"}],"cwes":["CWE-89"],"updated_at":"2026-11-02T14:37:50.750599Z"}],"osv":[{"id":"GHSA-7vw7-qx38-37vr","title":"Propel2 SQL injection possible with limit() on MySQL","description":"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()-\u003elimit('1;DROP TABLE users')-\u003efind();\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.","status":"published","published":"2024-05-20T17:34:33Z","modified":"2024-12-05T05:32:18.600809Z","cvss":[{"version":"3.1","score":9.8,"vector":"CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H","level":"critical"}],"cwes":["CWE-89"],"updated_at":"2026-11-02T15:46:41.349637Z"}]}}