Skip to content

Commit

Permalink
pam: debian: Explicitly include pam_limits for xrdp sessions
Browse files Browse the repository at this point in the history
This is required, as on Debian, pam_limits is not automatically included
for common sessions, and only for "login" sessions and session managers.
So, xrdp has to include it explicitly to make limits take effect.

See https://wiki.debian.org/Limits for details.

(cherry picked from commit f581d8e)
  • Loading branch information
ximion authored and matt335672 committed Dec 17, 2024
1 parent 153e67f commit 98cd5b4
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions instfiles/pam.d/xrdp-sesman.debian
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ auth required pam_env.so readenv=1 envfile=/etc/default/locale

@include common-password

# Ensure resource limits are applied
session required pam_limits.so
@include common-session
-session optional pam_gnome_keyring.so auto_start
-session optional pam_kwallet5.so auto_start

0 comments on commit 98cd5b4

Please sign in to comment.