Page MenuHomeFreeBSD

cp: Never follow symbolic links in destination.
ClosedPublic

Authored by des on Apr 1 2024, 1:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 2:39 AM
Unknown Object (File)
Mon, Nov 11, 4:37 PM
Unknown Object (File)
Mon, Nov 11, 2:47 PM
Unknown Object (File)
Mon, Nov 11, 8:55 AM
Unknown Object (File)
Sat, Nov 9, 4:43 PM
Unknown Object (File)
Thu, Nov 7, 11:15 PM
Unknown Object (File)
Thu, Nov 7, 11:01 PM
Unknown Object (File)
Thu, Nov 7, 10:59 PM
Subscribers

Details

Summary

Historically, BSD cp has followed symbolic links in the destination
when copying recursively, while GNU cp has not. POSIX is somewhat
vague on the topic, but both interpretations are within bounds. In
33ad990ce974, cp was changed to apply the same logic for symbolic
links in the destination as for symbolic links in the source: follow
if not recursing (which is moot, as this situation can only arise
while recursing) or if the -L option was given. There is no support
for this in POSIX. We can either switch back, or go all the way.

Having carefully weighed the kind of trouble you can run into by
following unexpected symlinks up against the kind of trouble you can
run into by not following symlinks you expected to follow, we choose
to go all the way.

Note that this means we need to stat the destination twice: once,
following links, to check if it is or references the same file as the
source, and a second time, not following links, to set the dne flag
and determine the destination's type.

While here, remove a needless complication in the dne logic. We don't
need to explicitly reject overwriting a directory with a non-directory,
because it will fail anyway.

Finally, add test cases for copying a directory to a symlink and
overwriting a directory with a non-directory.

MFC after: never
Relnotes: yes
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des requested review of this revision.Apr 1 2024, 1:21 PM
This revision is now accepted and ready to land.Apr 1 2024, 2:28 PM

fix “same file” logic, add tests

This revision now requires review to proceed.Apr 1 2024, 3:14 PM
des edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Apr 1 2024, 3:29 PM
This revision was automatically updated to reflect the committed changes.