Skip to content

Cross-site scripting in SimpleSAMLphp

Low severity GitHub Reviewed Published Jan 24, 2020 in simplesamlphp/simplesamlphp • Updated Jan 9, 2023

Package

composer simplesamlphp/simplesamlphp (Composer)

Affected versions

>= 1.18.0, < 1.18.4

Patched versions

1.18.4

Description

Background

SimpleSAMLphp allows users to report errors and failures to the system administrators via a web form. This web form gathers some contextual information automatically, but it also allows the user to provide their email address for follow-ups and a free-text explanation of what happened. Once submitted, the report is sent as an email to the administrative email address configured in the software.

Description

The www/erroreport.php script allows error reports to be submitted and sent to the system administrator. Starting with SimpleSAMLphp 1.18.0, a new SimpleSAML\Utils\EMail class was introduced to handle sending emails, implemented as a wrapper of an external dependency.

This new wrapper allows us to use Twig templates in order to create the email sent with an error report. Since Twig provides automatic escaping of variables, manual escaping of the free-text field in www/errorreport.php was removed to avoid double escaping. However, for those not using the new user interface yet, an email template is hardcoded into the class itself in plain PHP. Since no escaping is provided in this template, it is then possible to inject HTML inside the template by manually crafting the contents of the free-text field.

Affected versions

SimpleSAMLphp versions 1.18.0 to 1.18.3, both inclusive.

Impact

An attacker may take advantage of this issue to manually craft a modified email sent via the error reporting mechanism, such as this email might trick a system administrator into performing an action, such as introducing their credentials into a phishing web site that resembles the original.

Remote execution of javascript code is considered unfeasible since email clients do not run javascript present in emails, although there might be email clients which don't follow this rule, making the attack surface grow.

Resolution

Upgrade the SimpleSAMLphp installation to version 1.18.4.

Credit

This vulnerability was discovered and reported by Frederic Vleminckx.

References

@jaimeperez jaimeperez published to simplesamlphp/simplesamlphp Jan 24, 2020
Reviewed Jan 24, 2020
Published to the GitHub Advisory Database Jan 24, 2020
Last updated Jan 9, 2023

Severity

Low

Weaknesses

CVE ID

CVE-2020-5226

GHSA ID

GHSA-mj9p-v2r8-wf8w

Source code

No known source code
Loading Checking history
See something to contribute? Suggest improvements for this vulnerability.