Page MenuHomeFreeBSD
Differential D36688

shutdown.8: Add a note about needed priviledges to run the command
ClosedPublic
Actions

Authored by gbe on Sep 24 2022, 9:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 3:54 PM
Unknown Object (File)
Fri, Nov 1, 2:31 AM
Unknown Object (File)
Sep 29 2024, 2:26 AM
Unknown Object (File)
Sep 29 2024, 2:25 AM
Unknown Object (File)
Sep 29 2024, 2:24 AM
Unknown Object (File)
Sep 29 2024, 12:23 AM
Unknown Object (File)
Sep 26 2024, 12:51 PM
Unknown Object (File)
Sep 24 2024, 9:11 AM
View All 64 Files
Subscribers
cy
grahamperrin
imp
pauamma_gundo.com

Details

Reviewers
pauamma_gundo.com
Group Reviewers
manpages
Commits
rG572cb0ae4680: shutdown.8: Add a note about needed priviledges to run the command
rG604610253c0c: shutdown.8: Add a note about needed priviledges to run the command
rGe0dfa1c4c45e: shutdown.8: Add a note about needed priviledges to run the command
Summary

In order to use the shutdown command the user must have root
privileges or have a membership in the operator group.

PR: 266525
Reported by: Zsolt Udvari <uzsolt at uzsolt hu>
MFC after: 1 week

Test Plan

mandoc output review and 'mandoc -Tlint' checks

Diff Detail

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

Event Timeline

gbe created this revision.Sep 24 2022, 9:01 AM
Herald added a subscriber: imp. · View Herald TranscriptSep 24 2022, 9:01 AM
gbe requested review of this revision.Sep 24 2022, 9:01 AM
Herald added a subscriber: cy. · View Herald TranscriptSep 24 2022, 9:01 AM
Harbormaster completed remote builds in B47514: Diff 110911.Sep 24 2022, 9:02 AM
gbe edited the summary of this revision. (Show Details)Sep 24 2022, 9:02 AM
grahamperrin added a subscriber: grahamperrin.Sep 24 2022, 11:25 AM
pauamma_gundo.com requested changes to this revision.Sep 24 2022, 8:36 PM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.

Other than that wording simplification, LGTM.

sbin/shutdown/shutdown.8
61–62

Unless you mean something different by "have root privileges".

This revision now requires changes to proceed.Sep 24 2022, 8:36 PM
gbe updated this revision to Diff 112670.Nov 6 2022, 10:27 AM
  • reword sentence
Harbormaster completed remote builds in B48184: Diff 112670.Nov 6 2022, 10:27 AM
gbe marked an inline comment as done.Nov 6 2022, 10:28 AM

Reword sentence.

sbin/shutdown/shutdown.8
61–62

Unless you mean something different by "have root privileges".

'root privileges' is the better term here, since everything uid = 0 could be root, no matter what the username is. These privileges could also be obtained via doas or sudo.

pauamma_gundo.com accepted this revision.Nov 6 2022, 12:38 PM

Fixable on commit.

sbin/shutdown/shutdown.8
61
This revision is now accepted and ready to land.Nov 6 2022, 12:38 PM
Closed by commit rGe0dfa1c4c45e: shutdown.8: Add a note about needed priviledges to run the command (authored by gbe). · Explain WhyNov 7 2022, 11:01 AM
This revision was automatically updated to reflect the committed changes.
gbe marked an inline comment as done.
gbe added a commit: rGe0dfa1c4c45e: shutdown.8: Add a note about needed priviledges to run the command.
gbe added a commit: rG604610253c0c: shutdown.8: Add a note about needed priviledges to run the command.Nov 14 2022, 7:54 AM
gbe added a commit: rG572cb0ae4680: shutdown.8: Add a note about needed priviledges to run the command.Nov 14 2022, 7:59 AM

Revision Contents
Changeset List

PathSize
sbin/
shutdown/
6 lines

Diff 112717

View Options

sbin/shutdown/shutdown.8

Loading...