| commit | 0e8a1e4a5a118df278ea2e5dff5524a518972188 | [log] [tgz] |
|---|---|---|
| author | Yuyang Huang <yuyanghuang@google.com> | Tue Jul 23 19:07:18 2024 +0900 |
| committer | Cherrypicker Worker <android-build-cherrypicker-worker@google.com> | Sun Aug 04 11:48:18 2024 +0000 |
| tree | 992e56515bef180786460783d6e969719a86ab4d | |
| parent | 051dc5c508f467874ed492356b4685f35cf96ded [diff] |
Clear all APF memory in testDropPingReply() in preparing stage Modified testDropPingReply() to clear the entire APF memory instead of only the first byte. This addresses a known firmware issue with handling programs smaller than 8 bytes. Bug: 339147921 Test: TH (cherry picked from https://android-review.googlesource.com/q/commit:e749694eb49fa74236d2388faeafd04c725fb461) Merged-In: I651dae8c9daac24b195196c03b54e69a757de8a7 Change-Id: I651dae8c9daac24b195196c03b54e69a757de8a7
diff --git a/tests/cts/net/src/android/net/cts/ApfIntegrationTest.kt b/tests/cts/net/src/android/net/cts/ApfIntegrationTest.kt index 5662fca..710f0fb 100644 --- a/tests/cts/net/src/android/net/cts/ApfIntegrationTest.kt +++ b/tests/cts/net/src/android/net/cts/ApfIntegrationTest.kt
@@ -436,12 +436,7 @@ assumeApfVersionSupportAtLeast(4) // clear any active APF filter - var gen = ApfV4Generator( - caps.apfVersionSupported, - caps.maximumApfProgramSize, - caps.maximumApfProgramSize - ).addPass() - installProgram(gen.generate()) + clearApfMemory() readProgram() // wait for install completion // Assert that initial ping does not get filtered. @@ -455,7 +450,7 @@ assertThat(packetReader.expectPingReply()).isEqualTo(data) // Generate an APF program that drops the next ping - gen = ApfV4Generator( + val gen = ApfV4Generator( caps.apfVersionSupported, caps.maximumApfProgramSize, caps.maximumApfProgramSize