Fix deduplication of collected files on case insensitive targets

(DIS-2945)

Author: pyrco

acquire/acquire.py

@@ -99,11 +99,11 @@
 
 def misc_windows_user_homes(target: Target) -> Iterator[fsutil.TargetPath]:
     misc_dirs = {
-        ("windows/serviceprofiles/localservice", False),
-        ("windows/serviceprofiles/networkservice", False),
-        ("windows/system32/config/systemprofile", False),
-        ("users", True),
-        ("documents and settings", True),
+        ("Windows/ServiceProfiles/LocalService", False),
+        ("Windows/ServiceProfiles/NetworkService", False),
+        ("Windows/System32/config/systemprofile", False),
+        ("Users", True),
+        ("Documents and Settings", True),
     }
 
     for fs in target.fs.path().iterdir():
@@ -146,7 +146,7 @@ def misc_osx_user_homes(target: Target) -> Iterator[fsutil.TargetPath]:
 def from_user_home(target: Target, path: str) -> Iterator[str]:
     try:
         for user_details in target.user_details.all_with_home():
-            yield normalize_path(target, user_details.home_path.joinpath(path))
+            yield normalize_path(target, user_details.home_path.joinpath(path), lower_case=False)
     except Exception as e:
         log.warning("Error occurred when requesting all user homes")
         log.debug("", exc_info=e)

acquire/collector.py

@@ -74,8 +74,14 @@ def serialize_path(path: Any) -> str:
 
     # Naive way to serialize TargetPath filesystem's metadata is
     # to rely on uniqueness of `path._fs` object
-    fs_id = id(path._fs)
-    return f"{path._fs.__type__}:{fs_id}:{path}"
+    fs = path._fs
+    fs_id = id(fs)
+    fs_type = fs.__type__
+    path = str(path)
+    if not fs.case_sensitive:
+        path = path.lower()
+
+    return f"{fs_type}:{fs_id}:{path}"
 
 
 @dataclass

acquire/utils.py

@@ -371,13 +371,13 @@ def persist_execution_report(path: Path, report_data: dict) -> Path:
 SYSVOL_SUBST = re.compile(r"^(/\?\?/)?[cC]:")
 
 
-def normalize_path(target: Target, path: Path, resolve: bool = False) -> str:
+def normalize_path(target: Target, path: Path, resolve: bool = False, lower_case: bool = True) -> str:
     if resolve:
         path = path.resolve()
 
     path = path.as_posix()
 
-    if not target.fs.case_sensitive:
+    if not target.fs.case_sensitive and lower_case:
         path = path.lower()
 
     if target.os == "windows":