pw: Skip root check with alternate root
ClosedPublic
Actions

Authored by emaste on Jun 6 2025, 2:06 AM.

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

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste requested review of this revision.Jun 6 2025, 2:06 AM

emaste created this revision.

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

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

Closed by commit rG142b4309f42d: pw: Skip root check with alternate root. · Explain WhyAug 19 2025, 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 TranscriptAug 19 2025, 12:51 PM

emaste added a commit: rG4a4b3f8c0a99: pw: Skip root check with alternate root.Sep 10 2025, 3:03 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

pw/

pw.c

1 line

pw_user.c

18 line(s)

pwupd.h

1 line

Diff 156605

View Options

usr.sbin/pw/pw.c

View Options

usr.sbin/pw/pw_user.c

View Options

pw: Skip root check with alternate rootClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 156605

usr.sbin/pw/pw.c

usr.sbin/pw/pw_user.c

usr.sbin/pw/pwupd.h

pw: Skip root check with alternate root
ClosedPublic
Actions

Revision Contents
Changeset List