Page MenuHomeFreeBSD
Differential D52438

nuageinit: chmod sudoers directory instead of chmod (again) sudoers file
ClosedPublic
Actions

Authored by dtxdf on Sep 8 2025, 10:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 11, 12:56 AM
Unknown Object (File)
Thu, Sep 10, 3:52 PM
Unknown Object (File)
Tue, Sep 8, 11:21 PM
Unknown Object (File)
Tue, Sep 8, 7:53 AM
Unknown Object (File)
Tue, Sep 8, 2:59 AM
Unknown Object (File)
Mon, Sep 7, 3:42 PM
Unknown Object (File)
Wed, Sep 2, 10:45 PM
Unknown Object (File)
Wed, Sep 2, 8:25 PM
View All Files
Subscribers
imp
jlduran

Details

Reviewers
bapt
jlduran
Commits
rG5d8efb8f2578: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file
rGa00410fa19e9: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file
rGa5cc9b7b96dc: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file

Diff Detail

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

Event Timeline

dtxdf created this revision.Sep 8 2025, 10:23 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 8 2025, 10:23 PM
dtxdf requested review of this revision.Sep 8 2025, 10:23 PM
jlduran added a subscriber: jlduran.Sep 9 2025, 4:33 AM
jlduran added inline comments.
libexec/nuageinit/nuage.lua
363

Should also be "0750".

dtxdf updated this revision to Diff 161766.Sep 9 2025, 6:02 AM
  • Set mode of sudoers directory to 0750
dtxdf marked an inline comment as done.Sep 9 2025, 6:03 AM
jlduran accepted this revision.Sep 9 2025, 3:06 PM
jlduran added inline comments.
libexec/nuageinit/nuage.lua
360

Not related to this change, but the permissions for sudoers should be 0440.

This revision is now accepted and ready to land.Sep 9 2025, 3:06 PM
jlduran added a comment.Sep 9 2025, 3:10 PM

For future reviews, please attach a full diff, in order to display the context (or better yet, use git-arc(1)).

bapt accepted this revision.Sep 9 2025, 3:11 PM
dtxdf updated this revision to Diff 161835.Sep 9 2025, 7:26 PM
  • Set mode of sudoers to 0440.
This revision now requires review to proceed.Sep 9 2025, 7:26 PM
jlduran accepted this revision.Sep 9 2025, 8:41 PM
This revision is now accepted and ready to land.Sep 9 2025, 8:41 PM
Closed by commit rGa5cc9b7b96dc: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file (authored by dtxdf). · Explain WhySep 11 2025, 5:08 PM
This revision was automatically updated to reflect the committed changes.
dtxdf added a commit: rGa5cc9b7b96dc: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file.
bapt added a commit: rGa00410fa19e9: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file.Sep 17 2025, 7:34 PM
bapt added a commit: rG5d8efb8f2578: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file.Sep 17 2025, 7:42 PM

Revision Contents
Changeset List

PathSize
libexec/
nuageinit/
4 lines

Diff 161913

View Options

libexec/nuageinit/nuage.lua

Loading...