- Dec 26, 2023
-
-
Chiachang Wang authored
This commit optimizes the mechanism to examine the TCP sockets based on the VPN UID range. The automaticOnOffKeepalives will be paused/resumed by checking the TCP sockets status on the VPN network. The existing design considers all UIDs, not the UID ranges for the specific VPN network. This may lead to an inaccurate judgement to not to stop the keepalive even when there is no socket on the VPN network. This commit is also a part of follow-up work from aosp/2354202 to use structized class to parse the TCP sockets. Bug: 311119352 Bug: 265877161 Test: atest FrameworksNetTests Change-Id: I4a6b751cb904b9c18ed1df420cc880ea38022d6a
-
- Dec 25, 2023
-
-
Chiachang Wang authored
A clean up commit to correct the coding format. Test: atest FrameworksNetTests Change-Id: I7d7ed3765a0be503400ed3c33ea461e9baa3e72d
-
Chiachang Wang authored
Add nlAttrs parameter to store the remaining attributes data. As a part of follow-up work from aosp/2354202, the TCP socket parsing can be done by using existing InetDiagMessage class. The missing piece in the InetDiagMessage class is the possible netlink attributes fields that may be appended after an InetDiagMessage. Bug: 311119352 Bug: 265877161 Test: atest NetworkStaticLibTests Change-Id: I9c34166124fd4efba2ab211bfb537ccce5e774a4
-
- Nov 23, 2023
- Nov 22, 2023
-
-
Motomu Utsumi authored
-
Motomu Utsumi authored
framework-connectivity-flagged-apis-droidstubs is used by jarjar-rules-generator to avoid jarjaring flagged apis. But, this usage of --show-annotation FlaggedApi in the flag is not supported. So this CL removes --show-annotation FlaggedApi. As far as framework-connectivity-flagged-apis-droidstubs contains flagged APIs, it's fine that this droidstubs also contains non-flagged APIs. Test: TH Bug: 312408311 Change-Id: I2f1f5624d98873420c0c36ac6e31386102211045
-
Xiao Ma authored
Test: TH Change-Id: I260665a0afa056e32021c5255a17ea0af59b6575
-
KH Shi authored
HandlerThread was introduced in aosp/2493357. This patch moves the code originally located in buildNewRa() into the HandlerThread. Bug: 246928127 Test: atest TetheringPrivilegedTests TetheringIntegrationTests \ TetheringTests Change-Id: I5f6df0a3a79d7f4a865cfec7f88bc28816b1db95
-
KH Shi authored
This patch changes the implementation of UnicastResponder in the RouterAdvertisementDaemon with FdEventsReader. The change includes replacing Thread with HandlerThread, create socket in non-blocking mode (required by FdEventsReader) and use FdEventsReader to listen to the solicited RS packets. Two commands (CMD_START & CMD_STOP) were added to control the lifecycle of the FdEventsReader. Bug: 246928127 Test: atest TetheringPrivilegedTests TetheringIntegrationTests \ TetheringTests Test: Enable WiFi hotspot and connected with another Pixel client: 1. Tcpdump on the Pixel client, checking the RA packets 2. Verify IPv6 addresses were configured 3. ping6 2001:4860:4860::64 Change-Id: I6e89903df90864e3bb34d498e353d6f87193e9d6
-
Paul Hu authored
-
Chiachang Wang authored
-
Yuyang Huang authored
-
Kangping Dong authored
-
KH Shi authored
-
Mark Chien authored
-
Paul Hu authored
-
Kangping Dong authored
Per API council feedback and API guideline at go/android-api-guidelines#provide-executor, it should call Binder.clearCallingIdentity() before invoking the app-supplied callbacks so that the app doesn't use the system-server's identity for permission check in the callback. Bug: 309952666 Test: atest ThreadNetworkUnitTests Change-Id: I3cfbce29c0a516506bb76a44b151a583c8477806
-
Chiachang Wang authored
This is a clean up commit to address TODO for resetting the LISTEN_ACTIVITY_TIMEOUT_MS timer as aosp/2188755 was shipped. Fix: 252972908 Test: atest CtsNetTestCases Change-Id: Ibc1a1c68883a3772729f24b01f9dc8d3cc37c1eb
-
Yan Yan authored
-
- Nov 21, 2023
-
-
Yan Yan authored
- Change class names to start with Xfrm or StructXfrm for consistency with other messages and structs - Create xfrm sub-folder for xfrm messages - Add "finals" to input parameters Bug: 308011229 Test: atest NetworkStaticLibTests:com.android.net.moduletests.util.netlink (new tests added) Change-Id: Id6d34b4549bbc1890b3f4dd9f1638977f66e3236
-
Mark Chien authored
-
Mike Yu authored
-
Ken Chen authored
It's no longer needed because the only useful version is v1 and all functions are deprecated in v2. Bug: 298594687 Test: m Change-Id: I0226b7f3d3cda6fb897e02a035e84943913a944e
-
Ken Chen authored
-
Paul Hu authored
Also add MonitorThreadLeak annotation to check thread leak. Test: atest ConnectivityCoverageTests --test-filter mdns* Change-Id: Ie47f30360e36784428173f6cc87a4a6b09826405
-
Mark authored
This is no-op change. Test: atest TetheringTests Change-Id: Ia5c68c403bdaa981f35ecad8ffc3e034746dfdea
-
Paul Hu authored
The NsdManager constructor creates a new thread, which is not intended to be closed. However, the test in NsdMangerTest and NsdServiceTest create a new NsdManager instance, resulting in a large number of threads running during the tests. This can lead to an out-of-memory error for the tests. To resolve this issue, use the common singleton thread ConnectivityThread to handle tasks from the various instances. Fix: 308544734 Test: atest ConnectivityCoverageTests --test-filter Nsd* Change-Id: I1908a2bb1bd20da8e0498353bca26f7e89e16626
-
Mark authored
Instead of passing state machine to UpstreamNetworkMonitor, wrapped it by an interface would have two benefits: 1. Easier for testing. 2. Isolate state machine operation in callback. Test: atest TetheringTests Change-Id: I2a97fd5cbc8c49df8c3c98428c6570916ab31ea5
-
Yan Yan authored
-
Yan Yan authored
Test: Owners file changes only; no code changes Change-Id: I28b407a269befd4ce4e61c49dae435ac1b4ef4aa
-
Ken Chen authored
The only version of mdns_aidl_interface supported by Netd is V1. After that, the entire interface has been deprecated (which is the V2). The latest stable version is no longer meaningful because no one is supposed to use V2. Bug: 298594687 Test: m Change-Id: I9ac8199312cdc5d829889ea562979cad7d3bf054
-
- Nov 20, 2023
-
-
KH Shi authored
1. assertEquals syntax is (expected, actual) and not the other way around 2. use assertArrayEquals to compare arrays Test: atest TetheringTests Bug: 312072637 Change-Id: I5eb0c270323253f648d3a8fd488506b9ad1ca6a1
-
KH Shi authored
-
Ken Chen authored
MDNS native support from Netd is stopped in V. Bug: 298594687 Bug: 312092770 Test: run CtsNetTestCasesMaxTargetSdk30 on V, no fatal exception. Change-Id: I3774288cdf5b82e6aeeee7ee2d030fdc524e1aa8
-
KH Shi authored
TetherUpstream6Key#src64 should be stored in network order. Use a big-endian ByteBuffer to retrieve a value stored in network order is actually converts it into native order, which is little-endian in Android. In this CL, we changed it to use byte[] instead of long to store the top 64-bits of the source prefix. Test: atest TetheringTests manual test with IPv6-only tethering upstream, check BPF stats. Bug: 312072637 Change-Id: I79f9282d5eda28328aa6a764ea92b086d6285133
-
Jean Chalard authored
* changes: Rename hasCarrierPrivilege... to isCarrierServiceUid... Detect discrepancies in subIds Allow carrier service UIDs to access their own WiFi networks.
-
Chalard Jean authored
Bug: 308519360 Test: CarrierPrivilegeAuthenticatorTest Change-Id: I59164be00a9e1fcced8538537c1dcabd214f2170
-