diff options
Diffstat (limited to 'runtime/permissions.rs')
-rw-r--r-- | runtime/permissions.rs | 29 |
1 files changed, 27 insertions, 2 deletions
diff --git a/runtime/permissions.rs b/runtime/permissions.rs index 782c44e4f..a331ef3e9 100644 --- a/runtime/permissions.rs +++ b/runtime/permissions.rs @@ -581,8 +581,13 @@ impl Permissions { ) -> Result<(), AnyError> { let url = specifier.as_url(); if url.scheme() == "file" { - let path = url.to_file_path().unwrap(); - self.check_read(&path) + match url.to_file_path() { + Ok(path) => self.check_read(&path), + Err(_) => Err(uri_error(format!( + "Invalid file path.\n Specifier: {}", + specifier + ))), + } } else { self.check_net_url(url) } @@ -943,6 +948,26 @@ mod tests { } #[test] + fn check_invalid_specifiers() { + let perms = Permissions::allow_all(); + + let mut test_cases = vec![]; + + if cfg!(target_os = "windows") { + test_cases.push("file://"); + test_cases.push("file:///"); + } else { + test_cases.push("file://remotehost/"); + } + + for url in test_cases { + assert!(perms + .check_specifier(&ModuleSpecifier::resolve_url_or_path(url).unwrap()) + .is_err()); + } + } + + #[test] fn test_deserialize_perms() { let json_perms = r#" { |