GHSA-9p7c-v5x3-rfx8: Medium severity composer/craftcms/cms vulnerability
The reorder-sets action in Craft CMS’s GlobalsController is missing the requireAdmin() check that the adjacent save-set and delete-set actions both enforce. Any authenticated control panel user can POST to /actions/globals/reorder-sets and permanently reorder all global sets in the project config, regardless of whether they have admin access. The reordering is written through to the project config and persists across requests.
Description
GlobalsController exposes three administrative actions for managing global set structure. Two of them gate on admin status; the third does not.
Prerequisites
- A Craft CMS instance with at least two global sets and a non-admin control panel user account.
Impact
A non-admin control panel user can reorder all global sets. While this does not expose or modify content, reordering global sets modifies the project config -- a versioned artifact that is typically committed to source control and deployed across environments. An attacker can create noise in the project config history, trigger config-sync conflicts, or manipulate the display order seen by all editors in the admin panel. The same non-admin user cannot create or delete global sets because those actions correctly enforce requireAdmin().
Affected Software
Remediation
Recommended actions to resolve this vulnerability, in priority order.
- Upgrade
Upgrade
composer/craftcms/cmsto a version that resolves this vulnerability.Fixed in 5.10.3 - Upgrade
Upgrade
composer/craftcms/cmsto a version that resolves this vulnerability.Fixed in 4.18.1 - Configuration
Add/enable the missing requireAdmin() authorization check in the GlobalsController `reorder-sets` action (the `/actions/globals/reorder-sets` endpoint) so that non-admin control panel users cannot permanently reorder all global sets.
Craft CMS GlobalsController Require admin check for reorder-sets action = requireAdmin()