Check that paths only contain normal components

Author: cruessler

Diff for: Cargo.lock

@@ -49,6 +49,7 @@ gix-hashtable = { version = "^0.8.0", path = "../gix-hashtable" }
 gix-validate = { version = "^0.9.4", path = "../gix-validate" }
 gix-actor = { version = "^0.34.0", path = "../gix-actor" }
 gix-date = { version = "^0.9.4", path = "../gix-date" }
+gix-fs = { version = "^0.14.0", path = "../gix-fs" }
 gix-path = { version = "^0.10.15", path = "../gix-path" }
 gix-utils = { version = "^0.2.0", path = "../gix-utils" }
 

Diff for: gix-object/Cargo.toml

@@ -1,4 +1,5 @@
 use bstr::ByteSlice;
+use gix_fs::stack::ToNormalPathComponents;
 
 use crate::{
     bstr::{BStr, BString},
@@ -380,18 +381,29 @@ impl std::ops::Index<std::ops::RangeTo<usize>> for RepositoryPathPuf {
 
 ///
 pub mod repository_path_buf {
+    use gix_fs::stack::to_normal_path_components;
+
     /// The error used in [`RepositoryPathPuf`](super::RepositoryPathPuf).
     #[derive(Debug, thiserror::Error)]
     #[allow(missing_docs)]
-    pub enum Error {}
+    pub enum Error {
+        #[error(transparent)]
+        ContainsNonNormalComponents(#[from] to_normal_path_components::Error),
+        #[error(transparent)]
+        IllegalUtf8(#[from] gix_path::Utf8Error),
+    }
 }
 
 impl TryFrom<&str> for RepositoryPathPuf {
     type Error = repository_path_buf::Error;
 
     fn try_from(value: &str) -> Result<Self, Self::Error> {
-        // TODO
-        // Check whether the documented constraints are met, return `Err` if not.
+        let path = std::path::Path::new(value);
+
+        for component in path.to_normal_path_components() {
+            component?;
+        }
+
         Ok(Self { inner: value.into() })
     }
 }
@@ -400,8 +412,12 @@ impl TryFrom<&BStr> for RepositoryPathPuf {
     type Error = repository_path_buf::Error;
 
     fn try_from(value: &BStr) -> Result<Self, Self::Error> {
-        // TODO
-        // Check whether the documented constraints are met, return `Err` if not.
+        let path: &std::path::Path = &gix_path::try_from_bstr(value)?;
+
+        for component in path.to_normal_path_components() {
+            component?;
+        }
+
         Ok(Self { inner: value.into() })
     }
 }
@@ -410,8 +426,12 @@ impl TryFrom<BString> for RepositoryPathPuf {
     type Error = repository_path_buf::Error;
 
     fn try_from(value: BString) -> Result<Self, Self::Error> {
-        // TODO
-        // Check whether the documented constraints are met, return `Err` if not.
+        let path: &std::path::Path = &gix_path::try_from_bstr(&value)?;
+
+        for component in path.to_normal_path_components() {
+            component?;
+        }
+
         Ok(Self { inner: value })
     }
 }