Mailing-List: contact cygwin-developers-help AT cygwin DOT com; run by ezmlm List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-developers-owner AT cygwin DOT com Delivered-To: mailing list cygwin-developers AT cygwin DOT com Message-Id: <3.0.5.32.20030204140933.007f8880@h00207811519c.ne.client2.attbi.com> X-Sender: pierre AT h00207811519c DOT ne DOT client2 DOT attbi DOT com Date: Tue, 04 Feb 2003 14:09:33 -0500 To: cygwin-developers AT cygwin DOT com From: "Pierre A. Humblet" Subject: Re: [Bob DOT Burger AT sagian DOT com: 1.3.19: Everyone has write access to .lnk symbolic link files] In-Reply-To: <20030204173153.GH5822@cygbert.vinschen.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" At 06:31 PM 2/4/2003 +0100, Corinna Vinschen wrote: >According to the behaviour on Linux, the symlink should always have >permissions like "rw-r--r--" and only stat() returns "rwxrwxrwx". The >change is very easy > > { > /* symlinks are anything for everyone! */ > if ((attribute & S_IFLNK) == S_IFLNK) > - attribute |= S_IRWXU | S_IRWXG | S_IRWXO; > + attribute = STD_RBITS | STD_WBITS; Those two lines can actually go away completely because attribute is already set in path.cc . It should be changed there instead. >and I actually doubt that it will negatively affect Cygwin but I'm not >100% sure if I'm missing something. > >What do you think? If you are missing something, so am I. Pierre