linux-kernel - Re: [PATCH 6/7] autofs4 - use struct qstr in waitq.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 23 Jul 2008 13:08:40 +0800
From:	Ian Kent <raven@...maw.net>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	linux-kernel@...r.kernel.org, autofs@...ux.kernel.org,
	linux-fsdevel@...r.kernel.org, Al Viro <viro@...iv.linux.org.uk>,
	Jeff Moyer <jmoyer@...hat.com>
Subject: Re: [PATCH 6/7] autofs4 - use struct qstr in waitq.c

On Tue, 2008-07-22 at 16:13 -0700, Andrew Morton wrote:
> On Tue, 17 Jun 2008 20:24:06 +0800
> Ian Kent <raven@...maw.net> wrote:
> 
> > From: Jeff Moyer <jmoyer@...hat.com>
> > 
> > The autofs_wait_queue already contains all of the fields of the
> > struct qstr, so change it into a qstr.
> > 
> > This patch, from Jeff Moyer, has been modified a liitle by myself.
> > 
> > Signed-off-by: Jeff Moyer <jmoyer@...hat.com>
> > Signed-off-by: Ian Kent <raven@...maw.net>
> 
> So this patch which had been happily sitting in -mm for a month has
> suddenly broken because linux-next's three-day-old
> 4bce7ce7c7d0d57b78dacc3a2bd87ec63b2d9b4c has removed LOOKUP_ACCESS.
> 
> This is suboptimal.
> 
> Now what do I do?

Ummm .. I'm confused.

Your patch autofs4-use-lookup-intent-flags-to-trigger-mounts-fix.patch
allows the linux-next kernel to build with all the autofs4 patches
currently posted for inclusion in mm but the patch you mention here
isn't concerned with the lookup flags?

The removal of LOOKUP_ACCESS is quite interesting. AFAIKS it effectively
prevents the patch
autofs4-use-lookup-intent-flags-to-trigger-mounts.patch from also
resolving an issue with recursive autofs mounts while still resolving
the issue that the patch was actually meant to address.

It's hard to get exited about the former issue as Al Viro has NACKed a
previous patch that added the LOOKUP_ACCESS check, indicating the
availability of the lookup flags will be changing. Also there is a
question as to whether autofs will support the use mount points in
automount maps that themselves refer to an automount path (the recursive
bit).

Ian

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/