GHSA-vvwv-h69m-wg6f
XXE in PHPSpreadsheet due to incomplete fix for previous encoding issue
Quick fix
GHSA-vvwv-h69m-wg6f — phpoffice/phpspreadsheet: upgrade to the fixed version with the command below.
composer require phpoffice/phpspreadsheet:^1.8.0Details
PHPOffice PhpSpreadsheet before 1.8.0 has an XXE issue. The XmlScanner decodes the sheet1.xml from an .xlsx to utf-8 if something else than UTF-8 is declared in the header. This was a security measurement to prevent CVE-2018-19277 but the fix is not sufficient. By double-encoding the the xml payload to utf-7 it is possible to bypass the check for the string ?<!ENTITY? and thus allowing for an xml external entity processing (XXE) attack.
Are you affected?
Enter the version of the package you're using.
Affected packages
0Fixed in: 1.8.0composer require phpoffice/phpspreadsheet:^1.8.00No fixed version published yet for phpoffice/phpexcel (composer). Pin to a known-safe version or switch to an alternative.
References
- https://nvd.nist.gov/vuln/detail/CVE-2019-12331[ADVISORY]
- https://github.com/PHPOffice/PhpSpreadsheet/pull/1041[WEB]
- https://github.com/PHPOffice/PhpSpreadsheet/commit/0e6238c69e863b58aeece61e48ea032696c6dccd[WEB]
- https://github.com/FriendsOfPHP/security-advisories/blob/master/phpoffice/phpspreadsheet/CVE-2019-12331.yaml[WEB]
- https://github.com/PHPOffice/PhpSpreadsheet[PACKAGE]
- https://github.com/PHPOffice/PhpSpreadsheet/blob/master/CHANGELOG.md#180---2019-07-01[WEB]
- https://github.com/PHPOffice/PhpSpreadsheet/releases/tag/1.8.0[WEB]
- https://herolab.usd.de/security-advisories/usd-2019-0046[WEB]