Remove pointless memset in nfsacl_encode()
authorJesper Juhl <jj@chaosbits.net>
Mon, 21 Mar 2011 19:35:49 +0000 (20:35 +0100)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 23 Mar 2011 00:03:52 +0000 (20:03 -0400)
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

index 84c27d69d421e494c2c41359f06dd02eb2c32960..ec0f277be7f5d4162bc4f12f6e84de17025b0935 100644 (file)
@@ -117,7 +117,6 @@ int nfsacl_encode(struct xdr_buf *buf, unsigned int base, struct inode *inode,
                 * 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