On Tue, Sep 27, 2011 at 07:41:57AM -0500, Serge E. Hallyn wrote: > Quoting Greg KH (greg@xxxxxxxxx): > > On Mon, Sep 26, 2011 at 10:45:18AM -0500, Serge Hallyn wrote: > > > Add to the dev_state and alloc_async structures the user namespace > > > corresponding to the uid and euid. Pass these to kill_pid_info_as_uid(), > > > which can then implement a proper, user-namespace-aware uid check. > > > > > > Changelog: > > > Sep 20: Per Oleg's suggestion: Instead of caching and passing user namespace, > > > uid, and euid each separately, pass a struct cred. > > > Sep 26: Address Alan Stern's comments: don't define a struct cred at > > > usbdev_open(), and take and put a cred at async_completed() to > > > ensure it lasts for the duration of kill_pid_info_as_cred(). > > > > > > Signed-off-by: Serge Hallyn <serge.hallyn@xxxxxxxxxxxxx> > > > Cc: Greg KH <greg@xxxxxxxxx> > > > > I have no objection to this, is it going to go through your tree, or > > somewhere else? > > (Silly question from me, but just to make sure - were you asking this of > Alan?) Nope, you. Do you have a tree for this type of namespace work? I haven't been paying attention to it at all. If not, I'll gladly take it myself, I just don't want to cause any merge conflicts anywhere else if at all possible. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html