diff options
author | Volker Lendecke <vl@samba.org> | 2016-02-11 13:11:46 +0100 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2016-02-16 09:32:11 +0100 |
commit | 6774af16ea9b9fecb797a6fae2668b589ee22dc2 (patch) | |
tree | 2a6535fbf745073efaf18a3bfa5f3fabbeabd405 | |
parent | b65697fd937a5d5765349aad65d2d60b04e33191 (diff) | |
download | samba-6774af16ea9b9fecb797a6fae2668b589ee22dc2.tar.gz |
param: Fix str_list_v3 to accept ; again
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Thu Feb 11 19:19:55 CET 2016 on sn-devel-144
https://bugzilla.samba.org/show_bug.cgi?id=11732
-rw-r--r-- | lib/util/util_strlist.c | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/lib/util/util_strlist.c b/lib/util/util_strlist.c index 987fdfbdd33..2c1c2d5f4ff 100644 --- a/lib/util/util_strlist.c +++ b/lib/util/util_strlist.c @@ -552,7 +552,23 @@ char **str_list_make_v3(TALLOC_CTX *mem_ctx, const char *string, TALLOC_FREE(list); return NULL; } - if (!sep) sep = LIST_SEP; + + /* + * DON'T REPLACE THIS BY "LIST_SEP". The common version of + * LIST_SEP does not contain the ;, which used to be accepted + * by Samba 4.0 before param merges. It would be the far + * better solution to split the _v3 version again to source3/ + * where it belongs, see the _v3 in its name. + * + * Unfortunately it is referenced in /lib/param/loadparm.c, + * which depends on the version that the AD-DC mandates, + * namely without the ; as part of the list separator. I am + * missing the waf fu to properly work around the wrong + * include paths here for this defect. + */ + if (sep == NULL) { + sep = " \t,;\n\r"; + } num = 0; str = s; |