summaryrefslogtreecommitdiff
path: root/readconf.c
diff options
context:
space:
mode:
authorColin Watson <cjwatson@debian.org>2014-02-09 16:09:58 +0000
committerColin Watson <cjwatson@debian.org>2016-12-23 11:25:02 +0000
commitdf060c830ad66289a93be24268f3f70e7021be29 (patch)
tree4c41c8ff0deed41d671309af897a3ec6ca069668 /readconf.c
parent7083ae25ccce8bbdad40e7c7500f69f2c0cbce34 (diff)
Allow harmless group-writability
Allow secure files (~/.ssh/config, ~/.ssh/authorized_keys, etc.) to be group-writable, provided that the group in question contains only the file's owner. Rejected upstream for IMO incorrect reasons (e.g. a misunderstanding about the contents of gr->gr_mem). Given that per-user groups and umask 002 are the default setup in Debian (for good reasons - this makes operating in setgid directories with other groups much easier), we need to permit this by default. Bug: https://bugzilla.mindrot.org/show_bug.cgi?id=1060 Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=314347 Last-Update: 2013-09-14 Patch-Name: user-group-modes.patch
Diffstat (limited to 'readconf.c')
-rw-r--r--readconf.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/readconf.c b/readconf.c
index 3efba242f..c02cdf63a 100644
--- a/readconf.c
+++ b/readconf.c
@@ -1752,8 +1752,7 @@ read_config_file_depth(const char *filename, struct passwd *pw,
1752 1752
1753 if (fstat(fileno(f), &sb) == -1) 1753 if (fstat(fileno(f), &sb) == -1)
1754 fatal("fstat %s: %s", filename, strerror(errno)); 1754 fatal("fstat %s: %s", filename, strerror(errno));
1755 if (((sb.st_uid != 0 && sb.st_uid != getuid()) || 1755 if (!secure_permissions(&sb, getuid()))
1756 (sb.st_mode & 022) != 0))
1757 fatal("Bad owner or permissions on %s", filename); 1756 fatal("Bad owner or permissions on %s", filename);
1758 } 1757 }
1759 1758