GitHub Advisoryadvisory · PyPI

GHSA-9xgj-fcgf-x6mw Poetry Argument Injection can lead to Local Code Execution

HIGH Published 2022-09-16 Modified 2024-10-21 Official page ↗
Aliases
CVE-2022-36069
Ecosystem
PyPI
Packages
poetry
Severity
High
CVSS vector
CVSS:3.1/AV:L/AC:L/PR:L/UI:R/S:U/C:H/I:H/A:H
Weaknesses
CWE-88, CWE-94
Reviewed
2022-09-16

Affected packages

• PyPI poetry: < 1.1.9 → fixed in 1.1.9

Description

Observation When handling dependencies that come from a Git repository instead of a registry, Poetry uses various commands, such as `git clone`. These commands are being constructed using user input (e.g. the repository URL). When building the commands, Poetry correctly avoids Command Injection vulnerabilities by passing an array of arguments instead of a command string. However, there is the possibility that a user input starts with a dash (`-`) and is therefore treated as an optional argument instead of a positional one. This can lead to Code Execution because some of the commands have options that can be leveraged to run arbitrary executables. To clone a repository, Poetry builds a git clone command, but fails to validate or sanitize the repository location properly: `poetry/core/vcs/git.py`: ```python def clone(self, repository: str, dest: Path) -> str: return self.run("clone", "--recurse-submodules", repository, str(dest)) ``` Since this value comes from the `pyproject.toml` file, it can contain any character, including a leading dash. Impact This vulnerability can lead to Arbitrary Code Execution, which would lead to the takeover of the system. If a developer is exploited, the attacker could steal credentials or persist their access. If the exploit happens on a server, the attackers could use their access to attack other internal systems. Since this vulnerability requires a fair amount of user interaction, it is not as dangerous as a remotely exploitable one. However, it still puts developers at risk when dealing with untrusted files in a way they think is safe, because the exploit still works when the victim tries to make sure nothing can happen, e.g. by vetting any Git or Poetry config files that might be present in the directory. This kind of attack vector has been used in the past to target security researchers by sending them projects to collaborate on, so we believe that there is a non-negligible risk. Patches 1.1.8 || 1.2.0b1 Remediation Upgrade to version 1.1.9 || 1.2.0b1 References Fix PR For more information If you have any questions or comments about this advisory, email us at [email protected]

Related

Alias · CVE (1)

  • CVE-2022-36069 Poetry is a dependency manager for Python. vulnerability

Weakness · CWE (2)

  • CWE-88 Improper Neutralization of Argument Delimiters in a Command weakness
  • CWE-94 Improper Control of Generation of Code weakness

References