Page MenuHomeFreeBSD
Differential D50984

cp: Fix improper use of O_PATH.
ClosedPublic
Actions

Authored by des on Jun 23 2025, 8:23 PM.
Tags
None
Referenced Files
F151663513: D50984.id157464.diff
Thu, Apr 9, 8:55 PM
Unknown Object (File)
Thu, Apr 9, 2:50 AM
Unknown Object (File)
Sun, Apr 5, 1:25 AM
Unknown Object (File)
Sat, Apr 4, 11:43 AM
Unknown Object (File)
Fri, Apr 3, 12:59 AM
Unknown Object (File)
Tue, Mar 31, 12:00 PM
Unknown Object (File)
Tue, Mar 31, 4:19 AM
Unknown Object (File)
Thu, Mar 19, 2:22 AM
View All 73 Files
Subscribers
imp

Details

Reviewers
kevans
Group Reviewers
Klara
Commits
rG0e0df1fc6753: cp: Fix improper use of O_PATH.
Summary

This does not appear to make any practical difference at the moment, but
technically O_PATH means “I'm not going to use this descriptor for any
other purposes than vnode lookups”, so using it to read the directory's
ACLs is improper and might fail in the future.

Fixes: 82fc0d09e8625
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 65032
Build 61915: arc lint + arc unit

Revision Contents
Changeset List

PathSize
bin/
cp/
tests/
18 lines
2 lines

Diff 157464

View Options

bin/cp/tests/cp_test.sh

Loading...
View Options

bin/cp/utils.c

Loading...