From f62cfc218c7803efbf2f8c851131ee63c60cc3c3 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Wed, 1 Feb 2017 14:41:43 +0000 Subject: [PATCH] smbd: Fix "map acl inherit" = yes Brown-Paper-Bag bug in f85c2a6852a. The assignment contains a self-reference in get_pai_flags which I missed. Fix an uninitialized read. Bug: https://bugzilla.samba.org/show_bug.cgi?id=12551 Signed-off-by: Volker Lendecke --- source3/smbd/posix_acls.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/source3/smbd/posix_acls.c b/source3/smbd/posix_acls.c index 9d02e8a315c..c8d2bb3b5aa 100644 --- a/source3/smbd/posix_acls.c +++ b/source3/smbd/posix_acls.c @@ -2699,9 +2699,9 @@ static canon_ace *canonicalise_acl(struct connection_struct *conn, .attr = ALLOW_ACE, .trustee = sid, .unix_ug = unix_ug, - .owner_type = owner_type, - .ace_flags = get_pai_flags(pal, ace, is_default_acl) + .owner_type = owner_type }; + ace->ace_flags = get_pai_flags(pal, ace, is_default_acl); DLIST_ADD(l_head, ace); } -- 2.11.0