Use to_str instead of lossy

Author: Stringy

fact/src/bpf/mod.rs

@@ -133,11 +133,16 @@ impl Bpf {
         let mut new_paths = Vec::with_capacity(paths_config.len());
         let mut builder = GlobSetBuilder::new();
         for p in paths_config.iter() {
+            let Some(glob_str) = p.to_str() else {
+                bail!("failed to convert path {} to string", p.display());
+            };
+
             builder.add(
-                Glob::new(&p.to_string_lossy())
-                    .with_context(|| format!("invalid glob {}", p.display()))
+                Glob::new(glob_str)
+                    .with_context(|| format!("invalid glob {}", glob_str))
                     .unwrap(),
             );
+
             let prefix = path_prefix_t::try_from(p)?;
             path_prefix.insert(&prefix.into(), 0, 0)?;
             new_paths.push(prefix);

fact/src/host_scanner.rs

@@ -25,7 +25,7 @@ use std::{
     sync::Arc,
 };
 
-use anyhow::Context;
+use anyhow::{Context, bail};
 use aya::maps::MapData;
 use fact_ebpf::{inode_key_t, inode_value_t};
 use log::{debug, info, warn};
@@ -85,7 +85,11 @@ impl HostScanner {
     }
 
     fn scan_inner(&self, path: &Path) -> anyhow::Result<()> {
-        for entry in glob::glob(&path.to_string_lossy())? {
+        let Some(glob_str) = path.to_str() else {
+            bail!("invalid path {}", path.display());
+        };
+
+        for entry in glob::glob(glob_str)? {
             match entry {
                 Ok(path) => {
                     if path.is_file() {