From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subrata Modak Subject: Re: [LTP] [Bugme-new] [Bug 15909] New: open("a/", O_NOFOLLOW) fails with ELOOP if "a" is a symbolic link to a directory. Date: Tue, 11 May 2010 22:05:52 +0530 Message-ID: <1273595753.4875.1.camel@subratamodak.linux.ibm.com> References: <20100506143002.0381501b.akpm@linux-foundation.org> <20100511154850.GC2832@quack.suse.cz> <20100511162452.GD2832@quack.suse.cz> Reply-To: subrata@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: ltp-list@lists.sourceforge.net, tolzmann@molgen.mpg.de, bugzilla-daemon@bugzilla.kernel.org, Al Viro , bugme-daemon@bugzilla.kernel.org, linux-fsdevel@vger.kernel.org, Andrew Morton , Christoph Hellwig To: Jan Kara Return-path: Received: from e3.ny.us.ibm.com ([32.97.182.143]:40352 "EHLO e3.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751252Ab0EKQgS (ORCPT ); Tue, 11 May 2010 12:36:18 -0400 Received: from d01relay07.pok.ibm.com (d01relay07.pok.ibm.com [9.56.227.147]) by e3.ny.us.ibm.com (8.14.3/8.13.1) with ESMTP id o4BGNXao000866 for ; Tue, 11 May 2010 12:23:33 -0400 Received: from d01av03.pok.ibm.com (d01av03.pok.ibm.com [9.56.224.217]) by d01relay07.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id o4BGa2aA1613876 for ; Tue, 11 May 2010 12:36:02 -0400 Received: from d01av03.pok.ibm.com (loopback [127.0.0.1]) by d01av03.pok.ibm.com (8.14.3/8.13.1/NCO v10.0 AVout) with ESMTP id o4BGZxn7014134 for ; Tue, 11 May 2010 13:36:01 -0300 In-Reply-To: <20100511162452.GD2832@quack.suse.cz> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Hi Jan, On Tue, 2010-05-11 at 18:24 +0200, Jan Kara wrote: > On Tue 11-05-10 17:48:51, Jan Kara wrote: > > On Thu 06-05-10 14:30:02, Andrew Morton wrote: > > > > > > (switched to email. Please respond via emailed reply-to-all, not > via the > > > bugzilla web interface). > > > > > > On Wed, 5 May 2010 13:01:22 GMT > > > bugzilla-daemon@bugzilla.kernel.org wrote: > > > > > > > https://bugzilla.kernel.org/show_bug.cgi?id=15909 > > > > > > > > Summary: open("a/",O_NOFOLLOW) fails with ELOOP if > "a" is a > > > > symbolic link to a directory. > > > > Product: File System > > > > Version: 2.5 > > > > Kernel Version: 2.6.34-rc6 > > > > Platform: All > > > > OS/Version: Linux > > > > Tree: Mainline > > > > Status: NEW > > > > Severity: high > > > > Priority: P1 > > > > Component: Other > > > > AssignedTo: fs_other@kernel-bugs.osdl.org > > > > ReportedBy: tolzmann@molgen.mpg.de > > > > Regression: No > > > > > > > > > > > > mkdir c > > > > ln -s c a > > > > > > > > f=open("a/",O_RDONLY+O_NOFOLLOW) > > > > > > > > fails with ELOOP. However, this open should behave like > open("a/.") not like > > > > open("a") according to path_resolution(7). In kernel version > 2.6.32 the open > > > > worked as documented. > > > > > > > > On a higher level this bug makes > > > > > > > > find a/ > > > > > > > > to fail. > > > > > > > > > > It sounds like this 2.6.32->2.6.34-rc6 regression could have > pretty > > > serious ramifications for some users. Does anyone know whcih > commit > > > might have caused it? > > The patch below fixes the issue for me but someone should have a > look > > at it because I'm not really an expert in that code and the code > paths are so > > twisted that my mind is currently tied into a knot ;). > > > > Honza > > --- > > > > From d53d3cc6488d9135bb69c3ff7e034b3b624866ed Mon Sep 17 00:00:00 > 2001 > > From: Jan Kara > > Date: Tue, 11 May 2010 16:34:25 +0200 > > Subject: [PATCH] vfs: Fix O_NOFOLLOW behavior for paths with > trailing slashes > > > > According to specification > > mkdir d; ln -s d a; open("a/", O_NOFOLLOW | O_RDONLY) > > should return success but currently it did return ELOOP. Fix the > code to ignore > > O_NOFOLLOW in case the provided path has trailing slashes. This is a > regression > > caused by path lookup cleanup patch series. > > > > CC: stable@kernel.org > > Signed-off-by: Jan Kara > ... > BTW: It might be worthwhile to add the attached testcase to LTP? It would be great if you can shoot it in the form of a patch, mentioning 1. The GPL Lincense, 2. The purpose of the test case, 3. Exact location it goes, 4. Which LTPROOT/runtest/ executes it Regards-- Subrata :-) > > Honza > -- > Jan Kara > SUSE Labs, CR > > > > > > > > C++ source code > attachment > (open-follow-test.c) > > #define _GNU_SOURCE > #include > #include > #include > #include > #include > #include > #include > > int do_test(char *p) > { > int err; > char path[16]; > > strcpy(path, p); > err = open(path, O_NOFOLLOW | O_RDONLY); > if (err >= 0) { > fprintf(stderr, "open(\"%s\", O_NOFOLLOW | O_RDONLY) did not fail! > \n", path); > return 1; > } > strcat(path, "/"); > err = open(path, O_NOFOLLOW | O_RDONLY); > if (err < 0) { > fprintf(stderr, "open(\"%s\", O_NOFOLLOW | O_RDONLY) failed with % > d\n", path, errno); > return 1; > } > strcat(path, "."); > err = open(path, O_NOFOLLOW | O_RDONLY); > if (err < 0) { > fprintf(stderr, "open(\"%s\", O_NOFOLLOW | O_RDONLY) failed with % > d\n", path, errno); > return 1; > } > return 0; > } > > int main(void) > { > if (mkdir("d", 0700) < 0) { > perror("mkdir"); > return 1; > } > if (symlink("d", "a") < 0) { > perror("link"); > return 1; > } > if (do_test("a")) > return 1; > return 0; > }