Merge pull request #42 from omerbenamram/bugfix/only-consider-directo…

…ries-as-valid-parents fix
omerbenamram · Apr 14, 2020 · 48e2c6f · 48e2c6f
2 parents 5c3ce2e + 3cb88cd
commit 48e2c6f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
@@ -11,7 +11,7 @@ strategy:
       target: 'x86_64-pc-windows-msvc'
       rustup_toolchain: stable
     mac-stable:
-      imageName: 'macos-10.13'
+      imageName: 'macOS-10.14'
       target: 'x86_64-apple-darwin'
       rustup_toolchain: stable
     linux-stable:

diff --git a/src/mft.rs b/src/mft.rs
@@ -99,7 +99,8 @@ impl<T: ReadSeek> MftParser<T> {
         } else {
             let path = match self.get_entry(parent_entry_id).ok() {
                 Some(parent) => match self.get_full_path_for_entry(&parent) {
-                    Ok(Some(path)) => path,
+                    Ok(Some(path)) if parent.is_dir() => path,
+                    Ok(Some(_)) => PathBuf::from("[Unknown]"),
                     // I have a parent, which doesn't have a filename attribute.
                     // Default to root.
                     _ => PathBuf::new(),