summaryrefslogtreecommitdiffstats
path: root/fs
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-22 20:03:52 -0400
commit51e816564db151f7f08c739ad69b9d1dbd166663 (patch)
treefa273f72e32874154dfb0d219994c1087b2ac3c3 /fs
parent4667058b772857069378aa8f417203156e315e82 (diff)
downloadop-kernel-dev-51e816564db151f7f08c739ad69b9d1dbd166663.zip
op-kernel-dev-51e816564db151f7f08c739ad69b9d1dbd166663.tar.gz
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>
Diffstat (limited to 'fs')
-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 84c27d6..ec0f277 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
OpenPOWER on IntegriCloud