From 57890f0b8ef5bf410919aee9fae2d306d2a438b8 Mon Sep 17 00:00:00 2001 From: obrien Date: Fri, 15 Mar 2002 07:58:13 +0000 Subject: This vendor security fix, was part of 2.5.3 and later. So we don't need it anymore. --- net/rsync/files/patch-clientserver.c | 27 --------------------------- 1 file changed, 27 deletions(-) delete mode 100644 net/rsync/files/patch-clientserver.c (limited to 'net/rsync') 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"); -- cgit