Re: Red Hat's passwd

[Joshua Brindle <jbrindle@tresys.com>]

Colin Walters wrote:

>On Fri, 2005-08-05 at 15:07 -0400, Joshua Brindle wrote:
>
>  
>
>>Sounds good to me, trivial patch to do this.. Also, I assume you will 
>>change the previous patch for passwd.c to the new function.
>>    
>>
>
>Shouldn't we also remove checkPasswdAccess from the header?  Otherwise
>what's the point?
>
>  
>
no but we need to add the other..

The point is to allow some transition time so that both are available 
until everything is fixed, then we can remove the other without having 
any problems.

--- selinux.h   (revision 920)
+++ selinux.h   (working copy)
@@ -290,6 +290,7 @@

 /* Check a permission in the passwd class.
    Return 0 if granted or -1 otherwise. */
+extern int selinux_check_passwd_access(access_vector_t requested);
 extern int checkPasswdAccess(access_vector_t requested);

 /* Set the path to the selinuxfs mount point explicitly.

>I should note this approach addresses the consistency and namespacing
>from the developer point of view, but doesn't address possible symbol
>collisions.  Possibly that could wait until a soname bump, just thought
>I'd point it out.
>  
>


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.