This is the mail archive of the cygwin-cvs@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[newlib-cygwin] Use all ACEs from original ACL on Samba share


https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=b07326d29e0df7a8f21aaadefebecbec8da20144

commit b07326d29e0df7a8f21aaadefebecbec8da20144
Author: Corinna Vinschen <corinna@vinschen.de>
Date:   Thu Apr 30 17:26:55 2015 +0200

    Use all ACEs from original ACL on Samba share
    
    	* security.cc (convert_samba_sd): Fix accidental dropping of all
    	non-Unix User, non-Unix Group accounts.
    
    Signed-off-by: Corinna Vinschen <corinna@vinschen.de>

Diff:
---
 winsup/cygwin/ChangeLog   | 5 +++++
 winsup/cygwin/security.cc | 6 +++---
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 43bbbc1..414e148 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,8 @@
+2015-04-30  Corinna Vinschen  <corinna@vinschen.de>
+
+	* security.cc (convert_samba_sd): Fix accidental dropping of all
+	non-Unix User, non-Unix Group accounts.
+
 2015-04-27  Corinna Vinschen  <corinna@vinschen.de>
 
 	* include/asm/types.h: Add __s64 and __u64 types.
diff --git a/winsup/cygwin/security.cc b/winsup/cygwin/security.cc
index 1bc0c50..86ebe2c 100644
--- a/winsup/cygwin/security.cc
+++ b/winsup/cygwin/security.cc
@@ -1145,10 +1145,10 @@ convert_samba_sd (security_descriptor &sd_ret)
 		if (gid < UNIX_POSIX_OFFSET && (grp = internal_getgrgid (gid)))
 		  ace_sid.getfromgr (grp);
 	      }
-	    if (!add_access_allowed_ace (acl, i, ace->Mask, ace_sid, acl_len,
-					 ace->Header.AceFlags))
-	      return;
 	  }
+	if (!add_access_allowed_ace (acl, i, ace->Mask, ace_sid, acl_len,
+				     ace->Header.AceFlags))
+	  return;
       }
   acl->AclSize = acl_len;


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]