From: Wayne Davison <wayned@samba.org>
Date: Wed, 9 Jan 2008 16:59:02 +0000 (-0800)
Subject: Don't allow a slash to be specified in a module name.
X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/commitdiff_plain/ec8637f3679b891041c798400774141024e24a4e?hp=62a6b8df72d18f0b13462ee7ac33f9a8a46ef0b5

Don't allow a slash to be specified in a module name.
Document the module-name limitations in rsyncd.conf.yo.
---

diff --git a/loadparm.c b/loadparm.c
index 0915000d..010e089a 100644
--- a/loadparm.c
+++ b/loadparm.c
@@ -784,6 +784,11 @@ static BOOL do_section(char *sectionname)
      return(True);
    }
 
+   if (strchr(sectionname, '/') != NULL) {
+     rprintf(FLOG, "Warning: invalid section name in configuration file: %s\n", sectionname);
+     return False;
+   }
+
    /* if we have a current service, tidy it up before moving on */
    bRetval = True;
 
diff --git a/rsyncd.conf.yo b/rsyncd.conf.yo
index ffb9473b..b914570d 100644
--- a/rsyncd.conf.yo
+++ b/rsyncd.conf.yo
@@ -113,6 +113,10 @@ After the global options you should define a number of modules, each
 module exports a directory tree as a symbolic name. Modules are
 exported by specifying a module name in square brackets [module]
 followed by the options for that module.
+The module name cannot contain a slash or a closing square bracket.  If the
+name contains whitespace, each internal sequence of whitespace will be
+changed into a single space, while leading or trailing whitespace will be
+discarded.
 
 startdit()