GHSA-VRR2-G9GH-C3JC

Vulnerability from github – Published: 2026-07-13 23:55 – Updated: 2026-07-13 23:55
VLAI
Summary
Kimai: Timesheet PATCH/POST allows assigning to project outside user's team via query_builder OR-bypass
Details

Summary

The Timesheet API PATCH /api/timesheets/{id} and POST /api/timesheets endpoints accept a user-supplied project ID and resolve it through a Symfony EntityType whose query_builder allows the submitted ID to satisfy the access predicate via an unconditional OR branch. As a result, any authenticated user can re-assign their own timesheet to any project in the database — including projects that belong to teams or customers they have no membership in and cannot otherwise see. The user can then read serialized project/customer details via GET /api/timesheets/{id}?full=true, leaking metadata (name, currency, customer hierarchy) that would otherwise be filtered out by the team ACL.

Details

Entry point — only ownership is checked in src/API/TimesheetController.php:317-355

#[IsGranted('edit', 'timesheet')]
#[Route(methods: ['PATCH'], path: '/{id}', name: 'patch_timesheet', requirements: ['id' => '\d+'])]
public function patchAction(Request $request, Timesheet $timesheet): Response
{
    ...
    $form = $this->createForm(TimesheetApiEditForm::class, $timesheet, [...]);
    $form->setData($timesheet);
    $form->submit($request->request->all(), false);
    if (false === $form->isValid()) { ... }
    $this->service->saveTimesheet($timesheet);
    ...
}

src/Voter/TimesheetVoter.php:134-142:

if ($subject->getUser()?->getId() === $user->getId()) {
    return $this->permissionManager->hasRolePermission($user, $permission . '_own_timesheet');
}

if (!$this->permissionManager->checkTeamAccessTimesheet($subject, $user)) {
    return false;
}

For an own-timesheet, only edit_own_timesheet is required. The voter does not look at the new project being submitted; it only validates the existing record's ownership.

Form replays user-controlled project ID into the access query

src/Form/TimesheetEditForm.php:60-71:

$isNew = true;
if (isset($options['data']) && $options['data'] instanceof Timesheet) {
    ...
    if (null !== $entry->getId()) {
        $isNew = false;
    }
    ...
}
$this->addProject($builder, $isNew, $project, $customer);

src/Form/FormTrait.php:59-100:

$builder->addEventListener(
    FormEvents::PRE_SUBMIT,
    function (FormEvent $event) use ($builder, $project, $customer, $isNew, $options): void {
        $data = $event->getData();
        $customer = \array_key_exists('customer', $data) && $data['customer'] !== '' ? $data['customer'] : null;
        $project = \array_key_exists('project', $data) && $data['project'] !== '' ? $data['project'] : $project;

        $event->getForm()->add('project', ProjectType::class, array_merge($options, [
            'group_by' => null,
            'query_builder' => function (ProjectRepository $repo) use ($builder, $project, $customer, $isNew) {
                $project = \is_string($project) ? (int) $project : $project;
                ...
                if ($isNew && \is_int($project)) {
                    $project = $repo->find($project);
                    if ($project !== null) {
                        if (!$project->getCustomer()->isVisible()) { ... $project = null; }
                        elseif (!$project->isVisible())            { $project = null; }
                    }
                }
                ...
                $query = new ProjectFormTypeQuery($project, $customer);
                $query->setUser($builder->getOption('user'));
                $query->setWithCustomer(true);
                return $repo->getQueryBuilderForFormType($query);
            },
        ]));
    }
);

Two problems compound:

  1. The visibility re-check on line 73 is gated on $isNew. For PATCH, $isNew = false, so the closure passes the attacker-supplied ID straight through.
  2. Even when $isNew = true (POST), the re-check only validates isVisible() — it does not validate team membership.

The query-builder unconditionally accepts the submitted ID

src/Repository/ProjectRepository.php:150-208:

public function getQueryBuilderForFormType(ProjectFormTypeQuery $query): QueryBuilder
{
    ...
    $mainQuery = $qb->expr()->andX();
    $mainQuery->add($qb->expr()->eq('p.visible', ':visible'));
    $mainQuery->add($qb->expr()->eq('c.visible', ':customer_visible'));
    if (!$query->isIgnoreDate()) { ... }
    if ($query->hasCustomers()) { ... }

    $permissions = $this->getPermissionCriteria($qb, $query->getUser(), $query->getTeams());
    if ($permissions->count() > 0) {
        $mainQuery->add($permissions);
    }

    $outerQuery = $qb->expr()->orX();
    if ($query->hasProjects()) {
        $outerQuery->add($qb->expr()->in('p.id', ':project'));     // <-- unconditional
        $qb->setParameter('project', $query->getProjects());
    }
    ...
    $outerQuery->add($mainQuery);
    $qb->andWhere($outerQuery);
    return $qb;
}

