mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
avoid chroot paths (#32778)
* avoid chroot paths fixes #32764 * check name
This commit is contained in:
parent
7a82c49a5f
commit
e7941b0d4e
1 changed files with 25 additions and 23 deletions
|
@ -74,31 +74,33 @@ class VarsModule(BaseVarsPlugin):
|
|||
else:
|
||||
raise AnsibleParserError("Supplied entity must be Host or Group, got %s instead" % (type(entity)))
|
||||
|
||||
try:
|
||||
found_files = []
|
||||
# load vars
|
||||
opath = os.path.realpath(os.path.join(self._basedir, subdir))
|
||||
key = '%s.%s' % (entity.name, opath)
|
||||
if cache and key in FOUND:
|
||||
found_files = FOUND[key]
|
||||
else:
|
||||
b_opath = to_bytes(opath)
|
||||
# no need to do much if path does not exist for basedir
|
||||
if os.path.exists(b_opath):
|
||||
if os.path.isdir(b_opath):
|
||||
self._display.debug("\tprocessing dir %s" % opath)
|
||||
found_files = self._find_vars_files(opath, entity.name)
|
||||
FOUND[key] = found_files
|
||||
else:
|
||||
self._display.warning("Found %s that is not a directory, skipping: %s" % (subdir, opath))
|
||||
# avoid 'chroot' type inventory hostnames /path/to/chroot
|
||||
if not entity.name.startswith(os.path.sep):
|
||||
try:
|
||||
found_files = []
|
||||
# load vars
|
||||
opath = os.path.realpath(os.path.join(self._basedir, subdir))
|
||||
key = '%s.%s' % (entity.name, opath)
|
||||
if cache and key in FOUND:
|
||||
found_files = FOUND[key]
|
||||
else:
|
||||
b_opath = to_bytes(opath)
|
||||
# no need to do much if path does not exist for basedir
|
||||
if os.path.exists(b_opath):
|
||||
if os.path.isdir(b_opath):
|
||||
self._display.debug("\tprocessing dir %s" % opath)
|
||||
found_files = self._find_vars_files(opath, entity.name)
|
||||
FOUND[key] = found_files
|
||||
else:
|
||||
self._display.warning("Found %s that is not a directory, skipping: %s" % (subdir, opath))
|
||||
|
||||
for found in found_files:
|
||||
new_data = loader.load_from_file(found, cache=True, unsafe=True)
|
||||
if new_data: # ignore empty files
|
||||
data = combine_vars(data, new_data)
|
||||
for found in found_files:
|
||||
new_data = loader.load_from_file(found, cache=True, unsafe=True)
|
||||
if new_data: # ignore empty files
|
||||
data = combine_vars(data, new_data)
|
||||
|
||||
except Exception as e:
|
||||
raise AnsibleParserError(to_native(e))
|
||||
except Exception as e:
|
||||
raise AnsibleParserError(to_native(e))
|
||||
return data
|
||||
|
||||
def _find_vars_files(self, path, name):
|
||||
|
|
Loading…
Reference in a new issue