|
 |
54c5d5 |
From 4f0099156245ed2873d6945d5e58db741e15836d Mon Sep 17 00:00:00 2001
|
|
|
54c5d5 |
From: Lumir Balhar <lbalhar@redhat.com>
|
|
|
54c5d5 |
Date: Tue, 8 Jun 2021 09:51:47 +0200
|
|
|
54c5d5 |
Subject: [PATCH] CVE-2021-3572
|
|
|
54c5d5 |
|
|
|
54c5d5 |
---
|
|
|
54c5d5 |
src/pip/_internal/vcs/git.py | 10 ++++++++--
|
|
|
54c5d5 |
1 file changed, 8 insertions(+), 2 deletions(-)
|
|
|
54c5d5 |
|
|
|
54c5d5 |
diff --git a/src/pip/_internal/vcs/git.py b/src/pip/_internal/vcs/git.py
|
|
|
54c5d5 |
index 92b8457..7b3cc4a 100644
|
|
|
54c5d5 |
--- a/src/pip/_internal/vcs/git.py
|
|
|
54c5d5 |
+++ b/src/pip/_internal/vcs/git.py
|
|
|
54c5d5 |
@@ -120,9 +120,15 @@ class Git(VersionControl):
|
|
|
54c5d5 |
output = cls.run_command(['show-ref', rev], cwd=dest,
|
|
|
54c5d5 |
show_stdout=False, on_returncode='ignore')
|
|
|
54c5d5 |
refs = {}
|
|
|
54c5d5 |
- for line in output.strip().splitlines():
|
|
|
54c5d5 |
+ # NOTE: We do not use splitlines here since that would split on other
|
|
|
54c5d5 |
+ # unicode separators, which can be maliciously used to install a
|
|
|
54c5d5 |
+ # different revision.
|
|
|
54c5d5 |
+ for line in output.strip().split("\n"):
|
|
|
54c5d5 |
+ line = line.rstrip("\r")
|
|
|
54c5d5 |
+ if not line:
|
|
|
54c5d5 |
+ continue
|
|
|
54c5d5 |
try:
|
|
|
54c5d5 |
- sha, ref = line.split()
|
|
|
54c5d5 |
+ sha, ref = line.split(" ", maxsplit=2)
|
|
|
54c5d5 |
except ValueError:
|
|
|
54c5d5 |
# Include the offending line to simplify troubleshooting if
|
|
|
54c5d5 |
# this error ever occurs.
|
|
|
54c5d5 |
--
|
|
|
54c5d5 |
2.31.1
|
|
|
54c5d5 |
|