From ec8637f3679b891041c798400774141024e24a4e Mon Sep 17 00:00:00 2001
From: Wayne Davison <wayned@samba.org>
Date: Wed, 9 Jan 2008 08:59:02 -0800
Subject: [PATCH] Don't allow a slash to be specified in a module name.
 Document the module-name limitations in rsyncd.conf.yo.

---
 loadparm.c     | 5 +++++
 rsyncd.conf.yo | 4 ++++
 2 files changed, 9 insertions(+)

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