The final WHERE clause is roughly:

WHERE (p.id IN (:project)) OR (p.visible AND c.visible AND <date> AND <team-ACL>)

Because :project is the submitted ID itself, the first branch matches unconditionally, completely bypassing the team-ACL applied by getPermissionCriteria. Symfony's EntityType happily resolves the foreign Project entity, the form passes validation, and the timesheet is persisted with the new project_id.

No downstream validation closes the gap

  • TimesheetService::saveTimesheet → updateTimesheet (src/Timesheet/TimesheetService.php:154-177) is explicitly documented as not validating.
  • TimesheetBasicValidator only validates begin/end and project/activity coherence.
  • TimesheetDeactivatedValidator::validateActivityAndProject (src/Validator/Constraints/TimesheetDeactivatedValidator.php:36-42) returns early for non-running existing timesheets.
  • No validator anywhere in the timesheet pipeline checks that the project's team membership intersects the acting user's teams.

A PoC was provided, but removed for security reasons.

Impact

  • Integrity: any authenticated user can attribute their own tracked time to any project ID in the database — including projects belonging to teams/customers they cannot see. This pollutes per-project budgets, billing exports and reports for other teams. There is no in-app warning that records belonging to outsiders have been added.
  • Confidentiality: by reading the timesheet back via ?full=true, the attacker obtains serialized project and customer details (name, currency, start/end dates, customer hierarchy) which would normally be filtered by the team ACL.
  • Privilege model: the edit_own_timesheet permission is part of the default ROLE_USER, so the bypass is reachable by every regular user without any administrator action.

The blast radius is bounded by what an attacker can persist (their own timesheet rows) and what the ?full=true serializer exposes — there is no direct ability to modify other teams' existing data.

Solution

  • The FormTrait was updated to only pass the project forward for new timesheets
  • A new TimesheetTeamAccessValidatorwas added, which checks if project or activity were changed. If that is the case, the team access permission is checked first

Find out more at https://www.kimai.org/en/security/ghsa-vrr2-g9gh-c3jc

Severity
5.3 (Medium)
CVSS:4.0/AV:N/AC:L/AT:N/PR:L/UI:N/VC:L/VI:L/VA:N/SC:N/SI:N/SA:N
Show details on source website
To clipboard 

