Re: Default acl

Hello,

I've made a little patch.

acl_to_info() parser was expecting at least 4 lines from acl_to_text,
for my test-case only 3 lines were returned for ACL_TYPE_DEFAULT.

-xavier


2007/4/10, Joerg Schilling <[email protected]>:
>
> "Mr X" <[email protected]> wrote:
>
> > Thanks you for taking a look at this.
> >
> > on this folder there's an added acl "user:1001:rwx" (which is saved
> > correctly using star)
> > -xavier
>
> The tar archive does not contain default ACL entries.
>
> As I currently have no access to a Linux system with working ACLs,
> I cannot debug this. Try to check acl_unix.c and find out why the
> code does not see default acls.
>
> Jörg
>
> --
> EMail:[email protected] (home) Jörg Schilling D-13353
> Berlin
>        [email protected]                (uni)
>        [email protected]     (work) Blog:
> http://schily.blogspot.com/
> URL:  http://cdrecord.berlios.de/old/private/
> ftp://ftp.berlios.de/pub/schily
>

_______________________________________________
Star-users mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/star-users

diff -Naur star-1.5/star/acl_unix.c star-1.5-patched/star/acl_unix.c
--- star-1.5/star/acl_unix.c	2006-11-12 01:07:28.000000000 +0100
+++ star-1.5-patched/star/acl_unix.c	2007-04-10 14:39:08.000000000 +0200
@@ -239,7 +239,7 @@
 			entries++;
 		}
 	}
-	if (entries > 3) { /* > 4 on Solaris? */
+	if (entries >= 3) { /* > 4 on Solaris? */
 		if (!acl_add_ids(name, acltext, text)) {
 			acl_free((acl_t)text);
 			return (FALSE);