From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:57300 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755075Ab1FQJ5F (ORCPT ); Fri, 17 Jun 2011 05:57:05 -0400 From: David Howells In-Reply-To: References: <20110616235600.14345.16839.stgit@warthog.procyon.org.uk> To: Linus Torvalds Cc: dhowells@redhat.com, akpm@linux-foundation.org, jmorris@namei.org, shirishpargaonkar@gmail.com, keyrings@linux-nfs.org, linux-nfs@vger.kernel.org, linux-cifs@vger.kernel.org, Eric Paris Subject: Re: [PATCH] KEYS/DNS: Fix ____call_usermodehelper() to not lose the session keyring Date: Fri, 17 Jun 2011 10:56:51 +0100 Message-ID: <23040.1308304611@redhat.com> Sender: linux-nfs-owner@vger.kernel.org List-ID: Content-Type: text/plain MIME-Version: 1.0 Linus Torvalds wrote: > PS. Possibly it might make more sense to just put the "struct cred *" > pointer into the "struct subprocess_info" and not change any of the > function prototypes at all? That's the way I did it originally, but someone thought doing it with an init function was a better idea:-/ Your suggestion is slightly complicated by the capability restriction stuff that caused the breakage. That needs to hack the creds at some point. David