CVE-2026-33318

Summary

Summary Any authenticated user (including BASIC role) can escalate to ADMIN on servers migrated from password authentication to OpenID Connect. Three weaknesses combine: POST /account/change-password has no authorization check, allowing any session to overwrite the password hash; the inactive password auth row is never removed on migration; and the login endpoint accepts a client-supplied loginMethod that bypasses the server's active auth configuration. Together these allow an attacker to set a known password and authenticate as the anonymous admin account created during the multiuser migration. Details packages/sync-server/src/app-account.js:120-132, the /account/change-password route validates only that a session exists. No admin role check is performed packages/sync-server/src/accounts/password.js:113-125, changePassword() updates the hash with no current-password confirmation: packages/sync-server/src/accounts/password.js:56-62, loginWithPassword() always authenticates as the user with username = '', which is created by the multiuser migration with role = 'ADMIN': packages/sync-server/src/account-db.js:56-63, a client can force the password login method regardless of server configuration by sending loginMethod in the request body: When a server is migrated from password → OpenID via enableOpenID(), the password row is set to active = 0 but never deleted, leaving it available for exploitation. PoC Prerequisites: Server originally bootstrapped with password auth, then switched to OpenID. Default allowedLoginMethods configuration (includes password). Attacker has any valid OpenID session token (any role). The returned token belongs to the username = '' admin account (created by 1719409568000-multiuser.js). Verify admin access: Impact Privilege escalation, any authenticated user can gain full ADMIN access on affected deployments. An ADMIN can manage all users, access all budget files regardless of ownership, modify file access controls, and change server configuration. Affected deployments: multi-user servers running OpenID Connect that were previously configured with password authentication. Servers bootstrapped exclusively with OpenID from initial setup are not affected (no password row exists in the auth table). Recommendations Restrict POST /account/change-password to password-authenticated sessions only (packages/sync-server/src/app-account.js) The endpoint should reject requests from sessions that were authenticated via OpenID. The active auth method can be checked against the session's auth_method field or by querying the auth table for the currently active method. If the server is running in OpenID mode, this endpoint should return 403 Forbidden. Require current-password confirmation before accepting a new password (packages/sync-server/src/accounts/password.js) changePassword() should accept the current password as a parameter and verify it against the stored hash before applying the update. This prevents any session (even a legitimate password session) from silently overwriting the credential without proving possession of the existing one. Enforce active status and remove client control over login method selection (packages/sync-server/src/account-db.js, getLoginMethod()) Two issues exist in getLoginMethod(): (a) a client can supply loginMethod in the request body to select any method listed in allowedLoginMethods, regardless of whether it is currently active on the server; (b) the function does not check the active column, so an administratively disabled method (e.g. password after migrating to OpenID) remains accessible. The fix is to determine the permitted method server-side from WHERE active = 1 in the auth table and ignore any client-supplied loginMethod override entirely. Servers intentionally running both methods simultaneously can be supported by allowing multiple active = 1 rows rather than relying on client input. Immediate mitigation for existing deployments (OpenID-only servers) Administrators who have fully migrated to OpenID and do not need password auth can remove the orphaned row: The three weaknesses form a single, sequential exploit chain, none produces privilege escalation on its own: Missing authorization on POST /change-password, allows overwriting a password hash, but only matters if there is an orphaned row to target. Orphaned password row persisting after migration, provides the target row, but is harmless without the ability to authenticate using it. Client-controlled loginMethod: "password", allows forcing password-based auth, but is useless without a known hash established by step 1. All three must be chained in sequence to achieve the impact. No single weakness independently results in privilege escalation, which under CVE CNA rule 4.1.2 means they should not each be treated as standalone vulnerabilities. The single root cause is the missing authorization check on /change-password; the other two are preconditions that make it exploitable. A single CVE reflecting that root cause is the appropriate representation, splitting them would falsely imply each carries independent risk.

Impact

Affected versions

Security releases

Remediation advice