From: Andrew Tridgell <tridge@samba.org>
Date: Thu, 6 Jan 2000 00:26:00 +0000 (+0000)
Subject: when we do a lchown() on a file we have to flush the cached perms on
X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/commitdiff_plain/a5827a28d296b62cd9b20a221991903601938448

when we do a lchown() on a file we have to flush the cached perms on
the file if the file has the setuid or setgid bits set as the chown
has a side effect of removing the setuid and setgid bits.
we re-do the stat in this case
---

diff --git a/rsync.c b/rsync.c
index a65f01ff..2d267a0f 100644
--- a/rsync.c
+++ b/rsync.c
@@ -195,6 +195,12 @@ int set_perms(char *fname,struct file_struct *file,STRUCT_STAT *st,
 			rprintf(FERROR,"chown %s : %s\n", fname,strerror(errno));
 			return 0;
 		}
+		/* a lchown had been done - we have to re-stat if the
+                   destination had the setuid or setgid bits set due
+                   to the side effect of the chown call */
+		if (st->st_mode & (S_ISUID | S_ISGID)) {
+			link_stat(fname, st);
+		}
 		updated = 1;
 	}