Bug 15087 - Samba-Freigaben: Setzen von SUID, SGID und Sticky Bit auf dem Reiter Samba-Erweiterte-Rechte
Samba-Freigaben: Setzen von SUID, SGID und Sticky Bit auf dem Reiter Samba-Er...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Shares
UCS 3.1
All All
: P5 enhancement with 2 votes (vote)
: UCS 3.2
Assigned To: Erik Damrose
Felix Botner
: interim-3
: 23173 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-14 15:01 CEST by Andreas Büsching
Modified: 2013-11-19 06:43 CET (History)
8 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Büsching univentionstaff 2009-07-14 15:01:31 CEST
Samba kann die Rechte SUID, SGID und Sticky Bit ebenfalls auswerten und auch setzen. Dafür sollten diese auf dem Reiter Samba-Erweiiterte-Rechte einstellbar sein.

Es sollte geprüft werden, ob die Rechte jeweils für Dateien und Verzeichnisse sinnvoll sind. Beispielsweise auf UNIX-Ebene ist das Sticky-Bit nur an Verzeichnissen sinnvoll, daher vermute ich, ist es für Samba auch nur dort sinnvoll.
Comment 1 Tim Petersen univentionstaff 2013-02-25 15:15:22 CET
Im Forum angefragt
Comment 2 Stefan Gohmann univentionstaff 2013-02-25 16:48:46 CET
IMHO kann man auch einfach die Syntax für den Dateimodus umsetzen und den Wert dann numerisch setzen.
Comment 3 Thomas Manninger 2013-02-28 16:10:36 CET
Noch bessere wäre eine Möglichkeit einer Option, dass das setzen der ACLs deaktiviert ist.

Ich habe nun einen Fall:
ich habe einen Share auf /var/daten

In einem Unterordner muss ein eigener Share angelegt werden:
/var/daten/subfolder

Wenn jetzt jemand über den Windows Explorer die Rechte auf /var/daten/subfolder ändert, werden diese mit der nächsten Änderung am Share wieder gelöscht.
Comment 4 Stefan Gohmann univentionstaff 2013-04-03 22:02:44 CEST
*** Bug 23173 has been marked as a duplicate of this bug. ***
Comment 5 Stefan Gohmann univentionstaff 2013-04-03 22:03:01 CEST
(In reply to comment #4)
> *** Bug 23173 has been marked as a duplicate of this bug. ***

Die Hinweise sollten beachtet werden.
Comment 6 Moritz Muehlenhoff univentionstaff 2013-05-31 10:43:04 CEST
We will not ship a UCS 3.1-2 release; the next UCS release will be UCS 3.2.

As such, this bug is moved to the new target milestone.
Comment 7 Tim Petersen univentionstaff 2013-06-04 11:39:31 CEST
Asked again by customer.
Comment 8 Erik Damrose univentionstaff 2013-10-18 15:23:35 CEST
The setuid, setgid and sticky bit flags can now be set for share root-directories:

r45331 univention-directory-manager-modules 9.0.67-2.1134.201310181453
r45332 univention-management-console-module-udm 4.0.51-2.374.201310181457
r45333 univention-management-console-frontend 3.0.112-1.758.201310181502

Samba respects these values set for the share root-directory. Adding the setuid, setgid and sticky bit flags in a samba share definition did not change any behaviour within samba and are therefore not offered.
Comment 9 Felix Botner univentionstaff 2013-10-21 15:27:30 CEST
OK - setuid, setgid and sticky bits can be configured for the share root 
     directory
OK - changelog
Comment 10 Felix Botner univentionstaff 2013-10-21 15:50:55 CEST
(In reply to Thomas Manninger from comment #3)
> Noch bessere wäre eine Möglichkeit einer Option, dass das setzen der ACLs
> deaktiviert ist.
> 
> Ich habe nun einen Fall:
> ich habe einen Share auf /var/daten
> 
> In einem Unterordner muss ein eigener Share angelegt werden:
> /var/daten/subfolder
> 
> Wenn jetzt jemand über den Windows Explorer die Rechte auf
> /var/daten/subfolder ändert, werden diese mit der nächsten Änderung am Share
> wieder gelöscht.

reopen, set mode in listenerSharePath only if "univentionShareDirectoryMode" has changed
Comment 11 Moritz Muehlenhoff univentionstaff 2013-10-22 11:16:29 CEST
setuid has no effect on directories in Linux, I think we should rather remove it, it will only cause confusion with the setuid bit for executables.
Comment 12 Erik Damrose univentionstaff 2013-10-23 11:08:24 CEST
Directory mode and owner/group will only be set in case the attributes changed:
univention-lib 3.0.23-1.189.201310230945

Removed setuid option from the permission widget:
univention-management-console-frontend 3.0.117-1.764.201310231100

Changelog updated
Comment 13 Felix Botner univentionstaff 2013-10-23 12:21:19 CEST
OK - setgid and sticky bit flag
OK - directory mode and owner/group will only be set in case the attributes
OK - changelog
Comment 14 Felix Botner univentionstaff 2013-10-23 12:39:24 CEST
(In reply to Erik Damrose from comment #12)
> Directory mode and owner/group will only be set in case the attributes
> changed:

but mode/owner/group is not set if the share is created
Comment 15 Erik Damrose univentionstaff 2013-10-23 16:47:52 CEST
(In reply to Felix Botner from comment #14)
> 
> but mode/owner/group is not set if the share is created

* Modify share directory permissions, owner and group if it is new or
their defining udm attributes have been modified

univention-lib 3.0.24-1.190.201310231645
changelog updated
Comment 16 Felix Botner univentionstaff 2013-10-23 17:12:03 CEST
OK
Comment 17 Stefan Gohmann univentionstaff 2013-11-19 06:43:00 CET
UCS 3.2 has been released:
 http://docs.univention.de/release-notes-3.2-en.html
 http://docs.univention.de/release-notes-3.2-de.html

If this error occurs again, please use "Clone This Bug".