浏览代码

SUNRPC: Use conventional switch statement when reclassifying sockets

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 6bc9638a.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Chuck Lever 14 年之前
父节点
当前提交
4232e8634a
共有 1 个文件被更改,包括 7 次插入3 次删除
  1. 7 3
      net/sunrpc/xprtsock.c

+ 7 - 3
net/sunrpc/xprtsock.c

@@ -1558,7 +1558,7 @@ static int xs_bind(struct sock_xprt *transport, struct socket *sock)
 			nloop++;
 			nloop++;
 	} while (err == -EADDRINUSE && nloop != 2);
 	} 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__,
 		dprintk("RPC:       %s %pI4:%u: %s (%d)\n", __func__,
 				&((struct sockaddr_in *)&myaddr)->sin_addr,
 				&((struct sockaddr_in *)&myaddr)->sin_addr,
 				port, err ? "failed" : "ok", err);
 				port, err ? "failed" : "ok", err);
@@ -1594,10 +1594,14 @@ static inline void xs_reclassify_socket6(struct socket *sock)
 
 
 static inline void xs_reclassify_socket(int family, 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);
 		xs_reclassify_socket4(sock);
-	else
+		break;
+	case AF_INET6:
 		xs_reclassify_socket6(sock);
 		xs_reclassify_socket6(sock);
+		break;
+	}
 }
 }
 #else
 #else
 static inline void xs_reclassify_socket4(struct socket *sock)
 static inline void xs_reclassify_socket4(struct socket *sock)