Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Aug 2008 13:58:17 GMT
From:      Edward Tomasz Napierala <trasz@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 147298 for review
Message-ID:  <200808131358.m7DDwH4K048018@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=147298

Change 147298 by trasz@trasz_traszkan on 2008/08/13 13:57:57

	Relax two overly restrictive checks that prevented old ACLs
	to be longer than 31 instead of 32.

Affected files ...

.. //depot/projects/soc2008/trasz_nfs4acl/sys/kern/vfs_acl.c#12 edit

Differences ...

==== //depot/projects/soc2008/trasz_nfs4acl/sys/kern/vfs_acl.c#12 (text+ko) ====

@@ -71,7 +71,7 @@
 {
 	int i;
 
-	if (source->acl_cnt < 0 || source->acl_cnt >= OLDACL_MAX_ENTRIES)
+	if (source->acl_cnt < 0 || source->acl_cnt > OLDACL_MAX_ENTRIES)
 		return (EINVAL);
 	
 	bzero(dest, sizeof(*dest));
@@ -82,8 +82,6 @@
 		dest->acl_entry[i].ae_tag = source->acl_entry[i].ae_tag;
 		dest->acl_entry[i].ae_id = source->acl_entry[i].ae_id;
 		dest->acl_entry[i].ae_perm = source->acl_entry[i].ae_perm;
-		dest->acl_entry[i].ae_extended = ACL_EXTENDED_ALLOW;
-		dest->acl_entry[i].ae_flags = 0;
 	}
 
 	return (0);
@@ -94,7 +92,7 @@
 {
 	int i;
 
-	if (source->acl_cnt < 0 || source->acl_cnt >= OLDACL_MAX_ENTRIES)
+	if (source->acl_cnt < 0 || source->acl_cnt > OLDACL_MAX_ENTRIES)
 		return (EINVAL);
 
 	bzero(dest, sizeof(*dest));



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200808131358.m7DDwH4K048018>