diff options
| author | j223yang@asset.uwaterloo.ca <j223yang@asset.uwaterloo.ca> | 2011-03-10 18:40:28 +0100 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-03-16 01:16:14 +0100 |
| commit | 4d4a76f3309edc671918a767b336492fbc80a16d (patch) | |
| tree | d35e5df0599c095c69bbcae6b5a3207623da07a0 /net | |
| parent | nfs: BKL is no longer needed, so remove the include (diff) | |
| download | linux-4d4a76f3309edc671918a767b336492fbc80a16d.tar.xz linux-4d4a76f3309edc671918a767b336492fbc80a16d.zip | |
xprt: remove redundant null check
'req' is dereferenced before checked for NULL.
The patch simply removes the check.
Signed-off-by: Jinqiu Yang<crindy646@gmail.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/sunrpc/xprt.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/net/sunrpc/xprt.c b/net/sunrpc/xprt.c index 856274d7e85c..8bdcdbe07b98 100644 --- a/net/sunrpc/xprt.c +++ b/net/sunrpc/xprt.c @@ -202,10 +202,9 @@ int xprt_reserve_xprt(struct rpc_task *task) goto out_sleep; } xprt->snd_task = task; - if (req) { - req->rq_bytes_sent = 0; - req->rq_ntrans++; - } + req->rq_bytes_sent = 0; + req->rq_ntrans++; + return 1; out_sleep: |