Skip to content

Commit

Permalink
SUNRPC: Use conventional switch statement when reclassifying sockets
Browse files Browse the repository at this point in the history
Clean up.

Defensive coding: If "family" is ever something that is neither
AF_INET nor AF_INET6, xs_reclassify_socket6() is not the appropriate
default action.  Choose to do nothing in that case.

Introduced by commit 6bc9638.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
  • Loading branch information
Chuck Lever authored and J. Bruce Fields committed Oct 21, 2010
1 parent a25e758 commit 4232e86
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions net/sunrpc/xprtsock.c
Original file line number Diff line number Diff line change
Expand Up @@ -1558,7 +1558,7 @@ static int xs_bind(struct sock_xprt *transport, struct socket *sock)
nloop++;
} while (err == -EADDRINUSE && nloop != 2);

if (myaddr.ss_family == PF_INET)
if (myaddr.ss_family == AF_INET)
dprintk("RPC: %s %pI4:%u: %s (%d)\n", __func__,
&((struct sockaddr_in *)&myaddr)->sin_addr,
port, err ? "failed" : "ok", err);
Expand Down Expand Up @@ -1594,10 +1594,14 @@ static inline void xs_reclassify_socket6(struct socket *sock)

static inline void xs_reclassify_socket(int family, struct socket *sock)
{
if (family == PF_INET)
switch (family) {
case AF_INET:
xs_reclassify_socket4(sock);
else
break;
case AF_INET6:
xs_reclassify_socket6(sock);
break;
}
}
#else
static inline void xs_reclassify_socket4(struct socket *sock)
Expand Down

0 comments on commit 4232e86

Please sign in to comment.