Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751878AbWCAVE5 (ORCPT ); Wed, 1 Mar 2006 16:04:57 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751907AbWCAVE4 (ORCPT ); Wed, 1 Mar 2006 16:04:56 -0500 Received: from xproxy.gmail.com ([66.249.82.198]:36003 "EHLO xproxy.gmail.com") by vger.kernel.org with ESMTP id S1751878AbWCAVE4 (ORCPT ); Wed, 1 Mar 2006 16:04:56 -0500 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:date:from:to:cc:subject:message-id:in-reply-to:references:x-mailer:mime-version:content-type:content-transfer-encoding; b=AydR0uoRUq9HeYNcWojmTsjsRL0qjTK4gdSmBa0N2r9drA7LZPb8FuKa8Th4jXFK1t57nKGgtMTHT2wsqKYICtnhKoGIsx9TvBT8vXV6PKeU26hzAapZBs0lpZhEEkeoTgkxt1TLcjFkrN1qovdvbKt8sQ3kfGitS50F0Z+5d5w= Date: Wed, 1 Mar 2006 18:04:48 -0300 From: Davi Arnaut To: Andrew Morton Cc: dhowells@redhat.com, vsu@altlinux.ru, alan@lxorguk.ukuu.org.uk, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] strndup_user (v3), convert (keyctl) Message-Id: <20060301180448.8afa62f2.davi.arnaut@gmail.com> In-Reply-To: <20060301125053.15a68d59.akpm@osdl.org> References: <20060220170913.b232dc20.davi.arnaut@gmail.com> <4993.1140431092@warthog.cambridge.redhat.com> <20060218161122.f9d588fb.davi.arnaut@gmail.com> <20060218113602.edc06ce5.davi.arnaut@gmail.com> <3487.1140281089@warthog.cambridge.redhat.com> <5378.1140431896@warthog.cambridge.redhat.com> <4967.1141221980@warthog.cambridge.redhat.com> <20060301121651.8e023c8e.davi.arnaut@gmail.com> <20060301125053.15a68d59.akpm@osdl.org> X-Mailer: Sylpheed Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1698 Lines: 51 On Wed, 1 Mar 2006 12:50:53 -0800 Andrew Morton wrote: > Davi Arnaut wrote: > > > > On Wed, 01 Mar 2006 14:06:20 +0000 > > David Howells wrote: > > > > > Davi Arnaut wrote: > > > > > > > In keyctl_keyring_search() there wasn't a check for type[0] == '.', but your > > > > mm-patch added one implicitly. Which one is correct ? > > > > > > Ummm... good point. Key types beginning with a dot are special, and userspace > > > isn't allowed to create them. I'm not sure whether they should be findable or > > > not, but I'm happy go for not at the moment (so the patch is correct, not the > > > original). > > > > > > There's another minor problem with your patch: > > > > > > warthog>grep -r strndup_user * > > > warthog1> > > > > > > I take it that this isn't in Linus's kernel yet... However, I don't want my > > > patch to be held up too much since there are some awkward holes that need > > > fixing. I'm definitely in favour of strndup_user() though. > > > > > > David > > > > Andrew, are you willing to merge for .17 ? > > > > Which, strndup_user()? > > I guess so - would need to see it again. But David's > keys-deal-properly-with-strnlen_user.patch should go first, since it > kinda-fixes things. Ok, I will resend. > I think that's ok with David if we drop it and merge mine. It's much simpler. -- Davi Arnaut - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/