commit 51e816564db151f7f08c739ad69b9d1dbd166663
author Jesper Juhl <jj@chaosbits.net> Mon Mar 21 20:35:49 2011 +0100
committer Trond Myklebust <Trond.Myklebust@netapp.com> Tue Mar 22 20:03:52 2011 -0400
tree fa273f72e32874154dfb0d219994c1087b2ac3c3
parent 4667058b772857069378aa8f417203156e315e82

Remove pointless memset in nfsacl_encode()

Remove pointless memset() in nfsacl_encode().

Thanks to Trond Myklebust <Trond.Myklebust@netapp.com> for pointing out
that it is not needed since posix_acl_init() will set everything
regardless..

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>

fs/nfs_common/nfsacl.c

diff --git a/fs/nfs_common/nfsacl.c b/fs/nfs_common/nfsacl.c
index 84c27d6..ec0f277 100644
--- a/fs/nfs_common/nfsacl.c
+++ b/fs/nfs_common/nfsacl.c
@@ -117,7 +117,6 @@
 		 * invoked in contexts where a memory allocation failure is
 		 * fatal.  Fortunately this fake ACL is small enough to
 		 * construct on the stack. */
-		memset(acl2, 0, sizeof(acl2));
 		posix_acl_init(acl2, 4);
 
 		/* Insert entries in canonical order: other orders seem