diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-04-21 16:08:39 +0200 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2009-04-23 17:25:07 -0400 |
commit | 80492e7d49bee0a280a84a39075a7857b92836b2 (patch) | |
tree | dcf87df4579091000311fb9dd7d68faa23002798 /fs/nfsd | |
parent | 091069740304c979f957ceacec39c461d0192158 (diff) |
rpcgss: remove redundant test on unsigned
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/nfsd')
-rw-r--r-- | fs/nfsd/export.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c index 5839b229cd0..6eb918153fd 100644 --- a/fs/nfsd/export.c +++ b/fs/nfsd/export.c @@ -464,16 +464,11 @@ static int secinfo_parse(char **mesg, char *buf, struct svc_export *exp) if (err) return err; /* - * Just a quick sanity check; we could also try to check - * whether this pseudoflavor is supported, but at worst - * an unsupported pseudoflavor on the export would just - * be a pseudoflavor that won't match the flavor of any - * authenticated request. The administrator will - * probably discover the problem when someone fails to - * authenticate. + * XXX: It would be nice to also check whether this + * pseudoflavor is supported, so we can discover the + * problem at export time instead of when a client fails + * to authenticate. */ - if (f->pseudoflavor < 0) - return -EINVAL; err = get_int(mesg, &f->flags); if (err) return err; |