GHSA-C7R6-VX3H-W5G2
Vulnerability from github – Published: 2026-09-08 20:40 – Updated: 2026-09-08 20:40Summary
Excel::store() resolved the destination path against the process working
directory rather than the configured filesystem disk. When that path resolved to
an existing file, the export was written straight to it with fopen(),
bypassing the disk entirely. An application that passes a user-controlled value
as the export path could therefore be made to overwrite an arbitrary existing
file that the PHP process can write to, with content the user controls.
Details
Maatwebsite\Excel\Files\Disk::copy() contained two paths:
if (realpath($destination)) {
$tempStream = fopen($destination, 'rb+');
$success = stream_copy_to_stream($readStream, $tempStream) !== false;
} else {
$success = $this->put($destination, $readStream);
}
$destination is the $filePath argument given to Excel::store(),
$export->store() or ->storeExcel(). realpath() resolves it against the
current working directory — public/ for a typical web request — not
against the disk root. On a hit, the write went directly to the filesystem and
never reached Flysystem, which would otherwise have rejected ../ traversal and
confined absolute paths to the disk root. The disk argument was effectively
ignored for those paths, including for remote disks such as S3.
Two consequences follow:
- the destination could be any existing file the PHP process can write, in or out of the disk root;
- the stream was opened
'rb+', which does not truncate, so a shorter export left trailing bytes of the previous file behind.
Because the file must already exist, the primitive is an overwrite rather
than an arbitrary file creation. Overwriting a PHP file that is reachable by the
web server (for example a front controller or a cached view) turns
attacker-controlled row content into code execution, since CSV and HTML writers
emit cell values verbatim. Passing an explicit writer type to store() bypasses
the extension-based type detection that would otherwise reject a .php target.
Exploitation requires the application to pass an unsanitized, user-controlled value as the export path. Applications that pass a fixed or server-derived path are not affected.
Impact
Arbitrary overwrite of existing files writable by the PHP process, with partially attacker-controlled content, leading to remote code execution where the overwritten file is executed by the web server.
Patches
Fixed in 3.1.70. Disk::copy() now always writes through the configured
filesystem disk, so Flysystem enforces the disk root for every export.
Note the behaviour change: passing an absolute path to store() previously
wrote to that path once the file existed. Paths now always resolve relative to
the disk root. Applications that relied on that should configure a disk rooted
at the target location.
Workarounds
For anyone unable to upgrade, validate the path before passing it to store() —
reject absolute paths and any .. segment, or derive the filename server-side
and never build it from request input:
$name = basename($request->input('filename')); // strips any directory part
Excel::store($export, 'exports/' . $name, 'local');
Credit
Reported responsibly by @seck19 via the contact address in SECURITY.md.
{
"affected": [
{
"package": {
"ecosystem": "Packagist",
"name": "maatwebsite/excel"
},
"ranges": [
{
"events": [
{
"introduced": "3.1.8"
},
{
"fixed": "3.1.70"
}
],
"type": "ECOSYSTEM"
}
]
}
],
"aliases": [
"CVE-2026-84374"
],
"database_specific": {
"cwe_ids": [
"CWE-22",
"CWE-73"
],
"github_reviewed": true,
"github_reviewed_at": "2026-09-08T20:40:47Z",
"nvd_published_at": "2026-09-01T22:17:19Z",
"severity": "HIGH"
},
"details": "### Summary\n\n`Excel::store()` resolved the destination path against the process working\ndirectory rather than the configured filesystem disk. When that path resolved to\nan existing file, the export was written straight to it with `fopen()`,\nbypassing the disk entirely. An application that passes a user-controlled value\nas the export path could therefore be made to overwrite an arbitrary existing\nfile that the PHP process can write to, with content the user controls.\n\n### Details\n\n`Maatwebsite\\Excel\\Files\\Disk::copy()` contained two paths:\n\n```php\nif (realpath($destination)) {\n $tempStream = fopen($destination, \u0027rb+\u0027);\n $success = stream_copy_to_stream($readStream, $tempStream) !== false;\n} else {\n $success = $this-\u003eput($destination, $readStream);\n}\n```\n\n`$destination` is the `$filePath` argument given to `Excel::store()`,\n`$export-\u003estore()` or `-\u003estoreExcel()`. `realpath()` resolves it against the\n**current working directory** \u2014 `public/` for a typical web request \u2014 not\nagainst the disk root. On a hit, the write went directly to the filesystem and\nnever reached Flysystem, which would otherwise have rejected `../` traversal and\nconfined absolute paths to the disk root. The disk argument was effectively\nignored for those paths, including for remote disks such as S3.\n\nTwo consequences follow:\n\n* the destination could be any existing file the PHP process can write, in or\n out of the disk root;\n* the stream was opened `\u0027rb+\u0027`, which does not truncate, so a shorter export\n left trailing bytes of the previous file behind.\n\nBecause the file must already exist, the primitive is an **overwrite** rather\nthan an arbitrary file creation. Overwriting a PHP file that is reachable by the\nweb server (for example a front controller or a cached view) turns\nattacker-controlled row content into code execution, since CSV and HTML writers\nemit cell values verbatim. Passing an explicit writer type to `store()` bypasses\nthe extension-based type detection that would otherwise reject a `.php` target.\n\nExploitation requires the **application** to pass an unsanitized, user-controlled\nvalue as the export path. Applications that pass a fixed or server-derived path\nare not affected.\n\n### Impact\n\nArbitrary overwrite of existing files writable by the PHP process, with\npartially attacker-controlled content, leading to remote code execution where\nthe overwritten file is executed by the web server.\n\n### Patches\n\nFixed in **3.1.70**. `Disk::copy()` now always writes through the configured\nfilesystem disk, so Flysystem enforces the disk root for every export.\n\nNote the behaviour change: passing an absolute path to `store()` previously\nwrote to that path once the file existed. Paths now always resolve relative to\nthe disk root. Applications that relied on that should configure a disk rooted\nat the target location.\n\n### Workarounds\n\nFor anyone unable to upgrade, validate the path before passing it to `store()` \u2014\nreject absolute paths and any `..` segment, or derive the filename server-side\nand never build it from request input:\n\n```php\n$name = basename($request-\u003einput(\u0027filename\u0027)); // strips any directory part\nExcel::store($export, \u0027exports/\u0027 . $name, \u0027local\u0027);\n```\n\n### Credit\n\nReported responsibly by @seck19 via the contact address in `SECURITY.md`.",
"id": "GHSA-c7r6-vx3h-w5g2",
"modified": "2026-09-08T20:40:47Z",
"published": "2026-09-08T20:40:47Z",
"references": [
{
"type": "WEB",
"url": "https://github.com/SpartnerNL/Laravel-Excel/security/advisories/GHSA-c7r6-vx3h-w5g2"
},
{
"type": "ADVISORY",
"url": "https://nvd.nist.gov/vuln/detail/CVE-2026-84374"
},
{
"type": "WEB",
"url": "https://github.com/SpartnerNL/Laravel-Excel/commit/b5cafdfcf7ec63924e83303763be8fcae340f70b"
},
{
"type": "PACKAGE",
"url": "https://github.com/SpartnerNL/Laravel-Excel"
},
{
"type": "WEB",
"url": "https://github.com/SpartnerNL/Laravel-Excel/releases/tag/3.1.70"
}
],
"schema_version": "1.4.0",
"severity": [
{
"score": "CVSS:3.1/AV:N/AC:H/PR:L/UI:N/S:U/C:H/I:H/A:H",
"type": "CVSS_V3"
}
],
"summary": "Laravel Excel writes exports outside the configured filesystem disk when given a caller-controlled path"
}
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.
The approach is described in our paper Mapping CVEs to MITRE ATT&CK Techniques: A Curated Gold-Set Classifier and the Limits of LLM-Assisted Label Expansion.