diff options
author | David E. O'Brien <obrien@FreeBSD.org> | 2002-03-15 07:58:13 +0000 |
---|---|---|
committer | David E. O'Brien <obrien@FreeBSD.org> | 2002-03-15 07:58:13 +0000 |
commit | 13da324988ee3dac8060590019e28d4f96d5f3fb (patch) | |
tree | d8d064864dae9e162a77a598748c2ff8ffd38e57 /net/rsync | |
parent | Rsync has updated its copy of zlib from 1.1.2 to 1.1.4, so we don't need (diff) |
This vendor security fix, was part of 2.5.3 and later.
So we don't need it anymore.
Notes
Notes:
svn path=/head/; revision=56132
Diffstat (limited to 'net/rsync')
-rw-r--r-- | net/rsync/files/patch-clientserver.c | 27 |
1 files changed, 0 insertions, 27 deletions
diff --git a/net/rsync/files/patch-clientserver.c b/net/rsync/files/patch-clientserver.c deleted file mode 100644 index 9385abb3caf7..000000000000 --- a/net/rsync/files/patch-clientserver.c +++ /dev/null @@ -1,27 +0,0 @@ ---- clientserver.c 7 Feb 2002 16:36:12 -0000 1.82 -+++ clientserver.c 21 Feb 2002 00:45:48 -0000 1.86 -@@ -282,6 +285,24 @@ - } - - if (am_root) { -+ /* Get rid of any supplementary groups this process -+ * might have inheristed. */ -+ if (setgroups(0, NULL)) { -+ rsyserr(FERROR, errno, "setgroups failed"); -+ io_printf(fd, "@ERROR: setgroups failed\n"); -+ return -1; -+ } -+ -+ /* XXXX: You could argue that if the daemon is started -+ * by a non-root user and they explicitly specify a -+ * gid, then we should try to change to that gid -- -+ * this could be possible if it's already in their -+ * supplementary groups. */ -+ -+ /* TODO: Perhaps we need to document that if rsyncd is -+ * started by somebody other than root it will inherit -+ * all their supplementary groups. */ -+ - if (setgid(gid)) { - rsyserr(FERROR, errno, "setgid %d failed", (int) gid); - io_printf(fd,"@ERROR: setgid failed\n"); |