saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Ensure that failed recursion results in no blob object being returned

Browse source
This commit is contained in:
Erik Johnson 2016-08-05 14:07:50 -05:00
parent 67d8dd0fd0
commit 6764a88601
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
salt/utils/gitfs.py
View file

@ -857,6 +857,7 @@ class GitPython(GitProvider):
while True:
depth += 1
if depth > SYMLINK_RECURSE_DEPTH:
blob = None
break
try:
file_blob = tree / path
@ -878,6 +879,7 @@ class GitPython(GitProvider):
break
except KeyError:
# File not found or repo_path points to a directory
blob = None
break
return blob, blob.hexsha if blob is not None else blob
@ -1400,6 +1402,7 @@ class Pygit2(GitProvider):
while True:
depth += 1
if depth > SYMLINK_RECURSE_DEPTH:
blob = None
break
try:
if stat.S_ISLNK(tree[path].filemode):
@ -1415,6 +1418,7 @@ class Pygit2(GitProvider):
oid = tree[path].oid
blob = self.repo[oid]
except KeyError:
blob = None
break
return blob, blob.hex if blob is not None else blob
@ -1750,6 +1754,7 @@ class Dulwich(GitProvider): # pylint: disable=abstract-method
while True:
depth += 1
if depth > SYMLINK_RECURSE_DEPTH:
blob = None
break
prefix_dirs, _, filename = path.rpartition(os.path.sep)
tree = self.walk_tree(tree, prefix_dirs)
@ -1771,6 +1776,7 @@ class Dulwich(GitProvider): # pylint: disable=abstract-method
blob = self.repo.get_object(oid)
break
except KeyError:
blob = None
break
return blob, blob.sha().hexdigest() if blob is not None else blob