PT-2026-117121
Publicado
2026-10-09
·
Atualizado
2026-10-09
Descrição por
CVSS 3.1
5.4
MédiaMétricasAV:N/AC:L/PR:L/UI:N/S:U/C:N/I:L/A:L
Summary
In Vikunja v2.6.0 the task position recalculation that runs when a saved filter view is created fails open when the requesting user has no accessible projects. Instead of aborting, the search that feeds the recalculation loses its project scope entirely and returns every task in the instance. The server then writes one
task positions row per task per view (four views per filter) into views owned by the requesting user, including rows that reference tasks of other tenants the user can never see. Any authenticated user can trigger this. The impact is a cross-tenant integrity violation, a single-request write amplification primitive that scales with instance size, and a violation of the invariant stated in the fix for GHSA-w39f-h553-h2mx that position writes are scoped by project read access.Details
Vikunja computes kanban and list ordering in a shared
task positions table keyed by (task id, project view id). When a saved filter is created, SavedFilter.Create (pkg/models/saved filters.go:130) calls CreateDefaultViewsForProject (pkg/models/project view.go:816) which, for every default view it creates, calls RecalculateTaskPositions with addExistingTasksToView = true (pkg/models/project view.go:447).RecalculateTaskPositions (pkg/models/task position.go:312) builds a TaskCollection and, for filter views (view.ProjectID < -1), resets the project scope to zero and injects the saved filter's filter string. The scope of the subsequent search is derived from getRelevantProjectsFromCollection (pkg/models/task collection.go:188), which for ProjectID == 0 returns exactly the projects the requesting user can access.Inside
dbTaskSearcher.Search (pkg/models/task search.go), the WHERE clause is assembled at line 632:go
cond := builder.And(builder.Or(projectIDCond, favoritesCond), where, filterCond)projectIDCond is only set when len(opts.projectIDs) > 0 and favoritesCond only when the caller opted into the favorites arm, which RecalculateTaskPositions never does. With zero accessible projects, an empty filter string (which produces no filter condition in getTaskFiltersFromFilterString) and no search string, all four children of the builder.And are invalid. go-xorm's condition builders silently drop invalid children, so the query becomes SELECT ... FROM tasks with no WHERE clause at all.The result is written back unconditionally: the function deletes the view's position rows and bulk-inserts one row per returned task per view. Because the filter creation path runs this for four default views, one request writes
4 x total task count rows, every one of them referencing tasks the requesting user has no relationship with, including tasks in other users' private projects.Reaching the zero-project precondition does not require any bug beyond this one.
UpdateUserGeneralSettings (pkg/models/user settings.go:105) assigns default project id from the request body without validating that the referenced project exists or belongs to the caller. A user can therefore point their default project at any project id, which unblocks deleting their own Inbox (deletion is refused only while the Inbox is the default project, error 3012 in pkg/models/error.go:482). After the Inbox is gone, getRawProjectsForUser returns an empty list.Two neighboring controls confirm the intended invariant exists elsewhere and is enforced in sibling paths. The event-driven filter update path gates candidate filters on the filter owner's access to each task's project (
matchTasksToViewsOfFilter, introduced by commit b1ad4063e7), and the fix for GHSA-w39f-h553-h2mx validates position target views. The recalculation path on filter creation has no equivalent gate and contradicts the advisory statement that recalculation "aborts on its own project read-access check".PoC
The attached archive contains
deployment/deploy.sh (local Vikunja v2.6.0 on loopback with SQLite) and poc/poc.sh, which performs the full sequence:- Victim registers, creates a private project and a task.
- Attacker registers, sets
default project idto the victim's project viaPUT /api/v2/user/settings/general, deletes their own Inbox, and now has zero accessible projects. - Control:
GET /api/v2/projects/{victim}returns 403. - Attacker creates a saved filter with
{"filter":"","filter include nulls":true}. - Verification: direct SQLite inspection shows
task positionsrows referencing the victim's task in all four of the attacker's filter views. - Control:
GET /api/v2/projects/{filter pseudo project}/tasksreturns 0 items, so the written rows are not readable back through collection endpoints.
Result: 10 checks passed on two consecutive runs, each from a wiped database.
Impact
An authenticated user, even one with no project access at all, causes the server to persist cross-tenant references: rows pointing at arbitrary other users' tasks are written into views the attacker controls. Task content is not exposed through the documented read paths (verified), so this is an integrity and availability issue rather than a direct disclosure:
- Cross-tenant integrity violation. The
task positionsstate of the attacker's views now encodes other tenants' objects, against the access model the rest of the code enforces. - Write amplification and denial of service. One cheap request performs a full-table scan plus
4 x Ninserts, where N is the number of tasks in the instance. On an instance with 100,000 tasks that is 400,000 rows per request, and the request can be repeated without bound. CPU, IO and storage costs scale with instance size while attacker cost stays constant.
Explorabilidade
Correção
Impactos
Fornecedores
Produtos afetados
Identificadores relacionados
Encontrou algum problema na descrição? Tem algo a acrescentar? Fique à vontade para nos escrever 👾