Ok, so I didn't really get any feedback on this.  I tried it out on linux (ubuntu and centos8) a weekend or so ago and it works well.  So I think I will apply it soon.  It needs libpam-dev or equivalent, but has the great attribute that dtsession no longer needs to be setuid root.

I am going to disable the BSD versions though as they require extra stuff that has to be downloaded and compiled as they do not support PAM natively.  I'll apply it to master, and then at some point in the next weeks I'll merge current master with the autotools branch so it is up to date.

-jon

On 2/3/21 5:40 PM, Jon Trulson wrote:
I'm forwarding this to the list...

The PR looks interesting.  I have not looked at it yet as SF seems to make that more difficult than it should be, but this seems like a very good thing.  I'm just looking for eyeballs and opinions here.

-jon


-------- Forwarded Message --------
Subject:        [cdesktopenv:code] Merge request: Added PAM support to CDE.
Date:   Tue, 02 Feb 2021 17:22:33 -0000
From:   Liang Chang <l-ch...@users.sourceforge.net>
Reply-To: Merge Request #21: Added PAM support to CDE. <nore...@sourceforge.net> To: Merge Request #21: Added PAM support to CDE. <nore...@sourceforge.net>



Merge request: Added PAM support to CDE.

------------------------------------------------------------------------

Liang Chang has requested to merge changes from |ssh://@git.code.sf.net/u/l-chang/cdesktopenv| at commit |[e38633]| into the branch |master|

No need to set SUID on dtsession anymore.
With PAM, a normal user could use his own password to unlock screen.
Tested on FreeBSD, NetBSD and Linux.

For FreeBSD and NetBSD, need to install a PAM module:
https://sourceforge.net/p/cdesktopenv/discussion/general/thread/d8d1781162/#48ae <https://sourceforge.net/p/cdesktopenv/discussion/general/thread/d8d1781162/#48ae>

------------------------------------------------------------------------

Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/cdesktopenv/code/merge-requests/21/ <https://sourceforge.net/p/cdesktopenv/code/merge-requests/21/>

To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/ <https://sourceforge.net/auth/subscriptions/>



_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

--
Jon Trulson

  "Entropy.  It isn't what it used to be."
                           -- Sheldon

_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to