ports/127292: not work cancel inheritance on share

[Alex Keda]

>Number:         127292
>Category:       ports
>Synopsis:       not work cancel inheritance on share
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          update
>Submitter-Id:   current-users
>Arrival-Date:   Thu Sep 11 08:00:09 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Alex Keda
>Release:        6,2
>Organization:
ussr
>Environment:
FreeBSD vlgsrv.moskb.local 6.2-STABLE FreeBSD 6.2-STABLE #0: Wed Jun 13 12:23:52 MSD 2007     lissyara at chlsrv.moskb.local:/usr/obj/usr/src/sys/SMP  i386
>Description:
see full description https://bugzilla.samba.org/show_bug.cgi?id=5052

it actually for all release after 3.0.25

>How-To-Repeat:
create folder in share
>Fix:
vlgsrv# cat /usr/ports/net/samba3/files/patch-posix_acls.c
--- smbd/posix_acls.c.orig      2008-08-26 01:09:21.000000000 +0400
+++ smbd/posix_acls.c   2008-09-11 09:00:41.000000000 +0400
@@ -3300,10 +3300,17 @@

        }

-       parent_sd->dacl->aces = new_ace;
-       parent_sd->dacl->num_aces = i;
+       /* This sucks. psd should be const and we should
+        * be doing a deep-copy here. We're getting away
+        * with is as we know parent_sd is talloced off
+        * talloc_tos() as well as psd. JRA. */
+
+       psd->dacl->aces = new_ace;
+       psd->dacl->num_aces = i;
+       psd->type &= ~(SE_DESC_DACL_AUTO_INHERITED|
+                         SE_DESC_DACL_AUTO_INHERIT_REQ);

-       *pp_new_sd = parent_sd;
+       *pp_new_sd = psd;
        return status;
 }

vlgsrv#   

>Release-Note:
>Audit-Trail:
>Unformatted: