diff --git a/userspace/ksud/src/magic_mount.rs b/userspace/ksud/src/magic_mount.rs index f93fce07..7490706f 100644 --- a/userspace/ksud/src/magic_mount.rs +++ b/userspace/ksud/src/magic_mount.rs @@ -71,7 +71,7 @@ impl Node { if let Some(node) = node { has_file |= if let Directory = node.file_type { - node.collect_module_files(&dir.join(&node.name))? + node.collect_module_files(dir.join(&node.name))? } else { true } @@ -146,7 +146,7 @@ fn collect_module_files() -> Result> { } if has_file { - for partition in vec!["vendor", "system_ext", "product", "odm"] { + for partition in ["vendor", "system_ext", "product", "odm"] { let path_of_root = Path::new("/").join(partition); let path_of_system = Path::new("/system").join(partition); if path_of_root.is_dir() && path_of_system.is_symlink() { @@ -305,7 +305,7 @@ fn do_magic_mount, WP: AsRef>( Some(Gid::from_raw(metadata.gid())), )?; } - lsetfilecon(&work_dir_path, lgetfilecon(&path)?.as_str())?; + lsetfilecon(&work_dir_path, lgetfilecon(path)?.as_str())?; } if create_tmpfs { diff --git a/userspace/ksud/src/module.rs b/userspace/ksud/src/module.rs index f3ab6cb0..3ad24aaf 100644 --- a/userspace/ksud/src/module.rs +++ b/userspace/ksud/src/module.rs @@ -284,7 +284,7 @@ pub fn handle_updated_modules() -> Result<()> { log::error!("Failed to remove old {}: {}", old_dir.display(), e); } } - if let Err(e) = rename(&module, &old_dir) { + if let Err(e) = rename(module, &old_dir) { log::error!("Failed to move new module {}: {}", module.display(), e); } }