fix: don't expose the . & .. entries to the end user

Oakchris1955 · Sep 14, 2024 · 02eb213 · 02eb213
1 parent 842fce1
commit 02eb213
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/fat/fs.rs b/src/fat/fs.rs
@@ -1,6 +1,6 @@
 use super::*;
 
-use crate::{error::*, io::prelude::*, path::PathBuf, time::*, utils};
+use crate::{error::*, io::prelude::*, path::PathBuf, time::*, utils, SPECIAL_ENTRIES};
 
 use core::{cmp, ops};
 
@@ -1315,6 +1315,8 @@ where
         Ok(entries
             .into_iter()
             .filter(|x| self.filter.filter(x))
+            // we shouldn't expose the special entries to the user
+            .filter(|x| !SPECIAL_ENTRIES.contains(&x.name.as_str()))
             .map(|rawentry| {
                 let mut entry_path = path.clone();
 

diff --git a/src/path.rs b/src/path.rs
@@ -29,6 +29,9 @@ pub const RESERVED_FILENAMES: &[&str] = &[
     "COM8", "COM9", "COM¹", "COM²", "COM³", "LPT0", "LPT1", "LPT2", "LPT3", "LPT4", "LPT5", "LPT6",
     "LPT7", "LPT8", "LPT9", "LPT¹", "LPT²", "LPT³",
 ];
+/// Except for the root directory, each directory must contain
+/// the following (two) entries at the beginning of the directory
+pub(crate) const SPECIAL_ENTRIES: &[&str] = &[".", ".."];
 
 /// Check whether a [`PathBuf`] is forbidden for use in filenames or directory names
 fn is_forbidden(pathbuf: &PathBuf) -> bool {