diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-01-20 08:48:18 +0100 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2012-02-03 20:26:42 +0100 |
commit | 3476964dba98641716173445aade77d40cc6f27a (patch) | |
tree | e63392ae1073db8c91e96cab7c0127960c629925 /net/sunrpc | |
parent | nfsd: don't allow zero length strings in cache_parse() (diff) | |
download | linux-3476964dba98641716173445aade77d40cc6f27a.tar.xz linux-3476964dba98641716173445aade77d40cc6f27a.zip |
nfsd: remove some unneeded checks
We check for zero length strings in the caller now, so these aren't
needed.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/sunrpc')
-rw-r--r-- | net/sunrpc/svcauth_unix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/svcauth_unix.c b/net/sunrpc/svcauth_unix.c index 01153ead1dba..6ab35736d88f 100644 --- a/net/sunrpc/svcauth_unix.c +++ b/net/sunrpc/svcauth_unix.c @@ -509,7 +509,7 @@ static int unix_gid_parse(struct cache_detail *cd, time_t expiry; struct unix_gid ug, *ugp; - if (mlen <= 0 || mesg[mlen-1] != '\n') + if (mesg[mlen - 1] != '\n') return -EINVAL; mesg[mlen-1] = 0; |