diff options
author | Junio C Hamano <junkio@cox.net> | 2006-06-09 22:07:23 -0700 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-06-09 22:15:50 -0700 |
commit | 138086a72582620e0b3ec0aa60a78da208e89a2a (patch) | |
tree | ba4cb91659773ed349464247e93a765a10999ac6 /sha1_file.c | |
parent | f6407823d1467a6b6d2002878a64dec42df0a361 (diff) | |
download | git-138086a72582620e0b3ec0aa60a78da208e89a2a.tar.gz |
shared repository - add a few missing calls to adjust_shared_perm().
There were a few calls to adjust_shared_perm() that were
missing:
- init-db creates refs, refs/heads, and refs/tags before
reading from templates that could specify sharedrepository in
the config file;
- updating config file created it under user's umask without
adjusting;
- updating refs created it under user's umask without
adjusting;
- switching branches created .git/HEAD under user's umask
without adjusting.
This moves adjust_shared_perm() from sha1_file.c to path.c,
since a few SIMPLE_PROGRAM need to call repository configuration
functions which in turn need to call adjust_shared_perm().
sha1_file.c needs to link with SHA1 computation library which
is usually not linked to SIMPLE_PROGRAM.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'sha1_file.c')
-rw-r--r-- | sha1_file.c | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/sha1_file.c b/sha1_file.c index aea0f40d57..b4ff233bad 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -50,29 +50,6 @@ int get_sha1_hex(const char *hex, unsigned char *sha1) return 0; } -int adjust_shared_perm(const char *path) -{ - struct stat st; - int mode; - - if (!shared_repository) - return 0; - if (lstat(path, &st) < 0) - return -1; - mode = st.st_mode; - if (mode & S_IRUSR) - mode |= S_IRGRP; - if (mode & S_IWUSR) - mode |= S_IWGRP; - if (mode & S_IXUSR) - mode |= S_IXGRP; - if (S_ISDIR(mode)) - mode |= S_ISGID; - if (chmod(path, mode) < 0) - return -2; - return 0; -} - int safe_create_leading_directories(char *path) { char *pos = path; |