Poweradmin: API user-update endpoint leads to a non-admin reset any user's password and take over the superuser account

Where the API is missing the password permission check. lib/Domain/Service/UserManagementService.php, updateUser() (lines 393 to 413) writes the password whenever one is supplied, with no permission check at all (it only refuses when the target is an external-auth user):

if (!empty($userData['password'])) {
    $user = $this->userRepository->getUserById($userId);
    $authMethod = $user['auth_method'] ?? 'sql';
    $externalAuthMethods = ['oidc', 'saml', 'ldap'];
    if (in_array($authMethod, $externalAuthMethods, true)) {
        return [ 'success' => false, 'message' => '...', 'status' => 400 ];
    }
    // Hash password if allowed  <-- no permission check happens here
    $userData['password'] = password_hash($userData['password'], PASSWORD_DEFAULT);
}
$success = $this->userRepository->updateUser($userId, $userData);

Compare the web path lib/Domain/Model/UserManager.php, editUser(), which only writes the password column when the caller has the right permission:

$edit_own_perm = self::verifyPermission($this->db, 'user_edit_own');
$passwd_edit_others_perm = self::verifyPermission($this->db, 'user_passwd_edit_others');
if ($user_password != "" && ($edit_own_perm || $passwd_edit_others_perm)) {
    ...
    $query .= ", password = :password";
}

The call path. lib/Application/Controller/Api/V2/UsersController.php, updateUser() (lines 701 to 727):

if (!$this->apiPermissionService->canEditUser($currentUserId, $targetUserId)) {   // line 708
    ... 403 ...
}
$result = $this->userManagementService->updateUser($targetUserId, $input);         // line 727

So canEditUser returns true for any user_edit_others holder against any target, and updateUser then writes the password with no further check. The V1 controller (lib/Application/Controller/Api/V1/UsersController.php) has the same shape.

Note: perm_templ is separately protected (a PermissionTemplateAssignmentGuard rejects it unless the caller has user_edit_templ_perm), so an attacker cannot elevate their own template through the API. They do not need to. Resetting the administrator's password and logging in as the administrator achieves full control directly.

PoC

Tested on Poweradmin master (commit 7f28c3a97) and the code path is present in the 4.0.x, 4.1.x, 4.2.x and 4.3.x release branches.

Configuration required (both are common in real deployments, neither is the default):

• The REST API must be enabled (api.enabled = true in config/settings.php). The API is intended for automation such as the Terraform provider, so it is frequently turned on.
• A delegated "user manager" role must exist: a permission template that grants user_edit_others (and typically user_view_others) but not user_is_ueberuser and not user_passwd_edit_others. This is a normal way to let a helpdesk or team lead manage user accounts.

Setup (performed once by the administrator to create the delegated role and the attacker):

1. As admin, create a permission template named "UserMgr" of type "user" with the permissions "User is allowed to edit other users" and "User is allowed to see other users and their details" only.
2. Create a normal user, for example umgr, and assign it the UserMgr template.
3. As umgr, create an API key from the user's API keys page. Call it, for example, pwa_umgr_key. umgr is now the attacker: a non-admin user with a self-scoped API key.

Exploit request (Burp Suite Repeater). Send this single request. id 1 is the administrator account.

PUT /api/v2/users/1 HTTP/1.1
Host: TARGET_HOST
X-API-Key: pwa_umgr_key
Content-Type: application/json
Content-Length: 30

{"password":"NewAdminPass1!"}

Expected response (HTTP 200):

{"success":true,"data":{"user_id":1},"message":"User updated successfully"}

now we can login to admin user with a new password!

The page refuses with "You do not have permission to edit a superuser account." So the same user with the same permission is denied on the web but allowed on the API.

Record ids are the normal sequential user ids, and the administrator is usually id 1, so no guessing is required. The attacker can also change username, email, active and use_ldap on any account through the same request.

Impact

This is a broken access control and privilege escalation issue (CWE-285 Improper Authorization, CWE-266 Incorrect Privilege Assignment).

Who is impacted: any Poweradmin installation that has the REST API enabled and that has created at least one non-admin role holding user_edit_others (a delegated user-manager or helpdesk role). On such an installation, any holder of that role, using nothing more than their own API key, can reset the superuser's password and take over the administrator account. From there they control every zone, record, user and setting in Poweradmin, and, through the PowerDNS backend, the DNS data itself.

The severity is high because the outcome is full administrative takeover from a low-privilege authenticated position, over the network, with a single request and no user interaction. The reason it is not rated critical outright is the precondition that a delegated user_edit_others role exists and the API is enabled. The underlying defect is real regardless of configuration: the API does not enforce the superuser-edit protection or the user_passwd_edit_others permission that the web interface enforces, so the API grants more authority than the permission model intends.

Suggested fix: mirror the web guards in the API. In ApiPermissionService::canEditUser, reject a non-superuser editing a superuser target. In UserManagementService::updateUser, only write the password when the caller is editing their own account or holds user_passwd_edit_others. Apply the same to both V1 and V2 controllers.

Disclosure

Credit: Found by Saif Salah (https://saifsalah.github.io/) Found during a security review of Poweradmin. Happy to coordinate a fix timeline and CVE before any public write-up.