path: root/refs.c
diff options
authorJunio C Hamano <>2006-06-10 05:07:23 (GMT)
committerJunio C Hamano <>2006-06-10 05:15:50 (GMT)
commit138086a72582620e0b3ec0aa60a78da208e89a2a (patch)
treeba4cb91659773ed349464247e93a765a10999ac6 /refs.c
parentf6407823d1467a6b6d2002878a64dec42df0a361 (diff)
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 <>
Diffstat (limited to 'refs.c')
1 files changed, 5 insertions, 0 deletions
diff --git a/refs.c b/refs.c
index f91b771..713ca46 100644
--- a/refs.c
+++ b/refs.c
@@ -104,6 +104,11 @@ int create_symref(const char *git_HEAD, const char *refs_heads_master)
error("Unable to create %s", git_HEAD);
return -3;
+ if (adjust_shared_perm(git_HEAD)) {
+ unlink(lockpath);
+ error("Unable to fix permissions on %s", lockpath);
+ return -4;
+ }
return 0;