NLM: Rewrite IPv4 privileged requester's check
authorChuck Lever <chuck.lever@oracle.com>
Thu, 11 Dec 2008 22:56:52 +0000 (17:56 -0500)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Tue, 6 Jan 2009 16:53:56 +0000 (11:53 -0500)
Clean up.

For consistency, rewrite the IPv4 check to match the same style as the
new IPv6 check.  Note that ipv4_is_loopback() is somewhat broader in
its interpretation of what is a loopback address than simply
"127.0.0.1".

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
include/linux/lockd/lockd.h

index 80d7e8a8257d4e578f39d48c82800289847b9619..aa6fe7026de7dbd24fc9a1cdbb3d2fa2e8f191b4 100644 (file)
@@ -291,8 +291,11 @@ static inline struct inode *nlmsvc_file_inode(struct nlm_file *file)
 static inline int __nlm_privileged_request4(const struct sockaddr *sap)
 {
        const struct sockaddr_in *sin = (struct sockaddr_in *)sap;
-       return (sin->sin_addr.s_addr == htonl(INADDR_LOOPBACK)) &&
-                       (ntohs(sin->sin_port) < 1024);
+
+       if (ntohs(sin->sin_port) > 1023)
+               return 0;
+
+       return ipv4_is_loopback(sin->sin_addr.s_addr);
 }
 
 #if defined(CONFIG_IPV6) || defined(CONFIG_IPV6_MODULE)