From d1fd13163f9ab6b309996eee27fe7939a3e21fda Mon Sep 17 00:00:00 2001 From: Alex Zenla Date: Sat, 1 Nov 2025 18:51:07 -0400 Subject: [PATCH] fix(filesystem-device-match): make behavior of ignoring filesystem errors more explicit --- src/extractors/filesystem_device_match.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/extractors/filesystem_device_match.rs b/src/extractors/filesystem_device_match.rs index 0fc1577..bbbfaaa 100644 --- a/src/extractors/filesystem_device_match.rs +++ b/src/extractors/filesystem_device_match.rs @@ -138,14 +138,11 @@ pub fn extract( let mut filesystem = FileSystem::new(filesystem); // Check the metadata of the item. - let metadata = filesystem.metadata(Path::new(&want_item)); - // Ignore filesystem errors as we can't do anything useful with the error. - if metadata.is_err() { + let Some(metadata) = filesystem.metadata(Path::new(&want_item)).ok() else { continue; - } + }; - let metadata = metadata?; // Only check directories and files. if !(metadata.is_directory() || metadata.is_regular_file()) { continue;