GHSA-MR9H-45P9-FG8H

Vulnerability from github – Published: 2026-07-02 19:23 – Updated: 2026-07-02 19:23
VLAI
Summary
Froxlor: Authenticated customers can read other customers' allowed sender aliases
Details

Summary

An authenticated customer can read other customers' allowed sender aliases from Froxlor's sender-delete confirmation page when mail.enable_allow_sender is enabled. customer_email.php loads allowed_sender by global auto-increment senderid alone, so a customer can enumerate foreign sender alias IDs and make Froxlor disclose those values in the confirmation dialog for the attacker's own mailbox.

Details

The vulnerable read lives in customer_email.php:

$senderid = Request::any('senderid', 0);
...
$sel_stmt = Database::prepare("SELECT `allowed_sender` FROM `" . TABLE_MAIL_SENDER_ALIAS . "` WHERE `id` = :sid");
$sender_data = Database::pexecute_first($sel_stmt, ['sid' => $senderid]);

HTML::askYesNo('email_reallydelete_sender', $filename, [
    'id' => $id,
    'senderid' => $senderid,
    'page' => $page,
    'domainid' => $email_domainid,
    'action' => $action
], $idna_convert->decode($result['email_full']) . ' -> ' . $sender_data['allowed_sender']);

The query does not scope senderid to the current customer or to the mailbox being edited. mail_sender_aliases.id is a global AUTO_INCREMENT primary key:

CREATE TABLE `mail_sender_aliases` (
  `id` int(11) NOT NULL auto_increment,
  `email` varchar(255) NOT NULL,
  `allowed_sender` varchar(255) NOT NULL,
  PRIMARY KEY  (`id`)
)

That makes sender alias IDs enumerable. A customer who owns any mailbox with sender-alias management enabled can request the delete-confirmation page for their own mailbox while supplying foreign senderid values. Froxlor then renders the foreign allowed_sender string in the confirmation prompt.

Proof of Concept

Verified against Froxlor 2.3.6 on a fresh test install with mail.enable_allow_sender=1.

  1. Create two customers, victim and attacker.
  2. Give each customer one mailbox.
  3. Add a sender alias for the victim mailbox:
info@victim.local -> leaked-sender@example.org
  1. Log in as the attacker and request the delete-confirmation page for the attacker's own mailbox while supplying the victim's sender alias ID:
/customer_email.php?page=senders&domainid=4&action=delete&id=2&senderid=1

Observed in the live test instance:

Do you really want to delete the allowed sender info@attacker.local -> leaked-sender@example.org?

The attacker-controlled mailbox identifier stayed info@attacker.local, but the disclosed sender alias text came from the victim-owned row identified by senderid=1.

Impact

An authenticated customer can enumerate global sender alias IDs and read other customers' allowed sender values. This is a cross-tenant information disclosure. It does not let the attacker delete the foreign alias because the delete action revalidates ownership.

Recommended Fix

Scope the sender alias lookup to the current customer and mailbox before rendering the confirmation page. The confirmation flow should fetch the sender alias through the same ownership checks used by EmailSender::delete().


Found by aisafe.io

Show details on source website

