Ecosyste.ms: Advisories

Security Advisories: GSA_kwCzR0hTQS03MzIyLWpycTQteDVoZs0WCw

File reference keys leads to incorrect hashes on HMAC algorithms

Impact

Users of HMAC-based algorithms (HS256, HS384, and HS512) combined with Lcobucci\JWT\Signer\Key\LocalFileReference as key are having their tokens issued/validated using the file path as hashing key - instead of the contents.

The HMAC hashing functions take any string as input and, since users can issue and validate tokens, people are lead to believe that everything works properly.

Patches

All versions have been patched to always load the file contents, deprecated the Lcobucci\JWT\Signer\Key\LocalFileReference, and suggest Lcobucci\JWT\Signer\Key\InMemory as the alternative.

Workarounds

Use Lcobucci\JWT\Signer\Key\InMemory instead of Lcobucci\JWT\Signer\Key\LocalFileReference to create the instances of your keys:

-use Lcobucci\JWT\Signer\Key\LocalFileReference;
+use Lcobucci\JWT\Signer\Key\InMemory;

-$key = LocalFileReference::file(__DIR__ . '/public-key.pem');
+$key = InMemory::file(__DIR__ . '/public-key.pem');

Permalink: https://github.com/advisories/GHSA-7322-jrq4-x5hf
JSON: https://advisories.ecosyste.ms/api/v1/advisories/GSA_kwCzR0hTQS03MzIyLWpycTQteDVoZs0WCw
Source: GitHub Advisory Database
Origin: Unspecified
Severity: Moderate
Classification: General
Published: about 3 years ago
Updated: 10 months ago

CVSS Score: 4.4
CVSS vector: CVSS:3.1/AV:L/AC:L/PR:L/UI:N/S:U/C:L/I:L/A:N

Identifiers: GHSA-7322-jrq4-x5hf, CVE-2021-41106
References:

• https://github.com/lcobucci/jwt/commit/8175de5b841fbe3fd97d2d49b3fc15c4ecb39a73
• https://github.com/lcobucci/jwt/commit/c45bb8b961a8e742d8f6b88ef5ff1bd5cca5d01c
• https://nvd.nist.gov/vuln/detail/CVE-2021-41106
• https://github.com/lcobucci/jwt/security/advisories/GHSA-7322-jrq4-x5hf
• https://github.com/FriendsOfPHP/security-advisories/blob/master/lcobucci/jwt/CVE-2021-41106.yaml
• https://github.com/advisories/GHSA-7322-jrq4-x5hf

Repository: https://github.com/lcobucci/jwt
Blast Radius: 21.1

Affected Packages