[rsync-announce] security hole in non-chroot rsync daemon

rsync-announce at lists.samba.org rsync-announce at lists.samba.org
Thu Aug 12 08:46:17 GMT 2004


There is a path-sanitizing bug that affects daemon mode in all recent
rsync versions (including 2.6.2) but only if chroot is disabled.  It
does NOT affect the normal send/receive filenames that specify what
files should be transferred (this is because these names happen to get
sanitized twice, and thus the second call removes any lingering leading
slash(es) that the first call left behind).


I can think of two ways to exploit this sanitizing bug:

For anyone running an rsync daemon with chroot turned off while allowing
the uploading of files, this bug can allow a carefully crafted filename
for the --backup-dir option to cause rsync to overwrite a file outside
of the module's path (if the UID of the daemon module has adequate
permissions, of course).

For anyone running a 2.6.0 or 2.6.1 rsync daemon with chroot turned off,
this bug can potentially reveal the contents of a file outside the
module's hierarchy if the user uses a carefully crafted --files-from
filename.  This causes each line of the file to be revealed to the user
as link_stat errors (other rsync versions hide these errors from the
remote user).


The best fix is to apply this one-word patch to the sanitize_path()
function in util.c:

--- orig/util.c	2004-04-27 12:59:37 -0700
+++ util.c	2004-08-11 23:37:27 -0700
@@ -743,7 +743,7 @@
 				allowdotdot = 1;
 			} else {
 				p += 2;
-				if (*p == '/')
+				while (*p == '/')
 				if (sanp != start) {
 					/* back up sanp one level */

This bug is fixed in the CVS version of rsync, and will be released in
version 2.6.3 (which will begin release-testing soon).

One potential fix that doesn't require recompiling rsync is to set
"use chroot = true" for all the modules in the rsyncd.conf file.

A band-aid work-around for those running a 2.6.2 daemon is to exclude
the use of the above options in the rsyncd.conf file:

    refuse options = files-from backup-dir

Note, however, that this "refuse options" configure item was broken in
older rsync versions, such as all the 2.5.x versions and 2.6.0.


Yes this bug is similar to the last security problem fixed in rsync in
that the effect is the same even though the cause is slightly different.
In the older bug, there was a way to slip certain option values through
the option-parsing without sanitize_path() getting called on the path at
all.  In this new bug, the problem is that a carefully crafted path can
be cleaned improperly, resulting in an absolute filename being generated
instead of a relative one.  (I note that this cleaning problem in the
sanitize_path() function exists even as far back as rsync 2.3.0.)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.samba.org/archive/rsync-announce/attachments/20040812/6c340c5f/attachment.bin

More information about the rsync-announce mailing list