Core: Add recursion protection to VersionRequirement check

Author: ikelos

volatility3/framework/configuration/requirements.py

@@ -546,13 +546,26 @@ def __init__(
         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 @@ def unsatisfied(
                 ):
                     result.update(
                         requirement.unsatisfied(
-                            context,
-                            config_path,
+                            context, config_path, accumulator.copy()
                         )
                     )