GHSA-5w8r-8pgj-5jmf
Suggest an improvement- Source
- https://github.com/advisories/GHSA-5w8r-8pgj-5jmf
- Import Source
- https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2022/09/GHSA-5w8r-8pgj-5jmf/GHSA-5w8r-8pgj-5jmf.json
- JSON Data
- https://api.osv.dev/v1/vulns/GHSA-5w8r-8pgj-5jmf
- Aliases
- Published
- 2022-09-30T22:46:50Z
- Modified
- 2023-11-08T04:10:16.632776Z
- Severity
-
- 8.6 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:C/C:N/I:H/A:N CVSS Calculator
- Summary
- matrix-js-sdk subject to user impersonation due to key/device identifier confusion in SAS verification
- Details
-
Impact
An attacker cooperating with a malicious homeserver could interfere with the verification flow between two users, injecting its own cross-signing user identity in place of one of the users’ identities, leading to the other device trusting/verifying the user identity under the control of the homeserver instead of the intended one.
The vulnerability is a bug in the matrix-js-sdk, caused by checking and signing user identities and devices in two separate steps, and inadequately fixing the keys to be signed between those steps.
Even though the attack is partly made possible due to the design decision of treating cross-signing user identities as Matrix devices on the server side (with their device ID set to the public part of the user identity key), no other examined implementations were vulnerable.
Patches
The matrix-js-sdk has been modified to double check that the key signed is the one that was verified instead of just referencing the key by ID. An additional check has been made to report an error when one of the device ID matches a cross-signing key.
Workarounds
As this attack requires coordination between a malicious homeserver and an attacker -- if you trust your homeserver no particular workaround is needed.
As a potential way of detecting compromise, it’s possible to review your device list or the device list of other users for devices with IDs in the form of a base64 cross-signing key (
5XaczGNlfz0bl8R1IX5qn+tBoue2tWJqLMh+SDUuvCk) instead of classical device ID (SEHACYDHMG).References
Blog post: https://matrix.org/blog/2022/09/28/upgrade-now-to-address-encryption-vulns-in-matrix-sdks-and-clients
For more information
If you have any questions or comments about this advisory, e-mail us at security@matrix.org
- Database specific
{ "nvd_published_at": "2022-09-29T13:15:00Z", "github_reviewed_at": "2022-09-30T22:46:50Z", "severity": "HIGH", "github_reviewed": true, "cwe_ids": [ "CWE-287", "CWE-322" ] }- References
-
- https://github.com/matrix-org/matrix-js-sdk/security/advisories/GHSA-5w8r-8pgj-5jmf
- https://nvd.nist.gov/vuln/detail/CVE-2022-39250
- https://github.com/matrix-org/matrix-js-sdk/commit/a587d7c36026fe1fcf93dfff63588abee359be76
- https://github.com/matrix-org/matrix-js-sdk
- https://github.com/matrix-org/matrix-js-sdk/releases/tag/v19.7.0
- https://matrix.org/blog/2022/09/28/upgrade-now-to-address-encryption-vulns-in-matrix-sdks-and-clients
- https://security.gentoo.org/glsa/202210-35
Affected packages
npm / matrix-js-sdk
Package
- Name
- matrix-js-sdk
- View open source insights on deps.dev
- Purl
- pkg:npm/matrix-js-sdk