Am I affected?
Description
Improper Validation of Integrity Check Value vulnerability in hexpm hex allows a package mirror to substitute the tarball that mix hex.package fetch PACKAGE VERSION --output - writes to standard output.
Hex compares a fetched tarball's inner and outer checksums with the repository's signed registry before writing it to a file or unpacking it. The standard output branch of mix hex.package fetch writes the bytes returned by Hex.SCM.fetch/3 without that comparison, and Hex.SCM.fetch/3 doesn't verify freshly downloaded tarballs itself. When a mirror configured with HEX_MIRROR_URL or mirror_url serves altered bytes, the command prints them and exits successfully, while the same fetch with --output FILE fails with a checksum mismatch.
Hex also prints diagnostics to standard output while the command runs: the body of a failed HTTP response from the update check or a registry fetch, and API warnings from the x-hex-message header. A mirror that answers one of those requests with an error whose body is a tar archive gets that archive written to standard output ahead of the verified tarball, so a pipeline that unpacks standard output extracts the mirror's files. This happens even when the checksum check then fails, because the archive is already on standard output.
This issue affects hex: from 0.20.2 before 2.5.2.
Technical analysis
mix hex.package fetch PACKAGE VERSION --output -callsfetch_tarball!/3, which callsHex.SCM.fetch/3.- For a fresh download,
Hex.SCM.fetch/3writes the response body to the cache and returns it without comparing it to the registry checksum. - The standard output branch passes those bytes to
IO.binwrite/1. The file and--unpackbranches callverify_inner_checksum!/4andverify_outer_checksum!/4first. - While the task runs, Hex prints messages with
Hex.Shell.info/1, which writes to standard output.Hex.Utils.print_error_result/1prints the body of an error response that way; the update checker calls it when its request forinstalls/hex-1.x.csvfails, andHex.Registry.Servercalls it when a registry fetch fails and the cached registry is used.Hex.HTTPprints thex-hex-messagewarning of each response the same way. These are printed before the tarball is written, so the mirror chooses bytes that come before the tarball on standard output.
Proof of concept
- Set
HEX_MIRROR_URLto a server that returns modified bytes fortarballs/decimal-2.4.1.tarand proxies everything else to repo.hex.pm. mix hex.package fetch decimal 2.4.1 --output -prints the modified bytes and exits 0.mix hex.package fetch decimal 2.4.1 --output decimal.tarfails withChecksum mismatch against registry (outer).
Weaknesses & attack patterns
Weakness
CWE-354
·
Improper Validation of Integrity Check Value
in catalog →
MITRE ↗
Attack patterns
CAPEC-186
·
Malicious Software Update
MITRE ↗
Hex treats mirrors as untrusted and relies on registry signatures and checksums for package integrity. Scripts that pipe mix hex.package fetch --output - into another tool, such as an unpacker, a vendoring step or an installer, receive whatever the mirror served, so the mirror operator can deliver modified package contents that downstream steps treat as the published release.
Affected — Hex Mix Integration Repository ↗
modules · source files · routines
Affected — GitHub / hexpm/hex Repository ↗
modules · source files · routines
Workarounds
Write the tarball to a file with --output FILE, which verifies the checksums, and read that file instead of standard output.
Configurations
Affects mix hex.package fetch with --output - when packages are downloaded through a mirror set with HEX_MIRROR_URL or mirror_url. Downloads directly from the repository over HTTPS aren't affected unless TLS verification is broken.
Solutions
Upgrade Hex to 2.5.2 or later, for example with mix local.hex. mix hex.package fetch PACKAGE VERSION --output - then verifies the registry checksum before writing the tarball to standard output and prints its messages to standard error.
References
Credits
CVSS breakdown
CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:A/VC:N/VI:H/VA:N/SC:N/SI:H/SA:N