Skip to content

Commit

Permalink
Merge pull request FRRouting#15163 from opensourcerouting/fix/pam_acc…
Browse files Browse the repository at this point in the history
…ount

packaging: Require account validation with pam_unix.so if PAM enabled
  • Loading branch information
Jafaral authored Jan 30, 2024
2 parents d633a81 + e68c4f0 commit 471e4b7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion debian/frr.pam
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Any user may call vtysh but only those belonging to the group frrvty can
# actually connect to the socket and use the program.
auth sufficient pam_permit.so
account sufficient pam_rootok.so
account sufficient pam_permit.so
4 changes: 2 additions & 2 deletions redhat/frr.pam
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@
##### if running frr as root:
# Only allow root (and possibly wheel) to use this because enable access
# is unrestricted.
auth sufficient pam_rootok.so
account sufficient pam_rootok.so
auth sufficient pam_permit.so
account sufficient pam_permit.so

# Uncomment the following line to implicitly trust users in the "wheel" group.
#auth sufficient pam_wheel.so trust use_uid
Expand Down

0 comments on commit 471e4b7

Please sign in to comment.