Return appropriate error if path list defined and empty

harryfei · Oct 17, 2023 · e776ff0 · e776ff0
1 parent 70d2d1c
commit e776ff0
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/finder.rs b/src/finder.rs
@@ -83,8 +83,12 @@ impl Finder {
             }
             _ => {
                 // Search binary in PATHs(defined in environment variable).
-                let p = paths.ok_or(Error::CannotGetCurrentDirAndPathListEmpty)?;
-                let paths: Vec<_> = env::split_paths(&p).collect();
+                let paths =
+                    env::split_paths(&paths.ok_or(Error::CannotGetCurrentDirAndPathListEmpty)?)
+                        .collect::<Vec<_>>();
+                if paths.is_empty() {
+                    return Err(Error::CannotGetCurrentDirAndPathListEmpty);
+                }
 
                 Either::Right(Self::path_search_candidates(path, paths).into_iter())
             }