Bug 33846 - Share definition does not get removed if samba option is disabled
Share definition does not get removed if samba option is disabled
Product: UCS
Classification: Unclassified
Component: Samba4
UCS 4.2
Other Linux
: P5 normal (vote)
: ---
Assigned To: Samba maintainers
Samba maintainers
Depends on:
  Show dependency treegraph
Reported: 2014-01-06 10:33 CET by Arvid Requate
Modified: 2020-07-03 20:52 CEST (History)
4 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2013120621005229
Bug group (optional):
Max CVSS v3 score:
requate: Patch_Available+

fixbug33846.patch (11.19 KB, patch)
2017-04-20 19:27 CEST, Arvid Requate
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arvid Requate univentionstaff 2014-01-06 10:33:42 CET
The module is shipped as part of univention-samba4 as well.

+++ This bug was initially created as a clone of Bug #33698 +++

Seen at ticket #2013120621005229:

If disabling the "samba" option within UMC at a share, the share definition in filesystem (/etc/samba/shares.conf.d/) won't be removed. Seems to be related to the listener module, I suppose this should be done there.

Discovered in UCS 3.1-1, reproduceable with UCS 3.2
Comment 1 Arvid Requate univentionstaff 2017-04-20 19:27:01 CEST
Created attachment 8788 [details]

Apparently the listener only calls the module if the objectclass filter specified in the module matches the new object:


The attached patch fixes this:

* let the module be called also if only the old object had the samba option
* sanitize the share path (against ../../../../etc/passwd)
* skip samba reload in postrun in case the handler didn't need to do anything
Comment 2 Ingo Steuwer univentionstaff 2020-07-03 20:52:05 CEST
This issue has been filed against UCS 4.2.

UCS 4.2 is out of maintenance and many UCS components have changed in later releases. Thus, this issue is now being closed.

If this issue still occurs in newer UCS versions, please use "Clone this bug" or reopen it and update the UCS version. In this case please provide detailed information on how this issue is affecting you.