summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJesper Juhl <jj@chaosbits.net>2011-03-21 20:35:49 +0100
committerTrond Myklebust <Trond.Myklebust@netapp.com>2011-03-23 01:03:52 +0100
commit51e816564db151f7f08c739ad69b9d1dbd166663 (patch)
treefa273f72e32874154dfb0d219994c1087b2ac3c3
parentnfs4: Fix NULL dereference at d_alloc_and_lookup() (diff)
downloadlinux-51e816564db151f7f08c739ad69b9d1dbd166663.tar.xz
linux-51e816564db151f7f08c739ad69b9d1dbd166663.zip
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>
-rw-r--r--fs/nfs_common/nfsacl.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/nfs_common/nfsacl.c b/fs/nfs_common/nfsacl.c
index 84c27d69d421..ec0f277be7f5 100644
--- a/fs/nfs_common/nfsacl.c
+++ b/fs/nfs_common/nfsacl.c
@@ -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