chiark / gitweb /
shared: avoid semi-duplicating socket_address_equal()
authorMichal Schmidt <mschmidt@redhat.com>
Fri, 20 Feb 2015 01:13:03 +0000 (02:13 +0100)
committerMichal Schmidt <mschmidt@redhat.com>
Fri, 20 Feb 2015 02:35:11 +0000 (03:35 +0100)
Just call socket_address_equal() from socket_address_matches_fd()
instead of implementing similar comparing of addresses.

src/shared/socket-util.c

index c31f60ec7ea58a8f269266dae4ac5e6d704e1430..deecce8a8062da84ead7ec973323d0364ea8e84b 100644 (file)
@@ -464,23 +464,7 @@ bool socket_address_matches_fd(const SocketAddress *a, int fd) {
                         return false;
         }
 
                         return false;
         }
 
-        switch (b.sockaddr.sa.sa_family) {
-
-        case AF_INET:
-                return b.sockaddr.in.sin_port == a->sockaddr.in.sin_port &&
-                        b.sockaddr.in.sin_addr.s_addr == a->sockaddr.in.sin_addr.s_addr;
-
-        case AF_INET6:
-                return b.sockaddr.in6.sin6_port == a->sockaddr.in6.sin6_port &&
-                        memcmp(&b.sockaddr.in6.sin6_addr, &a->sockaddr.in6.sin6_addr, sizeof(struct in6_addr)) == 0;
-
-        case AF_UNIX:
-                return b.sockaddr.size == a->size &&
-                        memcmp(b.sockaddr.un.sun_path, a->sockaddr.un.sun_path, b.size - offsetof(struct sockaddr_un, sun_path)) == 0;
-
-        }
-
-        return false;
+        return socket_address_equal(a, &b);
 }
 
 int sockaddr_pretty(const struct sockaddr *_sa, socklen_t salen, bool translate_ipv6, char **ret) {
 }
 
 int sockaddr_pretty(const struct sockaddr *_sa, socklen_t salen, bool translate_ipv6, char **ret) {