| Package | Affected versions | Patched versions |
|---|---|---|
| typo3fluid/fluid | ≥ 2.0.0, < 2.0.8≥ 2.1.0, < 2.1.7≥ 2.2.0, < 2.2.4≥ 2.3.0, < 2.3.7≥ 2.4.0, < 2.4.4≥ 2.5.0, < 2.5.11≥ 2.6.0, < 2.6.10 | 2.0.82.1.72.2.42.3.72.4.42.5.112.6.10 |
| fluid | ≥ 2.1.0, < 2.1.7≥ 2.2.0, < 2.2.4≥ 2.3.0, < 2.3.7≥ 2.4.0, < 2.4.4≥ 2.5.0, < 2.5.11≥ 2.6.0, < 2.6.10< 2.0.8 | 2.1.72.2.42.3.72.4.42.5.112.6.102.0.8 |
Meta
- CVSS:
CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:L/I:L/A:N/E:F/RL:O/RC:C(5.7)- CWE-79
Three XSS vulnerabilities have been detected in Fluid:
additionalAttributes arrays by creating keys with attribute-closing quotes followed by HTML. When rendering such attributes, TagBuilder would not escape the keys.CompileWithContentArgumentAndRenderStatic trait, and which declared escapeOutput = false, would receive the content argument in unescaped format.then and else arguments in unescaped format.Update to versions 2.0.8, 2.1.7, 2.2.4, 2.3.7, 2.4.4, 2.5.11 or 2.6.10 of this typo3fluid/fluid package that fix the problem described.
Updated versions of this package are bundled in following TYPO3 (typo3/cms-core) releases:
The specific vulnerabilities are prevented by:
additionalAttributes array passed to a TagBasedViewHelper before using them as attribute names.escapeChildren when escapeOutput is toggled off. Escaping still will not occur if explicitly disabled by an enclosing ViewHelper. This homogenises escaping behavior of "content arguments" so the same strategy is used whether the "content" argument is passed as argument or child content.then and else arguments on AbstractConditionViewHelper subclasses as escaped and applying escaping in all cases where escaping is not explicitly disabled by an enclosing ViewHelper.additionalAttributes.escapeOutput = false is used with the content argument instead of passing variables as child node - e.g. instead of to intentionally circumvent escaping of any HTML in .Running one of these products? Scan your site to see whether it exposes a vulnerable version.
Scan my site free<v:h content="{variable}" /><v:h>{variable}</v:h>{variable}then or else arguments to render a variable containing HTML, excluding cases where the variable is intentionally unescaped - e.g. <f:if condition="1" then="{variable -> f:format.raw()}" />, and excluding any cases where a ViewHelper is used as argument value and the ViewHelper intentionally disables escaping - e.g. <f:if condition="1" then="{f:render(section: 'MySection')}" /> does not escape the then argument because f:render disables output escaping.Cases 2 and 3 can be mitigated to allow variables with HTML to not be escaped, by intentionally disabling escaping by chaining the variable used in the argument with f:format.raw as described in case 3. Note that this constitutes a potential security issue, for which the template author is solely responsible. Example: <f:if condition="1" then="{intentionalHtmlVariable}" /> can allow HTML in {intentionalHtmlVariable} by adding -> f:format.raw() - to become <f:if condition="1" then="{intentionalHtmlVariable -> f:format.raw()}" />.
Custom ViewHelpers which use CompileWithContentArgumentAndRenderStatic can alternatively pass a 6th argument with value false to the call to registerArgument which registers the "content argument", which explicitly disables escaping of the argument value: $this->registerArgument('arg', 'string', 'My argument', false, null, false);. Note that this constitutes a potential security issue for which the ViewHelper author is solely responsible. Variables containing HTML should only be allowed after taking great care to prevent XSS through other means, e.g. sanitising the variable before it is assigned to Fluid or only allowing such variables to come from trusted sources.
Thanks to Jonas Eberle and Sinan Sekerci (Dreamlab Technologies) who reported this issue and to TYPO3 core merger Claus Due who fixed the issue.