[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Fix for issue 3813

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 22 Jun 2011 18:08:40 +0300

Greg Hudson wrote on Wed, Jun 22, 2011 at 11:01:05 -0400:
> On Wed, 2011-06-22 at 02:29 -0400, Daniel Shahaf wrote:
> > From looking at the code, svn_io_open_unique_file3() would force the
> > file to have a mode of 0600|umask() instead of just 0600
> The umask removes file permissions from the mode argument to open(); it
> doesn't add permissions. (Unless there's something unusual about this
> code.)

The comments in the code are of the form "This is a hack to make the
file created with mode 0644 rather than 0600 as it was without/before
this hack". My '0600|umask()' syntax is inaccurate.

(It's 0600|foo(), but foo() != umask().)
Received on 2011-06-22 17:09:25 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.