Don't allow a slash to be specified in a module name.

Document the module-name limitations in rsyncd.conf.yo.

2 files changed, 9 insertions, 0 deletions

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()