A user holding the delegated users:update permission can reset the global administrator's password and take over the admin account. UpdateUser sets the target user's PasswordHash from the request body for ANY target with no comparison of caller-vs-target privilege. Because role assignment and role CRUD are deliberately gated by RequireGlobalAdmin, users:update is intended to sit BELOW admin (profile management); this finding defeats that separation.
// root causeUpdateUser (handler backend/api/handlers/users.go:317-323) loads the target by ID and sets userModel.PasswordHash from the request body for any user, then userService.UpdateUser (userservice.go:174) simply tx.Saves — with NO actor-vs-target privilege comparison. The only IsGlobalAdmin guard (userservice.go:345) is a last-admin check inside DeleteUser, not the update path. Role assignment / role CRUD are RequireGlobalAdmin (backend/pkg/authz/permissions.go:27-33), proving users:update is intended below admin.
// impactComplete administrator account takeover (C:H/I:H/A:H) by any holder of the delegated users:update permission.
// proof of concept- Admin delegates a custom "User Manager" role with users:update.
- PUT /users/{adminId} with {"password":"attacker123"}.
- POST /auth/login {"username":"arcane","password":"attacker123"} → admin session.
- Action: admin delegates custom role with users:update. Guard: role creation is RequireGlobalAdmin. Bypass: intended delegation (holder is meant to manage profiles, not seize admin).
- Action: PUT /users/{adminId} {"password":"attacker123"}. Guard: RequirePermission(users:update) (users.go:154). Bypass: holder has it; no hierarchy check (users.go:317-323, userservice.go:174 unconditional Save).
- Action: POST /auth/login as admin → admin session.
- users.go:317-323 sets PasswordHash for any target; userservice.go:174-194 has no actor/target privilege comparison.
- Only IsGlobalAdmin hits are DeleteUser's last-admin guard and DTO flags (grep).
- users:update is org-level, held by no built-in non-admin role (permissions.go:24).
<= 2.5.0.
Reported by zx (Jace) — GitHub: @manus-use
CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:H/I:H/A:H
- Attack vector
- Network
- Attack complexity
- Low
- Privileges required
- Low
- User interaction
- None
- Scope
- Unchanged
- Confidentiality
- High
- Integrity
- High
- Availability
- High