Re: PAM authentication patch - v2
Brian Murphy <[email protected]>
| Newsgroups | gmane.comp.version-control.cvs.bugs |
|---|---|
| Message-ID | <[email protected]> |
Max Bowsher wrote: >No, but is there any functional benefit from ignoring the warnings? > > Yes. It gives the flexibility that making a soft link to the cvs binary gives a new pam configuration. SSH is also configured this way. >The warnings: >http://www.kernel.org/pub/linux/libs/pam/Linux-PAM-html/pam_appl-4.html#ss4. >2 > >They are quite emphatic about it. > >Since there are valid reasons to run cvs setuid or setgid, is this worth the >risk? > How many people actually use cvs this way? There is nothing in the documentation saying how to use cvs suid and in fact recommends to use pserver to do the things you might want to do via an suid binary. >If you do decide it is worth the risk, I'm sure cautious sysadmins would >appreciate a configure option to force hardcoding. > > > I will do this, I would also prefer to have this option. /Brian