{
  "affected": [
    {
      "database_specific": {
        "last_known_affected_version_range": "\u003c= 2.3.6"
      },
      "package": {
        "ecosystem": "Packagist",
        "name": "froxlor/froxlor"
      },
      "ranges": [
        {
          "events": [
            {
              "introduced": "0"
            },
            {
              "fixed": "2.3.7"
            }
          ],
          "type": "ECOSYSTEM"
        }
      ]
    }
  ],
  "aliases": [],
  "database_specific": {
    "cwe_ids": [
      "CWE-200"
    ],
    "github_reviewed": true,
    "github_reviewed_at": "2026-07-02T19:23:31Z",
    "nvd_published_at": null,
    "severity": "MODERATE"
  },
  "details": "## Summary\n\nAn authenticated customer can read other customers\u0027 allowed sender aliases from Froxlor\u0027s sender-delete confirmation page when `mail.enable_allow_sender` is enabled. `customer_email.php` loads `allowed_sender` by global auto-increment `senderid` alone, so a customer can enumerate foreign sender alias IDs and make Froxlor disclose those values in the confirmation dialog for the attacker\u0027s own mailbox.\n\n## Details\n\nThe vulnerable read lives in `customer_email.php`:\n\n```php\n$senderid = Request::any(\u0027senderid\u0027, 0);\n...\n$sel_stmt = Database::prepare(\"SELECT `allowed_sender` FROM `\" . TABLE_MAIL_SENDER_ALIAS . \"` WHERE `id` = :sid\");\n$sender_data = Database::pexecute_first($sel_stmt, [\u0027sid\u0027 =\u003e $senderid]);\n\nHTML::askYesNo(\u0027email_reallydelete_sender\u0027, $filename, [\n    \u0027id\u0027 =\u003e $id,\n    \u0027senderid\u0027 =\u003e $senderid,\n    \u0027page\u0027 =\u003e $page,\n    \u0027domainid\u0027 =\u003e $email_domainid,\n    \u0027action\u0027 =\u003e $action\n], $idna_convert-\u003edecode($result[\u0027email_full\u0027]) . \u0027 -\u003e \u0027 . $sender_data[\u0027allowed_sender\u0027]);\n```\n\nThe query does not scope `senderid` to the current customer or to the mailbox being edited. `mail_sender_aliases.id` is a global `AUTO_INCREMENT` primary key:\n\n```sql\nCREATE TABLE `mail_sender_aliases` (\n  `id` int(11) NOT NULL auto_increment,\n  `email` varchar(255) NOT NULL,\n  `allowed_sender` varchar(255) NOT NULL,\n  PRIMARY KEY  (`id`)\n)\n```\n\nThat makes sender alias IDs enumerable. A customer who owns any mailbox with sender-alias management enabled can request the delete-confirmation page for their own mailbox while supplying foreign `senderid` values. Froxlor then renders the foreign `allowed_sender` string in the confirmation prompt.\n\n## Proof of Concept\n\nVerified against Froxlor `2.3.6` on a fresh test install with `mail.enable_allow_sender=1`.\n\n1. Create two customers, `victim` and `attacker`.\n2. Give each customer one mailbox.\n3. Add a sender alias for the victim mailbox:\n\n```text\ninfo@victim.local -\u003e leaked-sender@example.org\n```\n\n4. Log in as the attacker and request the delete-confirmation page for the attacker\u0027s own mailbox while supplying the victim\u0027s sender alias ID:\n\n```text\n/customer_email.php?page=senders\u0026domainid=4\u0026action=delete\u0026id=2\u0026senderid=1\n```\n\nObserved in the live test instance:\n\n```text\nDo you really want to delete the allowed sender info@attacker.local -\u003e leaked-sender@example.org?\n```\n\nThe attacker-controlled mailbox identifier stayed `info@attacker.local`, but the disclosed sender alias text came from the victim-owned row identified by `senderid=1`.\n\n## Impact\n\nAn authenticated customer can enumerate global sender alias IDs and read other customers\u0027 allowed sender values. This is a cross-tenant information disclosure. It does not let the attacker delete the foreign alias because the delete action revalidates ownership.\n\n## Recommended Fix\n\nScope the sender alias lookup to the current customer and mailbox before rendering the confirmation page. The confirmation flow should fetch the sender alias through the same ownership checks used by `EmailSender::delete()`.\n\n---\n*Found by [aisafe.io](https://aisafe.io)*",
  "id": "GHSA-mr9h-45p9-fg8h",
  "modified": "2026-07-02T19:23:31Z",
  "published": "2026-07-02T19:23:31Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://github.com/froxlor/froxlor/security/advisories/GHSA-mr9h-45p9-fg8h"
    },
    {
      "type": "PACKAGE",
      "url": "https://github.com/froxlor/froxlor"
    }
  ],
  "schema_version": "1.4.0",
  "severity": [
    {
      "score": "CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:L/I:N/A:N",
      "type": "CVSS_V3"
    }
  ],
  "summary": "Froxlor: Authenticated customers can read other customers\u0027 allowed sender aliases"
}



Log in or create an account to share your comment.




Tags
Taxonomy of the tags.


Loading…

Loading…

Loading…

Forecast uses a logistic model when the trend is rising, or an exponential decay model when the trend is falling. Fitted via linearized least squares.

Sightings

Author Source Type Date Other

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…

Loading…