DEV9: Improve logic for getting MacAddress

pcsx2/DEV9/AdapterUtils.cpp

@@ -8,22 +8,24 @@
 #include "common/StringUtil.h"
 
 #ifdef __POSIX__
+#include <unistd.h>
 #include <vector>
 #include <fstream>
 #include <net/if.h>
 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <arpa/inet.h>
-
-#ifdef __linux__
-#include <unistd.h>
 #include <sys/ioctl.h>
-#endif
+#include <string.h>
 
 #if defined(__FreeBSD__) || (__APPLE__)
+#include <sys/types.h>
+#include <net/if_dl.h>
 #include <sys/param.h>
 #include <sys/sysctl.h>
+#include <sys/sockio.h>
 #include <net/route.h>
+#include <net/if_var.h>
 #endif
 #endif
 
@@ -244,30 +246,66 @@ std::optional<MAC_Address> AdapterUtils::GetAdapterMAC(Adapter* adapter)
 
 	return std::nullopt;
 }
-#elif defined(__POSIX__)
-#ifdef __linux__
+#else
 std::optional<MAC_Address> AdapterUtils::GetAdapterMAC(Adapter* adapter)
 {
-	struct ifreq ifr;
-	strcpy(ifr.ifr_name, adapter->ifa_name);
-
-	int fd = socket(AF_INET, SOCK_DGRAM, 0);
-	int ret = ioctl(fd, SIOCGIFHWADDR, &ifr);
-	close(fd);
+	MAC_Address macAddr = {0};
+#if defined(AF_LINK)
+	struct ifaddrs *adapterInfo, *po;
+	int error = getifaddrs(&adapterInfo);
+	if (error)
+	{
+		Console.Error("DEV9: Failed to get adapter information %s", error);
+		return std::nullopt;
+	}
+	for (po = adapterInfo; po != NULL; po = po->ifa_next)
+	{
+		if (strcasecmp(po->ifa_name, adapter->ifa_name))
+			continue;
 
-	if (ret == 0)
-		return *(MAC_Address*)ifr.ifr_hwaddr.sa_data;
+		if (po->ifa_addr->sa_family != AF_LINK)
+			continue;
 
-	return std::nullopt;
-}
+		// We have a valid MAC address.
+		memcpy(&macAddr, LLADDR((struct sockaddr_dl*)po->ifa_addr), 6);
+		freeifaddrs(adapterInfo);
+		return macAddr;
+	}
+	freeifaddrs(adapterInfo);
+	Console.Error("DEV9: Failed to get MAC address for adapter using AF_LINK");
 #else
-std::optional<MAC_Address> AdapterUtils::GetAdapterMAC(Adapter* adapter)
-{
-	Console.Error("DEV9: Unsupported OS, can't get MAC address");
+	int sd = socket(AF_INET, SOCK_DGRAM, IPPROTO_IP);
+	if (sd < 0)
+	{
+		Console.Error("DEV9: Failed to open socket");
+		return std::nullopt;
+	}
+	struct ifreq ifr = {0};
+	strcpy(ifr.ifr_name, adapter->ifa_name);
+#if defined(SIOCGIFHWADDR)
+	if (ioctl(sd, SIOCGIFHWADDR, &ifr) < 0)
+	{
+		close(sd);
+		Console.Error("DEV9: Failed to get MAC address for adapter using SIOCGIFHWADDR");
+		return std::nullopt;
+	}
+	memcpy(&macAddr, &ifr.ifr_hwaddr.sa_data, 6);
+#elif defined(SIOCGENADDR)
+	if (ioctl(sd, SIOCGENADDR, &ifr) < 0)
+	{
+		close(sd);
+		Console.Error("DEV9: Failed to get MAC address for adapter using SIOCGENADDR");
+		return std::nullopt;
+	}
+	memcpy(&macAddr, &ifr.ifr_enaddr, 6);
+#endif
+	close(sd);
+	return macAddr;
+#endif
+	Console.Error("DEV9: Unsupported OS, can't get network features");
 	return std::nullopt;
 }
 #endif
-#endif
 
 // AdapterIP.
 #ifdef _WIN32
@@ -376,7 +414,7 @@ std::vector<IP_Address> AdapterUtils::GetGateways(Adapter* adapter)
 	}
 	return collection;
 }
-#elif defined(__FreeBSD__) || (__APPLE__)
+#elif defined(__FreeBSD__) || defined(__APPLE__)
 std::vector<IP_Address> AdapterUtils::GetGateways(Adapter* adapter)
 {
 	if (adapter == nullptr)