SteelDynamite/onyx-tasks
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

fix: correct operator precedence in Windows path validation

Browse source 
The condition `len <= 3 && ends_with(":\\") || ends_with(":")` was
missing parentheses, causing the second ends_with check to run
regardless of path length due to && binding tighter than ||.

https://claude.ai/code/session_013ooJht2HrZUTXgNJFU79cV
This commit is contained in:
Claude 2026-04-16 07:21:16 +00:00
parent 24a62b6685
commit ac72955d23
No known key found for this signature in database
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
apps/tauri/src-tauri/src/lib.rs
View file

@ -87,7 +87,7 @@ fn validate_workspace_path(path: &str) -> Result<(), String> {
#[cfg(windows)] #[cfg(windows)]
{ {
let upper = normalized.to_uppercase(); let upper = normalized.to_uppercase();
if upper.len() <= 3 && upper.ends_with(":\\") || upper.ends_with(":") { if upper.len() <= 3 && (upper.ends_with(":\\") || upper.ends_with(":")) {
return Err(format!("Cannot use drive root as workspace: {}", path)); return Err(format!("Cannot use drive root as workspace: {}", path));
} }
} }