Commit af6d5721 authored by J. Bruce Fields's avatar J. Bruce Fields

svcrpc: don't bother checking bad svc_addr_len result

None of the callers should see an unsupported address family (only one
of them even bothers to check for that case), so just check for the
buggy case in svc_addr_len and don't bother elsewhere.
Acked-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent f23abfdb
...@@ -167,8 +167,7 @@ static inline size_t svc_addr_len(const struct sockaddr *sa) ...@@ -167,8 +167,7 @@ static inline size_t svc_addr_len(const struct sockaddr *sa)
case AF_INET6: case AF_INET6:
return sizeof(struct sockaddr_in6); return sizeof(struct sockaddr_in6);
} }
BUG();
return 0;
} }
static inline unsigned short svc_xprt_local_port(const struct svc_xprt *xprt) static inline unsigned short svc_xprt_local_port(const struct svc_xprt *xprt)
......
...@@ -601,8 +601,6 @@ static int svc_udp_recvfrom(struct svc_rqst *rqstp) ...@@ -601,8 +601,6 @@ static int svc_udp_recvfrom(struct svc_rqst *rqstp)
return -EAGAIN; return -EAGAIN;
} }
len = svc_addr_len(svc_addr(rqstp)); len = svc_addr_len(svc_addr(rqstp));
if (len == 0)
return -EAFNOSUPPORT;
rqstp->rq_addrlen = len; rqstp->rq_addrlen = len;
if (skb->tstamp.tv64 == 0) { if (skb->tstamp.tv64 == 0) {
skb->tstamp = ktime_get_real(); skb->tstamp = ktime_get_real();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment