Am I affected?

type your ash_sql version to check

Description

Incorrect Authorization vulnerability in ash-project ash_sql allows a caller in a schema-based multitenant application to receive aggregate values computed from another tenant's rows.

When an aggregate is computed over a distinct query, AshSql.AggregateQuery.add_single_aggs/5 rebuilds the outer query from query.from.source alone, which is only the {table, schema} tuple and does not carry query.prefix or query.from.prefix. For strategy(:context) multitenancy those hold the tenant schema, so the rebuilt outer query reads the repo's default schema while the inner correlated subquery still reads the tenant schema, and the two are joined only on primary key. The aggregate, and any relationship join added off the prefix-less binding, is then computed against the wrong tenant's rows. The neighbouring limit and exists branches instead wrap the query with subquery/1, which preserves the prefix.

This issue affects ash_sql: from 0.1.0 before 0.7.1.

Weaknesses & attack patterns

Weakness

CWE-863 · Incorrect Authorization in catalog → MITRE ↗

Attack patterns

CAPEC-1 · Accessing Functionality Not Properly Constrained by ACLs MITRE ↗

Affected — Hex / ash_sql Hex.pm ↗ Repository ↗

0.1.0 < 0.7.1 affected
every other version: unaffected
cpe cpe:2.3:a:ash-project:ash_sql:*:*:*:*:*:*:*:*
modules · source files · routines
modules 'Elixir.AshSql.AggregateQuery'
source files lib/aggregate_query.ex
routines 'Elixir.AshSql.AggregateQuery':add_single_aggs/5

Affected — GitHub / ash-project/ash_sql Repository ↗

dd092ed < 3d95478 affected
every other version: unaffected
cpe cpe:2.3:a:ash-project:ash_sql:*:*:*:*:*:*:*:*
modules · source files · routines
modules 'Elixir.AshSql.AggregateQuery'
source files lib/aggregate_query.ex
routines 'Elixir.AshSql.AggregateQuery':add_single_aggs/5

Configurations

An application must use AshPostgres schema-based (strategy(:context)) multitenancy and compute an aggregate over a distinct query (for example an aggregate whose query filters across a to-many relationship).

References

Credits

Peter Ullrich Finder Reporter
Zach Daniel / Ash Project Remediation developer
Jonatan Männchen / EEF Coordinator

CVSS breakdown

CVSS:4.0/AV:L/AC:L/AT:P/PR:N/UI:N/VC:L/VI:N/VA:N/SC:N/SI:N/SA:N
« All CVEs