diff --git a/src/uu/ls/src/ls.rs b/src/uu/ls/src/ls.rs index 5c021aa5c..de6fabecf 100644 --- a/src/uu/ls/src/ls.rs +++ b/src/uu/ls/src/ls.rs @@ -3179,8 +3179,8 @@ fn display_len_or_rdev(metadata: &Metadata, config: &Config) -> SizeOrDeviceId { if ft.is_char_device() || ft.is_block_device() { // A type cast is needed here as the `dev_t` type varies across OSes. let dev = metadata.rdev() as dev_t; - let major = unsafe { major(dev) }; - let minor = unsafe { minor(dev) }; + let major = major(dev); + let minor = minor(dev); return SizeOrDeviceId::Device(major.to_string(), minor.to_string()); } } diff --git a/src/uu/wc/src/count_fast.rs b/src/uu/wc/src/count_fast.rs index 7edc02437..2211ae05d 100644 --- a/src/uu/wc/src/count_fast.rs +++ b/src/uu/wc/src/count_fast.rs @@ -51,7 +51,7 @@ fn count_bytes_using_splice(fd: &impl AsFd) -> Result { let null_rdev = stat::fstat(null_file.as_raw_fd()) .map_err(|_| 0_usize)? .st_rdev as libc::dev_t; - if unsafe { (libc::major(null_rdev), libc::minor(null_rdev)) } != (1, 3) { + if (libc::major(null_rdev), libc::minor(null_rdev)) != (1, 3) { // This is not a proper /dev/null, writing to it is probably bad // Bit of an edge case, but it has been known to happen return Err(0); diff --git a/tests/by-util/test_ls.rs b/tests/by-util/test_ls.rs index 7c9a75dec..ae09ab0b4 100644 --- a/tests/by-util/test_ls.rs +++ b/tests/by-util/test_ls.rs @@ -4407,7 +4407,7 @@ fn test_device_number() { let blk_dev_path = blk_dev.path(); let blk_dev_meta = metadata(blk_dev_path.as_path()).unwrap(); let blk_dev_number = blk_dev_meta.rdev() as dev_t; - let (major, minor) = unsafe { (major(blk_dev_number), minor(blk_dev_number)) }; + let (major, minor) = (major(blk_dev_number), minor(blk_dev_number)); let major_minor_str = format!("{major}, {minor}"); let scene = TestScenario::new(util_name!());