From 02af9f907c73bd68eddde83b094f2a6ebd068a6f Mon Sep 17 00:00:00 2001
From: Mike Auty <mike.auty@gmail.com>
Date: Tue, 10 Mar 2026 20:13:08 +0000
Subject: [PATCH] Add in more checks and better error handling

---
 .../framework/automagic/symbol_cache.py       | 44 ++++++++++---------
 1 file changed, 24 insertions(+), 20 deletions(-)

diff --git a/volatility3/framework/automagic/symbol_cache.py b/volatility3/framework/automagic/symbol_cache.py
index f0029d0f5..a4e792cf9 100644
--- a/volatility3/framework/automagic/symbol_cache.py
+++ b/volatility3/framework/automagic/symbol_cache.py
@@ -240,16 +240,19 @@ class SqliteCache(CacheManagerInterface):
         results = self._database.cursor().execute(statement, parameters).fetchall()
         result = None
         for row in results:
-            local_filepath = self._get_local_filepath(row["location"])
-            if not (
-                local_filepath is None
-                or local_filepath.startswith(tuple(constants.SYMBOL_BASEPATHS))
-            ):
-                vollog.debug(
-                    f"Location {row['location']} found but outside of the registered symbol paths"
-                )
-            else:
-                result = row["location"]
+            if row["location"]:
+                local_filepath = pathlib.Path(self._get_local_filepath(row["location"]))
+                if local_filepath and not any(
+                    [
+                        local_filepath.is_relative_to(basepath)
+                        for basepath in constants.SYMBOL_BASEPATHS
+                    ]
+                ):
+                    vollog.debug(
+                        f"Location {row['location']} found but outside of the registered symbol paths"
+                    )
+                else:
+                    result = row["location"]
         return result
 
     def get_local_locations(self) -> Generator[str, None, None]:
@@ -496,16 +499,17 @@ class SqliteCache(CacheManagerInterface):
                 vollog.debug(
                     f"Duplicate entry for identifier {row['identifier']}: {row['location']} and {output[row['identifier']]}"
                 )
-            local_filepath = pathlib.Path(self._get_local_filepath(row["location"]))
-            if local_filepath and not any(
-                [
-                    local_filepath.is_relative_to(basepath)
-                    for basepath in constants.SYMBOL_BASEPATHS
-                ]
-            ):
-                vollog.debug(
-                    f"Location {row['location']} was not in the registered symbol paths and therefore not in the identifier dictionary"
-                )
+            if row["location"]:
+                local_filepath = pathlib.Path(self._get_local_filepath(row["location"]))
+                if local_filepath and not any(
+                    [
+                        local_filepath.is_relative_to(basepath)
+                        for basepath in constants.SYMBOL_BASEPATHS
+                    ]
+                ):
+                    vollog.debug(
+                        f"Location {row['location']} was not in the registered symbol paths and therefore not in the identifier dictionary"
+                    )
             else:
                 output[row["identifier"]] = row["location"]
         return output