[PATCH] knfsd: nfsd4: better nfs4acl errors

We're returning -1 in a few places in the NFSv4<->POSIX acl translation code
where we could return a reasonable error.

Also allows some minor simplification elsewhere.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 3 insertions, 3 deletions

diff --git a/fs/nfsd/nfs4acl.c b/fs/nfsd/nfs4acl.c
index 63818a51c05c..edb107e61b91 100644
--- a/fs/nfsd/nfs4acl.c
+++ b/fs/nfsd/nfs4acl.c
@@ -710,9 +710,9 @@ calculate_posix_ace_count(struct nfs4_acl *n4acl)
                 /* Also, the remaining entries are for named users and
                  * groups, and come in threes (mask, allow, deny): */
                 if (n4acl->naces < 7)
-                        return -1;
+                        return -EINVAL;
                 if ((n4acl->naces - 7) % 3)
-                        return -1;
+                        return -EINVAL;
                 return 4 + (n4acl->naces - 7)/3;
         }
 }
@@ -866,7 +866,7 @@ nfs4_acl_add_ace(struct nfs4_acl *acl, u32 type, u32 flag, u32 access_mask,
         struct nfs4_ace *ace;
 
         if ((ace = kmalloc(sizeof(*ace), GFP_KERNEL)) == NULL)
-                return -1;
+                return -ENOMEM;
 
         ace->type = type;
         ace->flag = flag;