8000 stat: panic on calling with `-` and an open pipe/fifo · Issue #3485 · uutils/coreutils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

stat: panic on calling with - and an open pipe/fifo #3485

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
jhscheer opened this issue May 5, 2022 · 0 comments · Fixed by #3498
Closed

stat: panic on calling with - and an open pipe/fifo #3485

jhscheer opened this issue May 5, 2022 · 0 comments · Fixed by #3498
Labels
U - stat

Comments

@jhscheer
Copy link
Contributor
jhscheer commented May 5, 2022

The workaround from #3280 does not distinguish between calling stat - with an open pipe/fifo or without.

GNU stat:

 $ stat -
  File: -
  Size: 0               Blocks: 0          IO Block: 1024   character special file
$ echo | stat -
  File: -
  Size: 0               Blocks: 0          IO Block: 4096   fifo

uu_stat:

$ target/debug/stat -
  File: /dev/pts/7
  Size: 0               Blocks: 0          IO Block: 1024   character special file
$ echo | target/debug/stat -
thread 'main' panicked at 'unable to canonicalize /dev/stdin: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/uu/stat/src/stat.rs:482:18
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
jhscheer added a commit to jhscheer/coreutils that referenced this issue May 5, 2022
@jhscheer
test_stat: add tests for issues with stdin
776f3ae 
* add tests for: uutils#3485
* add test for: uutils#3280
@jhscheer jhscheer mentioned this issue May 5, 2022
Merged
jhscheer added a commit to jhscheer/coreutils that referenced this issue 8000 May 5, 2022
@jhscheer
test_stat: add tests for issues with stdin
f5ffa94 
* add tests for: uutils#3485
* add test for: uutils#3280
jhscheer added a commit to jhscheer/coreutils that referenced this issue May 6, 2022
@jhscheer
stat: improve handling of stdin/fifo (fix uutils#3485)
d4dc6ef 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
@jhscheer jhscheer mentioned this issue May 6, 2022
Merged
jhscheer added a commit to jhscheer/coreutils that referenced this issue May 6, 2022
@jhscheer
stat: improve handling of stdin/fifo (fix uutils#3485)
02c9c56 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
jhscheer added a commit to jhscheer/coreutils that referenced this issue May 6, 2022
@jhscheer
stat: improve handling of stdin/fifo (fix uutils#3485)
cba5205 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
jhscheer added a commit to jhscheer/coreutils that referenced this issue May 6, 2022
@jhscheer
stat: improve handling of stdin/fifo (fix uutils#3485)
a0d9709 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
jhscheer 8000 added a commit to jhscheer/coreutils that referenced this issue May 6, 2022
@jhscheer
stat: improve handling of stdin/fifo (fix uutils#3485)
0b774c5 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
jhscheer added a commit to jhscheer/coreutils that referenced this issue May 7, 2022
@jhscheer
stat: improve handling of stdin/fifo (fix uutils#3485)
79e5d80 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
sylvestre pushed a commit to jhscheer/coreutils that referenced this issue May 11, 2022
@jhscheer @sylvestre
stat: improve handling of stdin/fifo (fix uutils#3485)
d906f09 
* fix uutils#3485
* improve the workaround from uutils#3280
* add tests
sylvestre added a commit that referenced this issue May 12, 2022
@sylvestre
Merge pull request #3498 from jhscheer/fix_stat_redirect
6a9a7d7 
stat: improve handling of stdin/fifo (fix #3485)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
U - stat
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

stat: improve handling of stdin/fifo (fix #3485) jhscheer/coreutils
2 participants
@jfinkels @jhscheer
0