diff options
author | J. Bruce Fields <bfields@fieldses.org> | 2007-07-17 07:04:36 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-17 13:23:07 -0400 |
commit | f7fede4b27bfc6c987d6da8e40384b1b098830bb (patch) | |
tree | c9ec028a7d0a1ff1814108a61177713512830acd | |
parent | 33a1060ae7dc671a0208b341bd454009625bb5a6 (diff) |
knfsd: nfsd4: silence a compiler warning in ACL code
Silence a compiler warning in the ACL code, and add a comment making clear the
initialization serves no other purpose.
Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/nfsd/nfs4acl.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/fs/nfsd/nfs4acl.c b/fs/nfsd/nfs4acl.c index cc3b7badd486..4adb5ee4759b 100644 --- a/fs/nfsd/nfs4acl.c +++ b/fs/nfsd/nfs4acl.c | |||
@@ -183,8 +183,13 @@ static void | |||
183 | summarize_posix_acl(struct posix_acl *acl, struct posix_acl_summary *pas) | 183 | summarize_posix_acl(struct posix_acl *acl, struct posix_acl_summary *pas) |
184 | { | 184 | { |
185 | struct posix_acl_entry *pa, *pe; | 185 | struct posix_acl_entry *pa, *pe; |
186 | pas->users = 0; | 186 | |
187 | pas->groups = 0; | 187 | /* |
188 | * Only pas.users and pas.groups need initialization; previous | ||
189 | * posix_acl_valid() calls ensure that the other fields will be | ||
190 | * initialized in the following loop. But, just to placate gcc: | ||
191 | */ | ||
192 | memset(pas, 0, sizeof(*pas)); | ||
188 | pas->mask = 07; | 193 | pas->mask = 07; |
189 | 194 | ||
190 | pe = acl->a_entries + acl->a_count; | 195 | pe = acl->a_entries + acl->a_count; |