X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=HnwbF2Jt5NCrFJ9a1UE6duOuNYfFqXTGA+IrKqatMs9T52RuiZjj4
	li1cLSsXAcm4WEwZlWFiDba5jr2M6wZKBkXfzubV40YrIpkJEwyYnlv/9RGiAito
	nacLguuT9yx8XytgLnbLENPVgIyks4Tyb2bX5pwfOR7BJ7/+ubojCg=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; s=default;
	bh=S0wC/Fzi05gggO1N8t8lFw3ZFOE=; b=kxX4OJO6PUNi28YnI1tLks6TM6CK
	xdbzUi4BiY3zxR/FBKrt27xRYmzYqlaoDVHCbX6W8/mngka04KzhBodHCeUFik5U
	wUGLceyGaeRl849tss2bKQBI1l7kI4b1sb72Vih84RZte9zHTWv+kObe+JCSKty0
	OmVLV41rvJvJnkQ=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
X-Spam-SWARE-Status:	No, score=-1.8 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.1
Date:	Wed, 12 Jun 2013 19:46:00 +0200
From:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Subject:	[GOLDSTAR] Re: [PATCH] Check for existence of the path before processing '..'
Message-ID:	<20130612174600.GA1736@calimero.vinschen.de>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
References:	<001401ce66a4$bb9029a0$32b07ce0$%fedin AT samsung DOT com>
MIME-Version:	1.0
In-Reply-To:	<001401ce66a4$bb9029a0$32b07ce0$%fedin@samsung.com>
User-Agent:	Mutt/1.5.21 (2010-09-15)

Hi Fedin,

On Jun 11 17:08, Fedin Pavel wrote:
>  Hello!
> 
>  Some time ago i reported ability to access things like
> "/usr/nonexistent/..bin". I still had this problem and i tried my hands on
> fixing it.
>  The patch works by checking the actual existence of the path before
> removing the last component from it. For performance reasons, only one check
> is done for things like "../..". Because, obviously, if "/foo/bar/baz"
> exists, then "/foo/bar" exists too. Also, the check is done only after some
> components have been added to the path. So, for example, current directory
> (obtained when processing relative paths), will not be checked.
>  I tried to add a similar test also to normalize_win32_path() function,
> however this broke things like "cd /usr/src/..". For some reason, a POSIX
> version of the path (but with reversed slashes) is passed to this routine
> when expanding mount points, so, consequently, test for "\usr\src" using
> GetFileType() fails.
>  I think it's ok, at least POSIX paths now behave in POSIX way. I have
> tested against performance, there is some loss (~0.2 seconds), but only for
> referencing '..'.
>  With this patch i am able to compile the latest version of glibc with no
> problems.

I applied your patch with a little stretch of imagination as falling
under the trivial patch rule, which doesn't require a copyright
assignment.  I only tweaked it slightly since I found that moving the
setting of check_parent has a tiny performance advantage.

Cgf and I talked privately about this patch and we're both happy you
found such a simple solution to fix a long-standing problem.  Sometimes,
when you're working long enough on some code, you just miss to see the
wood for the trees.

Andrew, can you please polish one of the goldstar's in your vault and
give it to Fedin?

Btw., Fedin, even if I let this go in under the trivial patch rule, it
would be very nice if you could fill out the Cygwin copyright assignment
form http://cygwin.com/assign.txt and send it to the given address.
This allows you to provide any length of patch in future.


Thanks again,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

- Raw text -