diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2018-03-27 10:49:38 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2018-04-03 15:08:09 -0400 |
commit | 989f881ebf77d70e883dd0fbcfa04a058d97f771 (patch) | |
tree | 2c9ab7de3394a05cc07f69062d63265627cbd25f /net/sunrpc/svc_xprt.c | |
parent | 63a1b1569372860fdef9e25edfc2320766b2f4c2 (diff) |
svc: Simplify ->xpo_secure_port
Clean up: Instead of returning a value that is used to set or clear
a bit, just make ->xpo_secure_port mangle that bit, and return void.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/sunrpc/svc_xprt.c')
-rw-r--r-- | net/sunrpc/svc_xprt.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/sunrpc/svc_xprt.c b/net/sunrpc/svc_xprt.c index 6dca0f513c95..4e3b4c596bae 100644 --- a/net/sunrpc/svc_xprt.c +++ b/net/sunrpc/svc_xprt.c | |||
@@ -844,10 +844,7 @@ int svc_recv(struct svc_rqst *rqstp, long timeout) | |||
844 | 844 | ||
845 | clear_bit(XPT_OLD, &xprt->xpt_flags); | 845 | clear_bit(XPT_OLD, &xprt->xpt_flags); |
846 | 846 | ||
847 | if (xprt->xpt_ops->xpo_secure_port(rqstp)) | 847 | xprt->xpt_ops->xpo_secure_port(rqstp); |
848 | set_bit(RQ_SECURE, &rqstp->rq_flags); | ||
849 | else | ||
850 | clear_bit(RQ_SECURE, &rqstp->rq_flags); | ||
851 | rqstp->rq_chandle.defer = svc_defer; | 848 | rqstp->rq_chandle.defer = svc_defer; |
852 | rqstp->rq_xid = svc_getu32(&rqstp->rq_arg.head[0]); | 849 | rqstp->rq_xid = svc_getu32(&rqstp->rq_arg.head[0]); |
853 | 850 | ||