diff options
author | obrien <obrien@FreeBSD.org> | 2002-03-15 07:58:13 +0000 |
---|---|---|
committer | obrien <obrien@FreeBSD.org> | 2002-03-15 07:58:13 +0000 |
commit | 57890f0b8ef5bf410919aee9fae2d306d2a438b8 (patch) | |
tree | 8ddde06a9902744c4a30bea05ca74f6fd0bce513 /net/rsync | |
parent | d9acf0ecda8605c414f780abba69c90b9c9bb30c (diff) | |
download | FreeBSD-ports-57890f0b8ef5bf410919aee9fae2d306d2a438b8.zip FreeBSD-ports-57890f0b8ef5bf410919aee9fae2d306d2a438b8.tar.gz |
This vendor security fix, was part of 2.5.3 and later.
So we don't need it anymore.
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 9385abb..0000000 --- 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"); |