From a9494e84d02518646d0a072e1b4f4396687dc1e1 Mon Sep 17 00:00:00 2001 From: vasilito Date: Mon, 27 Jul 2026 21:25:50 +0900 Subject: [PATCH] wifictl: fix backend.rs set_mode call split by SAFETY comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Commit 222d5186eb ('add minimal # Safety comments to 70 files') inserted a '// SAFETY: ...' line mid-token in backend.rs:1405, splitting 'perms.set_mode(0o755);' into 'perms.set_mo' + comment + 'de(0o755);'. The result is a compilation error that prevents the entire redbear-wifictl crate from building — including the new dbus_nm.rs interface from e65a23fd6b. Restore the single-line call. The Safety rationale is captured by the surrounding cfg(unix) block. Verified: cargo check --features dbus-nm compiles, 35 unit tests + 2 cli_transport pass. --- local/recipes/system/redbear-wifictl/source/src/backend.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/local/recipes/system/redbear-wifictl/source/src/backend.rs b/local/recipes/system/redbear-wifictl/source/src/backend.rs index 5ff7ec1b62..e5a5aa3d27 100644 --- a/local/recipes/system/redbear-wifictl/source/src/backend.rs +++ b/local/recipes/system/redbear-wifictl/source/src/backend.rs @@ -1402,8 +1402,7 @@ esac { use std::os::unix::fs::PermissionsExt; let mut perms = fs::metadata(&driver).unwrap().permissions(); - perms.set_mo// SAFETY: caller must verify the safety contract for this operation -de(0o755); + perms.set_mode(0o755); fs::set_permissions(&driver, perms).unwrap(); }