Add proper exception handling in file descriptor enumeration

volatilityfoundation · Jan 31, 2025 · e6f9ddb · e6f9ddb
1 parent b51acfd
commit e6f9ddb
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/volatility3/framework/symbols/linux/__init__.py b/volatility3/framework/symbols/linux/__init__.py
@@ -336,15 +336,23 @@ def files_descriptors_for_process(
         symbol_table: str,
         task: interfaces.objects.ObjectInterface,
     ):
-        # task.files can be null
-        if not (task.files and task.files.is_readable()):
+        try:
+            files = task.files
+        except exceptions.InvalidAddressException:
+            return None
+
+        if not files.is_readable():
+            return None
+
+        try:
+            fd_table = files.get_fds()
+        except exceptions.InvalidAddressException:
             return None
 
-        fd_table = task.files.get_fds()
         if fd_table == 0:
             return None
 
-        max_fds = task.files.get_max_fds()
+        max_fds = files.get_max_fds()
 
         # corruption check
         if max_fds > 500000: