diff options
| author | Gerald Carter <jerry@samba.org> | 2003-11-22 04:33:36 +0000 |
|---|---|---|
| committer | Gerald Carter <jerry@samba.org> | 2003-11-22 04:33:36 +0000 |
| commit | 11f4893145f94c85b4f1268544a84116d3a38751 (patch) | |
| tree | 72b5add6f724e945ddbb05a04bdd7bdb0d2d03cc /source3/param | |
| parent | d66def408ea394fe6475e87cd2405173eb7c8c8c (diff) | |
| download | samba-11f4893145f94c85b4f1268544a84116d3a38751.tar.gz samba-11f4893145f94c85b4f1268544a84116d3a38751.tar.bz2 samba-11f4893145f94c85b4f1268544a84116d3a38751.zip | |
Ensure that items in a list of strings containing whitespace
are written out surrounded by single quotes. This means that
both double and single quotes are now used to surround
strings in smb.conf. This is a slight change from the previous
behavior but needed or else things like
printer admin = +ntadmin, 'VALE\Domain, Admin'
get written to smb.conf by SWAT.
(This used to be commit 5bf91c79d620e34ac71d72c80f74e47754d49dcb)
Diffstat (limited to 'source3/param')
| -rw-r--r-- | source3/param/loadparm.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c index 8643ecb028..e98d924964 100644 --- a/source3/param/loadparm.c +++ b/source3/param/loadparm.c @@ -3301,9 +3301,13 @@ static void print_parameter(struct parm_struct *p, void *ptr, FILE * f) if ((char ***)ptr && *(char ***)ptr) { char **list = *(char ***)ptr; - for (; *list; list++) - fprintf(f, "%s%s", *list, - ((*(list+1))?", ":"")); + for (; *list; list++) { + /* surround strings with whitespace in single quotes */ + if ( strchr_m( *list, ' ' ) ) + fprintf(f, "\'%s\'%s", *list, ((*(list+1))?", ":"")); + else + fprintf(f, "%s%s", *list, ((*(list+1))?", ":"")); + } } break; |