{
  "affected": [
    {
      "database_specific": {
        "last_known_affected_version_range": "\u003c= 2.56.0"
      },
      "package": {
        "ecosystem": "Packagist",
        "name": "kimai/kimai"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "0"
            },
            {
              "fixed": "2.57.0"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    }
  ],
  "aliases": [
    "CVE-2026-52820"
  ],
  "database_specific": {
    "cwe_ids": [
      "CWE-639"
    ],
    "github_reviewed": true,
    "github_reviewed_at": "2026-07-13T23:55:35Z",
    "nvd_published_at": null,
    "severity": "MODERATE"
  },
  "details": "## Summary\n\nThe Timesheet API `PATCH /api/timesheets/{id}` and `POST /api/timesheets` endpoints accept a user-supplied `project` ID and resolve it through a Symfony `EntityType` whose `query_builder` allows the submitted ID to satisfy the access predicate via an unconditional OR branch. As a result, any authenticated user can re-assign their own timesheet to any project in the database \u2014 including projects that belong to teams or customers they have no membership in and cannot otherwise see. The user can then read serialized project/customer details via `GET /api/timesheets/{id}?full=true`, leaking metadata (name, currency, customer hierarchy) that would otherwise be filtered out by the team ACL.\n\n## Details\n\n### Entry point \u2014 only ownership is checked in `src/API/TimesheetController.php:317-355`\n\n```php\n#[IsGranted(\u0027edit\u0027, \u0027timesheet\u0027)]\n#[Route(methods: [\u0027PATCH\u0027], path: \u0027/{id}\u0027, name: \u0027patch_timesheet\u0027, requirements: [\u0027id\u0027 =\u003e \u0027\\d+\u0027])]\npublic function patchAction(Request $request, Timesheet $timesheet): Response\n{\n    ...\n    $form = $this-\u003ecreateForm(TimesheetApiEditForm::class, $timesheet, [...]);\n    $form-\u003esetData($timesheet);\n    $form-\u003esubmit($request-\u003erequest-\u003eall(), false);\n    if (false === $form-\u003eisValid()) { ... }\n    $this-\u003eservice-\u003esaveTimesheet($timesheet);\n    ...\n}\n```\n\n`src/Voter/TimesheetVoter.php:134-142`:\n\n```php\nif ($subject-\u003egetUser()?-\u003egetId() === $user-\u003egetId()) {\n    return $this-\u003epermissionManager-\u003ehasRolePermission($user, $permission . \u0027_own_timesheet\u0027);\n}\n\nif (!$this-\u003epermissionManager-\u003echeckTeamAccessTimesheet($subject, $user)) {\n    return false;\n}\n```\n\nFor an own-timesheet, only `edit_own_timesheet` is required. The voter does **not** look at the *new* project being submitted; it only validates the existing record\u0027s ownership.\n\n### Form replays user-controlled project ID into the access query\n\n`src/Form/TimesheetEditForm.php:60-71`:\n\n```php\n$isNew = true;\nif (isset($options[\u0027data\u0027]) \u0026\u0026 $options[\u0027data\u0027] instanceof Timesheet) {\n    ...\n    if (null !== $entry-\u003egetId()) {\n        $isNew = false;\n    }\n    ...\n}\n$this-\u003eaddProject($builder, $isNew, $project, $customer);\n```\n\n`src/Form/FormTrait.php:59-100`:\n\n```php\n$builder-\u003eaddEventListener(\n    FormEvents::PRE_SUBMIT,\n    function (FormEvent $event) use ($builder, $project, $customer, $isNew, $options): void {\n        $data = $event-\u003egetData();\n        $customer = \\array_key_exists(\u0027customer\u0027, $data) \u0026\u0026 $data[\u0027customer\u0027] !== \u0027\u0027 ? $data[\u0027customer\u0027] : null;\n        $project = \\array_key_exists(\u0027project\u0027, $data) \u0026\u0026 $data[\u0027project\u0027] !== \u0027\u0027 ? $data[\u0027project\u0027] : $project;\n\n        $event-\u003egetForm()-\u003eadd(\u0027project\u0027, ProjectType::class, array_merge($options, [\n            \u0027group_by\u0027 =\u003e null,\n            \u0027query_builder\u0027 =\u003e function (ProjectRepository $repo) use ($builder, $project, $customer, $isNew) {\n                $project = \\is_string($project) ? (int) $project : $project;\n                ...\n                if ($isNew \u0026\u0026 \\is_int($project)) {\n                    $project = $repo-\u003efind($project);\n                    if ($project !== null) {\n                        if (!$project-\u003egetCustomer()-\u003eisVisible()) { ... $project = null; }\n                        elseif (!$project-\u003eisVisible())            { $project = null; }\n                    }\n                }\n                ...\n                $query = new ProjectFormTypeQuery($project, $customer);\n                $query-\u003esetUser($builder-\u003egetOption(\u0027user\u0027));\n                $query-\u003esetWithCustomer(true);\n                return $repo-\u003egetQueryBuilderForFormType($query);\n            },\n        ]));\n    }\n);\n```\n\nTwo problems compound:\n\n1. The visibility re-check on line 73 is gated on `$isNew`. For PATCH, `$isNew = false`, so the closure passes the attacker-supplied ID straight through.\n2. Even when `$isNew = true` (POST), the re-check only validates `isVisible()` \u2014 it does not validate team membership.\n\n### The query-builder unconditionally accepts the submitted ID\n\n`src/Repository/ProjectRepository.php:150-208`:\n\n```php\npublic function getQueryBuilderForFormType(ProjectFormTypeQuery $query): QueryBuilder\n{\n    ...\n    $mainQuery = $qb-\u003eexpr()-\u003eandX();\n    $mainQuery-\u003eadd($qb-\u003eexpr()-\u003eeq(\u0027p.visible\u0027, \u0027:visible\u0027));\n    $mainQuery-\u003eadd($qb-\u003eexpr()-\u003eeq(\u0027c.visible\u0027, \u0027:customer_visible\u0027));\n    if (!$query-\u003eisIgnoreDate()) { ... }\n    if ($query-\u003ehasCustomers()) { ... }\n\n    $permissions = $this-\u003egetPermissionCriteria($qb, $query-\u003egetUser(), $query-\u003egetTeams());\n    if ($permissions-\u003ecount() \u003e 0) {\n        $mainQuery-\u003eadd($permissions);\n    }\n\n    $outerQuery = $qb-\u003eexpr()-\u003eorX();\n    if ($query-\u003ehasProjects()) {\n        $outerQuery-\u003eadd($qb-\u003eexpr()-\u003ein(\u0027p.id\u0027, \u0027:project\u0027));     // \u003c-- unconditional\n        $qb-\u003esetParameter(\u0027project\u0027, $query-\u003egetProjects());\n    }\n    ...\n    $outerQuery-\u003eadd($mainQuery);\n    $qb-\u003eandWhere($outerQuery);\n    return $qb;\n}\n```\n\nThe final WHERE clause is roughly:\n\n```\nWHERE (p.id IN (:project)) OR (p.visible AND c.visible AND \u003cdate\u003e AND \u003cteam-ACL\u003e)\n```\n\nBecause `:project` is the submitted ID itself, the first branch matches unconditionally, completely bypassing the team-ACL applied by `getPermissionCriteria`. Symfony\u0027s `EntityType` happily resolves the foreign `Project` entity, the form passes validation, and the timesheet is persisted with the new `project_id`.\n\n### No downstream validation closes the gap\n\n- `TimesheetService::saveTimesheet` \u2192 `updateTimesheet` (`src/Timesheet/TimesheetService.php:154-177`) is explicitly documented as *not* validating.\n- `TimesheetBasicValidator` only validates begin/end and project/activity coherence.\n- `TimesheetDeactivatedValidator::validateActivityAndProject` (`src/Validator/Constraints/TimesheetDeactivatedValidator.php:36-42`) returns early for non-running existing timesheets.\n- No validator anywhere in the timesheet pipeline checks that the project\u0027s team membership intersects the acting user\u0027s teams.\n\n*A PoC was provided, but removed for security reasons.*\n\n## Impact\n\n- **Integrity:** any authenticated user can attribute their own tracked time to any project ID in the database \u2014 including projects belonging to teams/customers they cannot see. This pollutes per-project budgets, billing exports and reports for other teams. There is no in-app warning that records belonging to outsiders have been added.\n- **Confidentiality:** by reading the timesheet back via `?full=true`, the attacker obtains serialized project and customer details (name, currency, start/end dates, customer hierarchy) which would normally be filtered by the team ACL.\n- **Privilege model:** the `edit_own_timesheet` permission is part of the default ROLE_USER, so the bypass is reachable by every regular user without any administrator action.\n\nThe blast radius is bounded by what an attacker can persist (their own timesheet rows) and what the `?full=true` serializer exposes \u2014 there is no direct ability to modify other teams\u0027 existing data.\n\n## Solution\n\n- The FormTrait was updated to only pass the project forward for new timesheets\n- A new `TimesheetTeamAccessValidator`was added, which checks if `project` or `activity` were changed. If that is the case, the team access permission is checked first\n\nFind out more at [https://www.kimai.org/en/security/ghsa-vrr2-g9gh-c3jc](https://www.kimai.org/en/security/ghsa-vrr2-g9gh-c3jc)",
  "id": "GHSA-vrr2-g9gh-c3jc",
  "modified": "2026-07-13T23:55:35Z",
  "published": "2026-07-13T23:55:35Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://github.com/kimai/kimai/security/advisories/GHSA-vrr2-g9gh-c3jc"
    },
    {
      "type": "PACKAGE",
      "url": "https://github.com/kimai/kimai"
    }
  ],
  "schema_version": "1.4.0",
  "severity": [
    {
      "score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:L/UI:N/VC:L/VI:L/VA:N/SC:N/SI:N/SA:N",
      "type": "CVSS_V4"
    }
  ],
  "summary": "Kimai: Timesheet PATCH/POST allows assigning to project outside user\u0027s team via query_builder OR-bypass"
}



Log in or create an account to share your comment.




Tags
Taxonomy of the tags.





Forecast uses a logistic model when the trend is rising, or an exponential decay model when the trend is falling. Fitted via linearized least squares.

Sightings

Author Source Type Date Other

Nomenclature

  • Seen: The vulnerability was mentioned, discussed, or observed by the user.
  • Confirmed: The vulnerability has been validated from an analyst's perspective.
  • Published Proof of Concept: A public proof of concept is available for this vulnerability.
  • Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
  • Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
  • Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
  • Not confirmed: The user expressed doubt about the validity of the vulnerability.
  • Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.


Detection rules are retrieved from Rulezet.