ctdb-common: Use correct macro for checking Ethernet hardware family
authorMartin Schwenke <martin@meltin.net>
Thu, 19 May 2016 06:22:36 +0000 (16:22 +1000)
committerMartin Schwenke <martins@samba.org>
Thu, 26 May 2016 09:42:46 +0000 (11:42 +0200)
Both of these expand to 1.  However, AF_LOCAL is a Unix domain socket,
which makes no sense when reading the code.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Martin Schwenke <martins@samba.org>
Autobuild-Date(master): Thu May 26 11:42:46 CEST 2016 on sn-devel-144

ctdb/common/system_linux.c

index f8c793299b60e5f0b7b60946de7808f203806108..2dca174e5900e13f61603241110eba85829e654c 100644 (file)
@@ -127,7 +127,7 @@ int ctdb_sys_send_arp(const ctdb_sock_addr *addr, const char *iface)
                        close(s);
                        return 0;
                }
-               if (if_hwaddr.ifr_hwaddr.sa_family != AF_LOCAL) {
+               if (if_hwaddr.ifr_hwaddr.sa_family != ARPHRD_ETHER) {
                        close(s);
                        errno = EINVAL;
                        DEBUG(DEBUG_CRIT,(__location__ " not an ethernet address family (0x%x)\n",
@@ -221,7 +221,7 @@ int ctdb_sys_send_arp(const ctdb_sock_addr *addr, const char *iface)
                        close(s);
                        return 0;
                }
-               if (if_hwaddr.ifr_hwaddr.sa_family != AF_LOCAL) {
+               if (if_hwaddr.ifr_hwaddr.sa_family != ARPHRD_ETHER) {
                        close(s);
                        errno = EINVAL;
                        DEBUG(DEBUG_CRIT,(__location__ " not an ethernet address family (0x%x)\n",