Bug 45338 - Don't allow share names with consecutive whitespaces
Don't allow share names with consecutive whitespaces
Product: UCS
Classification: Unclassified
Component: Samba4
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.2-2-errata
Assigned To: Florian Best
Felix Botner
Depends on: 38688
Blocks: 38689
  Show dependency treegraph
Reported: 2017-09-07 10:55 CEST by Felix Botner
Modified: 2017-09-13 16:35 CEST (History)
7 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 5: Blocking further progress on the daily work
User Pain: 0.143
Enterprise Customer affected?: Yes
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2017021021000308, 2017033121000455, 2017090521000618
Bug group (optional): External feedback, Troubleshooting
Max CVSS v3 score:


Note You need to log in before you can comment on or make changes to this bug.
Description Felix Botner univentionstaff 2017-09-07 10:55:39 CEST
+++ This bug was initially created as a clone of Bug #38688 +++


Samba config parser reduces consecutive whitespaces to one.
If a share "a   b" is defined "/etc/samba/shares.conf.d/a\ \ \ b" is created and included in "/etc/samba/shares.conf" but samba tries to include "/etc/samba/shares.conf.d/a\ b".

testparm just gives a warning about that but samba-tool testparm dies with "ERROR: Unable to load default file" leading to bug38686

We should reduce consecutive whitespaces (maybe in general, for user and group names etc.?) and/or throw a syntax warning.
Comment 1 Felix Botner univentionstaff 2017-09-07 10:57:00 CEST
please merge univention-samba changes to univention-samba4
Comment 2 Felix Botner univentionstaff 2017-09-07 12:20:41 CEST
and create a bug for 4.1-4
Comment 3 Florian Best univentionstaff 2017-09-08 15:31:22 CEST
univention-samba4 (6.0.10-29):
a18852b52b68353d61ab017087d450f59240e516 | Merge branch 'fbest/45338-samba4-share-name-escaping' into 4.2-2
841d5fadda9ef6aeb8df92cc013740622b48f897 | Bug #45338: escape share names

3e90c29c35a0fcefb71d5e61f4c267ab167288f3 | Merge branch 'fbest/45338-samba4-share-name-escaping' into 4.2-2
2a62133546481fc5613b5fcfd8c3d1ba9e8022d9 | YAML Bug #45338
Comment 4 Felix Botner univentionstaff 2017-09-11 11:51:23 CEST
OK - shares spaces
OK - update
OK - samba-tool drs showrepl
Comment 5 Arvid Requate univentionstaff 2017-09-13 16:35:10 CEST