diff options
author | Yuyang Huang <yuyanghuang@google.com> | 2024-01-04 15:01:43 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2024-01-04 15:01:43 +0000 |
commit | 01aac3afecc5bd0d52e389b40438bf8460835260 (patch) | |
tree | c09018bf590f02ff20d5c47dc3f65132de0cbcb7 | |
parent | c4e6dd1786805c651eb7fe01a6329c8667b99e3f (diff) | |
parent | 77fcc2194e9a4aa1f2bfbbfdc3149732da44106e (diff) | |
download | netd-01aac3afecc5bd0d52e389b40438bf8460835260.tar.gz |
Merge "Fix IPV4/6 privacy leak" into main am: 681ef1d9c3 am: 533d180901 am: 77fcc2194e
Original change: https://android-review.googlesource.com/c/platform/system/netd/+/2891931
Change-Id: I1d1fdd4eff08662d33705ca104d0aad07187a22e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r-- | server/SockDiag.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/server/SockDiag.cpp b/server/SockDiag.cpp index 49ca8d71..ef3a840b 100644 --- a/server/SockDiag.cpp +++ b/server/SockDiag.cpp @@ -55,6 +55,8 @@ using netdutils::Stopwatch; namespace net { namespace { +static const bool isUser = (android::base::GetProperty("ro.build.type", "") == "user"); + int getAdbPort() { return android::base::GetIntProperty("service.adb.tcp.port", 0); } @@ -335,18 +337,20 @@ int SockDiag::destroySockets(const char* addrstr, int ifindex) { if (!strchr(addrstr, ':')) { // inet_ntop never returns something like ::ffff:192.0.2.1 if (int ret = destroySockets(IPPROTO_TCP, AF_INET, addrstr, ifindex)) { - ALOGE("Failed to destroy IPv4 sockets on %s: %s", where.c_str(), strerror(-ret)); + ALOGE("Failed to destroy IPv4 sockets on %s: %s", + (isUser ? "[hidden: user build]" : where.c_str()), strerror(-ret)); return ret; } } if (int ret = destroySockets(IPPROTO_TCP, AF_INET6, addrstr, ifindex)) { - ALOGE("Failed to destroy IPv6 sockets on %s: %s", where.c_str(), strerror(-ret)); + ALOGE("Failed to destroy IPv6 sockets on %s: %s", + (isUser ? "[hidden: user build]" : where.c_str()), strerror(-ret)); return ret; } if (mSocketsDestroyed > 0) { - ALOGI("Destroyed %d sockets on %s in %" PRId64 "us", mSocketsDestroyed, where.c_str(), - s.timeTakenUs()); + ALOGI("Destroyed %d sockets on %s in %" PRId64 "us", mSocketsDestroyed, + (isUser ? "[hidden: user build]" : where.c_str()), s.timeTakenUs()); } return mSocketsDestroyed; |