CVE-2024-47186 – filament/tables

Package

Manager: composer
Name: filament/tables
Vulnerable Version: >=3.0.0 <3.2.115

Severity

Level: Critical

CVSS v3.1: CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:C/C:L/I:N/A:N/E:H/RL:U/RC:R

CVSS v4.0: CVSS:4.0/AV:N/AC:L/AT:N/PR:L/UI:N/VC:L/VI:N/VA:N/SC:L/SI:N/SA:N

EPSS: 0.0007 pctl0.21985

Details

Filament has unvalidated ColorColumn and ColorEntry values that can be used for Cross-site Scripting ### Summary If values passed to a `ColorColumn` or `ColumnEntry` are not valid and contain a specific set of characters, applications are vulnerable to XSS attack against a user who opens a page on which a color column or entry is rendered. Versions of Filament from v3.0.0 through v3.2.114 are affected. Please upgrade to Filament [v3.2.115](https://github.com/filamentphp/filament/releases/tag/v3.2.115). ### PoC For example, using a value such as: ```html blue;"><script>alert('There\'s a security problem here')</script style=" ``` Would get passed into the `@style()` Blade directive from Laravel to render the correct background color, where `$state` contains the value: ```blade <div @style([ "background-color: {$state}" => $state, ])></div> ``` Since Laravel does not escape special characters within the `@style` Blade directive, the effective output HTML would be: ```html <div style="background-color: blue;"><script>alert('There\'s a security problem here')</script style=""></div> ``` Creating the opportunity for arbitrary JS to run if it was stored in the database. ### Response This vulnerability (in `ColorColumn` only) was reported by @sv-LayZ, who reported the issue and patched the issue during the evening of 25/09/2024. Thank you Mattis. The review process concluded on 27/09/2024, which revealed the issue was also present in `ColorEntry`. This was fixed the same day and Filament [v3.2.115](https://github.com/filamentphp/filament/releases/tag/v3.2.115) followed to escape any special characters while outputting inline styles like this: ```blade <div @style([ 'background-color: ' . e($state) => $state, ])></div> ``` Although these components are no longer vulnerable to this type of XSS attack, it is good practice to validate colors, and since many Filament users may be accepting color input using the `ColorPicker` form component, [additional color validation documentation was published](https://filamentphp.com/docs/3.x/forms/fields/color-picker#color-picker-validation).

Metadata

Created: 2024-09-27T20:51:01Z
Modified: 2024-10-23T17:40:43Z
Source: https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2024/09/GHSA-9h9q-qhxg-89xr/GHSA-9h9q-qhxg-89xr.json
CWE IDs: ["CWE-79"]
Alternative ID: GHSA-9h9q-qhxg-89xr
Finding: F425
Auto approve: 1