Clippy lints / Correctness
Clippy lint · Correctness · checked with Clippy 0.1.99
clippy::non_octal_unix_permissions: Checks for non-octal values used to set Unix file permissions
What it meansChecks for non-octal values used to set Unix file permissions. It is in the correctness group and denied by default (the build fails).
Does it matter?
They will be converted into octal, creating potentially unintended file permissions.
Code that is almost certainly wrong. Denied by default: the build fails until you fix it or allow it.
How to fix it
Clippy can fix this one for you: run cargo clippy --fix (add --allow-dirty if you have uncommitted changes). Review the diff before you commit.
Flagged
use std::fs::OpenOptions;
use std::os::unix::fs::OpenOptionsExt;
let mut options = OpenOptions::new();
options.mode(644);
Write this instead
use std::fs::OpenOptions;
use std::os::unix::fs::OpenOptionsExt;
let mut options = OpenOptions::new();
options.mode(0o644);
Turn it off, or on
Allow it where it does not fit your code, as narrowly as you can:
On one item
#[allow(clippy::non_octal_unix_permissions)]
fn my_function() { /* ... */ }Whole crate (Cargo.toml)
[lints.clippy]
non_octal_unix_permissions = "allow"One run
cargo clippy -- -A clippy::non-octal-unix-permissionsThe [lints] table needs Rust 1.74 or newer. In a workspace, put it under [workspace.lints.clippy] and add [lints] workspace = true to each member.
Clippy's own documentation
What it does
Checks for non-octal values used to set Unix file permissions.
Why is this bad?
They will be converted into octal, creating potentially unintended file permissions.
Example
use std::fs::OpenOptions;
use std::os::unix::fs::OpenOptionsExt;
let mut options = OpenOptions::new();
options.mode(644);
Use instead:
use std::fs::OpenOptions;
use std::os::unix::fs::OpenOptionsExt;
let mut options = OpenOptions::new();
options.mode(0o644);
From the Clippy lint list (0.1.99), part of the Rust project, MIT or Apache-2.0. Official entry.
Related lints
- modulo_oneChecks for getting the remainder of integer division by one or minus one
- mut_from_refThis lint checks for functions that take immutable references and return mutable ones
- never_loopthis loop never actually loops
- nonsensical_open_optionsChecks for duplicate open options as well as combinations that make no sense
- not_unsafe_ptr_arg_derefthis public function might dereference a raw pointer but is not marked `unsafe`
- option_env_unwrapthis will panic at run-time if the environment variable doesn't exist at compile-time
All 847 Clippy lints · Rust compiler errors · Cargo.toml checker
Up next
Related lintclippy::modulo_oneChecks for getting the remainder of integer division by one or minus oneRead it next →Did this answer your question?