system/coreutils
system/coreutils
1
0
Fork 0
mirror of https://github.com/uutils/coreutils synced 2024-07-22 18:34:12 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #5372 from sanpii/ls-panic

Browse source 
ls: fix panic when file removed too quickly
This commit is contained in:
Sylvestre Ledru 2023-10-08 10:54:19 +02:00 committed by GitHub
parent 90b3f6bffb e1b7f254a6
commit c53ca6d6fc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/uu/ls/src/ls.rs
View file

@ -2889,7 +2889,11 @@ fn classify_file(path: &PathData, out: &mut BufWriter<Stdout>) -> Option<char> {
Some('=')
} else if file_type.is_fifo() {
Some('|')
} else if file_type.is_file() && file_is_executable(path.md(out).as_ref().unwrap()) {
} else if file_type.is_file()
// Safe unwrapping if the file was removed between listing and display
// See https://github.com/uutils/coreutils/issues/5371
&& path.md(out).map(file_is_executable).unwrap_or_default()
{
Some('*')
} else {
None