commit eabc5da4df01564063eed65351d7973b7aa44341
author Remi NGUYEN VAN <reminv@google.com> Tue Apr 24 18:54:58 2018 +0900
committer Remi NGUYEN VAN <reminv@google.com> Thu Apr 26 11:25:02 2018 +0900
tree 049f09504dcd6025ccd22bccc4e60b3dd699050c
parent 05827de37e5413d165bf655bdfd13e3d49c906d1

Return -EAFNOSUPPORT for invalid sockets

Previously NetworkTest#testBindSocketOfLocalSocketThrows would fail when
selinux is disabled because netd does not check for local sockets.

Bug: b/78484123
Test: NetworkTest#testBindSocketOfLocalSocketThrows passes w/o selinux
Change-Id: Ia696bb9f25df46177303bac5127ed2f6ada56e7d

client/FwmarkClient.cpp

diff --git a/client/FwmarkClient.cpp b/client/FwmarkClient.cpp
index 97d509d..9435528 100644
--- a/client/FwmarkClient.cpp
+++ b/client/FwmarkClient.cpp
@@ -53,7 +53,7 @@
     if (isOverriddenBy(ANDROID_NO_USE_FWMARK_CLIENT)) {
         return false;
     }
-    return family == AF_INET || family == AF_INET6;
+    return FwmarkCommand::isSupportedFamily(family);
 }
 
 bool FwmarkClient::shouldReportConnectComplete(int family) {