L2capPacketForwarder forwards packets between an L2CAP BluetoothSocket
and a tun fd.
It uses blocking i/o to do so, since the BluetoothSocket API does not
expose a file descriptor to poll() on.
It remains to be seen whether shutdown() on a tun interface fd is
actually implemented. Currently, the test uses a pair of SOCK_SEQPACKET
sockets which require to be shutdown to interrupt the blocking read.
Test: L2capPacketForwarderTest
Change-Id: I03d2a2dd76ec2b71fdb9e3ea7008daa6d1d124fe
Revert submission 3477737-move_apex_shortcut
Reason for revert: Likely culprit for b/394990888 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.
Reverted changes: /q/submissionid:3477737-move_apex_shortcut
Change-Id: I8acc4fe8628cdef2f58f3423576fe5c582fcb023
Chaitanya Cheemala (xWF)authored and Gerrit Code Reviewcommited 2025-02-07 01:38:05 -08:00
com.android.remoteauth doesn't exist, but the value wasn't being
checked because no apexes depended on the module, triggering a shortcut
that skipped the apex_available check.
Bug: 394664568
Test: builds
Change-Id: I25fd078093fd71a0e0df01fb31be9f2b811108ff
Ignore this error code during DEL operations.
It is returned when the specific thing to be deleted
is not found and is therefore a valid response.
Gets rid of 30+ logs worth of:
E TcUtils : NLMSG_ERROR message return error: -2
during netd startup (cleanup phase).
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Ie542f8ff418dbed604eea5b31b87f9b974ccdbcb
Maciej Żenczykowskicommited 2025-02-06 12:56:09 -08:00
Otherwise, if adb over ethernet is used, assumeNoInterfaceForTetheringAvailable() tears down the adb
connection because the call to requestTetheredInterface() tears
down IpClient.
Bug: 357332207
Test: atest EthernetManagerTest
Change-Id: I9d04c1a8d39d0accd8061dbfb195ae2359f48e93
chayemmeauthored and Patrick Rohrcommited 2025-02-06 08:11:29 -08:00
This commit removes the obsolete CtsTetheringTestLatestSdk target, which
is no longer needed. This target was previously required for CTS to
function on the mainline-prod branch, but with the migration to trunk
stable, these branches no longer exist.
Test: TH
Change-Id: Ib6600071dd82e9e8b429b69844703429dea12438
individually passed parameters
This CL also moves the DataStore updates of the failure count into
CTLogger directly (CTDownloader will only modify the failure count on
complete success).
Note: due to limitations with Mockito (not being able to mock static
classes like the auto-generated CTStatsLog), there is no test class for
CTLoggerImpl.
Bug: 394278886
Bug: 378626065
Flag: com.android.net.ct.flags.certificate_transparency_service
Test: atest NetworkSecurityUnitTests
Change-Id: I4498bfe75e84c6ba1654cfa55912c46836f61e03
TetheringManagerTest#testRequestLatestEntitlementResult in particular
set carrier config overrides without waiting for the override to be
applied, which caused flakes.
Wait for the override to be applied when adding and clearing them.
This makes the logic to wait for the update in NetworkAgentTest
unnecessary.
Bug: 391966043
Test: atest
Change-Id: I371ff5742627ca0b1b6eb6be5f0fd19a0487ae83
This commit updates the TEST_MAPPING to use CtsTetheringTest directly
instead of CtsTetheringTestLatestSdk. Since the *-mainline-prod branch
is no longer used, the latest SDK test is redundant, and
CtsTetheringTest should be used instead.
This commit also move the CtsTetheringTest to presubmit.
Test: TH
Change-Id: I78ef1368e9122c9b69466bb7e295ec38b5d1e829
Test cases testSoftApConfigurationGetPersistentRandomizedMacAddress and testSetAirplaneMode
are using the wifi service even wifi feature is disabled.
This change skips the test based on WiFi feature
Bug: 349198732
Change-Id: I1f1a9968dc716dbb2fbca42c1a9ca6271c474238
Merged-In: I3ba614e3b4e1b63973510d687459f822cb441bd7
chayemmeauthored and Lorenzo Colitticommited 2025-02-04 18:19:57 -08:00
Tethering will ignore interfaceStatusChanged for enabled interfaces
starting in B, instead only creating the IpServer when the link layer
event comes. Fix the assertions around interfaceStatusChanged for B.
Bug: 393978354
Test: atest TetheringTest
Change-Id: I74d0843785c5d9353e59b73d6d7b30d6d07ac0e0
Quang Anh Luongcommited 2025-02-05 10:43:59 +09:00
(it's a bit to pointlessly repetitive)
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Idc84890401c480a562f433a547b5ba6d2ffdc111
Maciej Żenczykowskicommited 2025-02-04 13:54:38 -08:00
As previously discussed, we will be logging the signature to help with
on-path attacker detection. See go/ct-android-notes for summary of the past
discussions. SignatureVerifier has been modified to return a status
object to better capture the various potential signature verification
failure statuses.
The log list timestamp will be added in a later change as
this will require modifying CompatibilityVersion's logic.
Bug: 378626065
Bug: 391327942
Bug: 394278886
Flag: com.android.net.ct.flags.certificate_transparency_service
Test: atest NetworkSecurityUnitTests
Change-Id: I33d08b2d8c0260237bb3c5a66770fdf409adf8b1
PERMISSION_INTERNET, PERMISSION_UPDATE_DEVICE_STATS,
PERMISSION_UNINSTALLED and NO_PERMISSIONS are used only internally
and don't need to be maintained as part of the public interface.
Bug: b/326383676
Test: PermissionMonitorTest
Change-Id: If906382965c8593f9078278af37cb0bc2825117f
.toList() was only introduced to android in api level 34, which is newer than
this module's min_sdk_version. Replace it with .collect().
This was found while updating android lint.
Flag: EXEMPT refactor
Bug: 394096385
Test: Presubmits
Change-Id: I472bff7749aebb21bd8681ecf4235ce7a6cfc7de
All of the auto-generated enum naming in CTStatsLog is over 100+ chars
long, causing there to be numerous lint errors. This CL creates an
intermediate enum to make it easier to reference these status enums
without introducing lint warning fatigue.
Bug: 378626065
Flag: com.android.net.ct.flags.certificate_transparency_service
Test: atest NetworkSecurityUnitTests
Change-Id: I451a006961ee5d801b3fbf702d3f5282d0b02242
Test cases testSoftApConfigurationGetPersistentRandomizedMacAddress and testSetAirplaneMode
are using the wifi service even wifi feature is disabled.
This change skips the test based on WiFi feature
Bug: 349198732
Test: atest CtsNetTestCases
Change-Id: I1ed6bd9522fac0151c5ac3b20d1b002bab79dd12
Merged-In: I3ba614e3b4e1b63973510d687459f822cb441bd7
chayemmeauthored and Lorenzo Colitticommited 2025-02-02 20:31:25 -08:00
most of these are spammy and/or not useful.
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Iab101ce7de076ef025a046d5e3e9007e091b271f
Maciej Żenczykowskicommited 2025-01-31 12:53:11 -08:00
This commit adds CtsTetheringTestLatestSdk to the postsubmit
configuration. This is a preparatory step towards moving the test to
presubmit, ensuring it runs regularly and catches issues earlier in the
development cycle.
Test: TH
Change-Id: I6932b0b660a675a20102d2683119f689557d8590
Add a basic implementation of stopTetheringRequest that simply stops
tethering for that specific type if it is active, or fail with
TETHER_ERROR_UNKNOWN_REQUEST if not. In the future, we will do fuzzy
matching to ensure the correct request is removed.
Bug: 216524590
Test: atest TetheringManagerTest#testStopTetheringRequest
Change-Id: Ib4a660aacdb83e645e39ceb61b7cda45bb24d89d
Quang Anh Luongcommited 2025-01-31 16:23:04 +09:00
Treehugger Robot
authored and
Gerrit Code Review
commited 
Patrick Rohr
authored and
Sarup Dalwani
authored and
Jacque Li
commited 
Colin Cross
authored and
Chaitanya Cheemala (xWF)
authored and
Maciej Żenczykowski
commited 
chayemme
authored and
Bessie Jiang
authored and
Jimi Chen
commited 
Yuyang Huang
commited 
Automerger Merge Worker
commited 
Sandro Montanari
authored and
Remi NGUYEN VAN
commited 
Lorenzo Colitti
commited 
Quang Anh Luong
commited 
Melisa Carranza Zúñiga
authored and
Cole Faust
commited