Impact
Affected versions of Winter CMS allow authenticated backend users with CMS template-editing permissions to escape the Twig sandbox ("safe mode") that is meant to restrict what template code can do. Using any of the following permissions, an attacker can read and modify arbitrary database records, execute arbitrary SQL (including DDL such as DROP TABLE), exfiltrate sensitive data such as backend administrator credentials, and achieve remote code execution by injecting PHP into a CMS page, layout, or partial code section:
cms.manage_pagescms.manage_layoutscms.manage_partials
This is an incomplete-fix follow-up to CVE-2024-54149 (GHSA-xhw3-4j3m-hq53). That fix added a blocklist of dangerous methods to System\Twig\SecurityPolicy, but the blocklist missed a large number of equivalent methods and did not account for the way Eloquent models forward calls to the query builder. As a result the sandbox could be bypassed through — among others — saveQuietly()/deleteQuietly(), increment()/decrement(), newQuery(), getConnection(), getConnectionResolver(), relation and pivot methods, and higher-order collection methods that execute callables.
To actively exploit this issue, an attacker would need an authenticated backend account with one of the permissions listed above. These permissions are intended for trusted developers/administrators, and the sandbox is the additional protection layer this advisory is concerned with.
Patches
System\Twig\SecurityPolicy has been reworked so that the blocklist reflects the real method-forwarding behaviour of the database layer rather than a flat list of method names. A method blocked on the query builder is now also blocked when it is reached through a model, Eloquent builder, or relation (via a transitive forwarder chain), which closes the __call forwarding escape that made the previous blocklist bypassable. In addition, the per-class blocklists have been expanded, the database connection and connection resolver are locked down, the source() and constant() Twig functions are restricted, and a SafeCollection/SafePaginator layer neutralises callable arguments passed to higher-order collection and paginator methods. Read-only query building continues to work as before; only data modification, raw SQL/connection access, and callable execution are blocked.
This security issue has been fixed in v1.2.13.
After upgrading, clear the compiled Twig template cache (e.g. php artisan cache:clear) so that existing templates recompile under the updated policy.
Workarounds
If you cannot upgrade immediately, apply wintercms/winter@725bbcd manually. As an interim mitigation, restrict cms.manage_pages, cms.manage_layouts, and cms.manage_partials to fully trusted administrators only, since these permissions grant the ability to edit template code that the sandbox is designed to contain.
References
Credit to Mounir Elsrogy (@M9nx) for reporting the issue.
For more information
If you have any questions or comments about this advisory:
References
M9nx Reporter
Impact
Affected versions of Winter CMS allow authenticated backend users with CMS template-editing permissions to escape the Twig sandbox ("safe mode") that is meant to restrict what template code can do. Using any of the following permissions, an attacker can read and modify arbitrary database records, execute arbitrary SQL (including DDL such as
DROP TABLE), exfiltrate sensitive data such as backend administrator credentials, and achieve remote code execution by injecting PHP into a CMS page, layout, or partial code section:cms.manage_pagescms.manage_layoutscms.manage_partialsThis is an incomplete-fix follow-up to CVE-2024-54149 (GHSA-xhw3-4j3m-hq53). That fix added a blocklist of dangerous methods to
System\Twig\SecurityPolicy, but the blocklist missed a large number of equivalent methods and did not account for the way Eloquent models forward calls to the query builder. As a result the sandbox could be bypassed through — among others —saveQuietly()/deleteQuietly(),increment()/decrement(),newQuery(),getConnection(),getConnectionResolver(), relation and pivot methods, and higher-order collection methods that execute callables.To actively exploit this issue, an attacker would need an authenticated backend account with one of the permissions listed above. These permissions are intended for trusted developers/administrators, and the sandbox is the additional protection layer this advisory is concerned with.
Patches
System\Twig\SecurityPolicyhas been reworked so that the blocklist reflects the real method-forwarding behaviour of the database layer rather than a flat list of method names. A method blocked on the query builder is now also blocked when it is reached through a model, Eloquent builder, or relation (via a transitive forwarder chain), which closes the__callforwarding escape that made the previous blocklist bypassable. In addition, the per-class blocklists have been expanded, the database connection and connection resolver are locked down, thesource()andconstant()Twig functions are restricted, and aSafeCollection/SafePaginatorlayer neutralises callable arguments passed to higher-order collection and paginator methods. Read-only query building continues to work as before; only data modification, raw SQL/connection access, and callable execution are blocked.This security issue has been fixed in v1.2.13.
After upgrading, clear the compiled Twig template cache (e.g.
php artisan cache:clear) so that existing templates recompile under the updated policy.Workarounds
If you cannot upgrade immediately, apply wintercms/winter@725bbcd manually. As an interim mitigation, restrict
cms.manage_pages,cms.manage_layouts, andcms.manage_partialsto fully trusted administrators only, since these permissions grant the ability to edit template code that the sandbox is designed to contain.References
Credit to Mounir Elsrogy (@M9nx) for reporting the issue.
For more information
If you have any questions or comments about this advisory:
References