Re: cd -s symlink hangs (sometimes?)

[Mikael Magnusson <mikachu@xxxxxxxxx>]

2009/3/20 Peter Stephenson <p.w.stephenson@xxxxxxxxxxxx>:
> On Fri, 20 Mar 2009 22:12:30 +0100
> Mikael Magnusson <mikachu@xxxxxxxxx> wrote:
>> zsh -f
>> % mkdir a
>> % ln -s a b
>> % cd -s b
>> cd: not a directory: b
>> % cd -s b
>> # kill -9 in another terminal
>> zsh: killed     zsh -f
>
> The following is reproducible (note I didn't actually create the
> directory a):
>
> % zsh -f
> % cd ~
> % pwd
> /home/pws
> % rm -f a b
> % ln -s a b
> % cd -s b
> cd: not a directory: b
> % /bin/pwd
> /

At first I thought you misunderstood my problem, since the above seems
like a different problem, but the patch does fix my infinite loop.

Just for fun I tried mkdir a; cd a; rmdir ../a; cd -s ../symlink, and
it didn't break either.

[5 minutes later]

For some more fun I tried this, and it does still break:
mkdir a
ln -s a b
cd a
chmod -x .
cd -s $PWD:h/b
#infinite loop here still
chdir("../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../../..")
= 0

I think this is related to .. not being accessible when . is -x? I
didn't study the code very closely (ie, I only read the patch
context). I should also note that this is not something I tend to do
very often, I was just thinking up ways in which the code might still
fail.

-- 
Mikael Magnusson