uutils / uutils/coreutils

mv: two self-referential symlinks cause very long runtime

Open
#14,285 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

U - mv
Dominant language
Rust
Stars
24.1k
Forks
2k
Avg merge
1d 5h
Merged PRs (30d)
365

Description

coreutils 0.11.0 (multi-call binary)
commit 4a41d0ba51f (latest)

Commit 175cfdc (mv: don't put a symlink in its target's hardlink group) causes mv to burn hours of cpu cycles:

mkdir -p /tmp/poc/d /tmp/dest
ln -s . /tmp/poc/d/l1
ln -s . /tmp/poc/d/l2
mv /tmp/poc/d /tmp/dest/

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the issue with the mkdir, symlink, and mv commands shown in the report, then inspect the mv implementation and the handling introduced by commit 175cfdc. Trace why two self-referential symlinks keep the operation running and add a regression test for this reproduction. Done means mv terminates promptly without the reported hours of CPU use.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.