WEBDIENSTE/volatility3
1
0
Fork 0
mirror of https://github.com/volatilityfoundation/volatility3.git synced 2026-09-02 14:28:58 +02:00
Code Issues Packages Projects Releases Wiki Activity

Core: Add recursion protection to VersionRequirement check

Browse Source
This commit is contained in:
Mike Auty
2024-09-08 21:42:59 +01:00
parent ace590e8f0
commit bf000ff0a0
1 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
volatility3/framework/configuration/requirements.py
+15 -3
View File
@@ -546,13 +546,26 @@ class VersionRequirement(interfaces.configuration.RequirementInterface):
self._version = version
def unsatisfied(
self, context: interfaces.context.ContextInterface, config_path: str
self,
context: interfaces.context.ContextInterface,
config_path: str,
accumulator: Optional[
List[interfaces.configuration.VersionableInterface]
] = None,
) -> Dict[str, interfaces.configuration.RequirementInterface]:
# Mypy doesn't appreciate our classproperty implementation, self._plugin.version has no type
config_path = interfaces.configuration.path_join(config_path, self.name)
if not self.matches_required(self._version, self._component.version):
return {config_path: self}
if accumulator is None:
accumulator = set([self._component])
else:
if self._component in accumulator:
return {config_path: self}
else:
accumulator.add(self._component)
# Check for child requirements
if issubclass(self._component, interfaces.configuration.ConfigurableInterface):
result = {}
@@ -562,8 +575,7 @@ class VersionRequirement(interfaces.configuration.RequirementInterface):
):
result.update(
requirement.unsatisfied(
context,
config_path,
context, config_path, accumulator.copy()
)
)