Quoting Bastian Blank ([email protected]): > On Mon, Jan 10, 2011 at 09:14:07PM +0000, Serge E. Hallyn wrote: > > - if (pcred->uid != cred->euid && > > - pcred->euid != cred->euid && !capable(CAP_SYS_NICE)) { > > + if (pcred->user->user_ns != cred->user->user_ns && > > + pcred->uid != cred->euid && > > + pcred->euid != cred->euid && > > + !ns_capable(pcred->user->user_ns, CAP_SYS_NICE)) { > > I don't think this is correct. This would not error out if the both > userns are the same. Because the same patern (check uid if same userns, > otherwise only capability) shows up in several parts of the code, maybe > this should be factored out.
Yeah, I'd really like to factor this out because it shows up everywhere and I have to think about it every time I look at it. But each time it shows up, the uids being compared slightly change. There must be some clever way of doing it, hopefully it'll fall out soon. Eric's ns_capable() has already simplified this quite a bit - which is part of why I've sometimes not been thinking about it right, it's now simpler than it used to be :) thanks, -serge _______________________________________________ Containers mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/containers _______________________________________________ Devel mailing list [email protected] https://openvz.org/mailman/listinfo/devel
