Page MenuHomeFreeBSD
Differential D50710

pw: Skip root check with alternate root
ClosedPublic
Actions

Authored by emaste on Jun 6 2025, 2:06 AM.
Tags
None
Referenced Files
F128683737: D50710.diff
Sat, Sep 13, 8:14 AM
F128670830: D50710.id156605.diff
Sat, Sep 13, 5:11 AM
Unknown Object (File)
Wed, Sep 3, 3:19 PM
Unknown Object (File)
Tue, Sep 2, 3:52 AM
Unknown Object (File)
Mon, Sep 1, 7:01 AM
Unknown Object (File)
Sun, Aug 31, 11:03 AM
Unknown Object (File)
Tue, Aug 26, 10:16 PM
Unknown Object (File)
Tue, Aug 26, 9:24 PM
View All 31 Files
Subscribers
imp
ziaee

Details

Reviewers
markj
allanjude
Commits
rG4a4b3f8c0a99: pw: Skip root check with alternate root
rG142b4309f42d: pw: Skip root check with alternate root
Summary

Skip the you must be root check when -R or -V are specified. pw may be run by an unprivileged user for creating an image or jail. EPERM will still be reported from the open if the user does not have appropriate permission.

Diff Detail

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

Event Timeline

emaste requested review of this revision.Jun 6 2025, 2:06 AM
emaste created this revision.
markj accepted this revision.Jul 14 2025, 8:42 PM

There is a sentence in pw.8 about needing to run as root that should be updated to match the new reality.

This revision is now accepted and ready to land.Jul 14 2025, 8:42 PM
markj added a comment.Mon, Aug 18, 1:54 PM

@emaste will you commit this soon? It's required for rootless release builds.

Closed by commit rG142b4309f42d: pw: Skip root check with alternate root (authored by emaste). · Explain WhyTue, Aug 19, 12:51 PM
This revision was automatically updated to reflect the committed changes.
emaste added a commit: rG142b4309f42d: pw: Skip root check with alternate root.
Herald added subscribers: ziaee, imp. · View Herald TranscriptTue, Aug 19, 12:51 PM
emaste added a commit: rG4a4b3f8c0a99: pw: Skip root check with alternate root.Wed, Sep 10, 3:03 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
pw/
9 lines
1 line
18 lines
1 line

Diff 160590

View Options

usr.sbin/pw/pw.8

Loading...
View Options

usr.sbin/pw/pw.c

Loading...
View Options

usr.sbin/pw/pw_user.c

Loading...
View Options

usr.sbin/pw/pwupd.h

Loading...