From 286d6525f095e435266cbc6fb0b4983a1a4d9218 Mon Sep 17 00:00:00 2001 From: maheshkkv Date: Fri, 14 Feb 2025 16:32:22 -0800 Subject: Fix RangingResult parcelling Bug: 397709104 Test: atest com.android.server.wifi.rtt Flag: EXEMPT bug fix Change-Id: I775b85765de05a0403705a1720c76fdfbf650ef4 --- framework/java/android/net/wifi/rtt/RangingResult.java | 12 ++++++++++++ framework/java/android/net/wifi/rtt/ResponderConfig.java | 5 +++-- 2 files changed, 15 insertions(+), 2 deletions(-) (limited to 'framework/java') diff --git a/framework/java/android/net/wifi/rtt/RangingResult.java b/framework/java/android/net/wifi/rtt/RangingResult.java index 604c879daa..166a13b1d9 100644 --- a/framework/java/android/net/wifi/rtt/RangingResult.java +++ b/framework/java/android/net/wifi/rtt/RangingResult.java @@ -1168,6 +1168,12 @@ public final class RangingResult implements Parcelable { if (SdkLevel.isAtLeastV()) { dest.writeList(mVendorData); } + dest.writeBoolean(mIsRangingAuthenticated); + dest.writeBoolean(mIsRangingFrameProtected); + dest.writeBoolean(mIsSecureHeLtfEnabled); + dest.writeLong(mPasnComebackAfterMillis); + dest.writeByteArray(mPasnComebackCookie); + dest.writeInt(mSecureHeLtfProtocolVersion); } public static final @android.annotation.NonNull Creator CREATOR = @@ -1208,6 +1214,12 @@ public final class RangingResult implements Parcelable { if (SdkLevel.isAtLeastV()) { builder.setVendorData(ParcelUtil.readOuiKeyedDataList(in)); } + builder.setRangingAuthenticated(in.readBoolean()) + .setRangingFrameProtected(in.readBoolean()) + .setSecureHeLtfEnabled(in.readBoolean()) + .setPasnComebackAfterMillis(in.readLong()) + .setPasnComebackCookie(in.createByteArray()) + .setSecureHeLtfProtocolVersion(in.readInt()); return builder.build(); } }; diff --git a/framework/java/android/net/wifi/rtt/ResponderConfig.java b/framework/java/android/net/wifi/rtt/ResponderConfig.java index 5b4f2da08a..ffa98499ed 100644 --- a/framework/java/android/net/wifi/rtt/ResponderConfig.java +++ b/framework/java/android/net/wifi/rtt/ResponderConfig.java @@ -514,13 +514,14 @@ public final class ResponderConfig implements Parcelable { if (scanResult.getWifiSsid() != null) { pasnConfigBuilder.setWifiSsid(scanResult.getWifiSsid()); } + // If the responder is capable of PASN, always enable frame protection for secure ranging + // irrespective of responder mandates or not. return new SecureRangingConfig.Builder(pasnConfigBuilder.build()) .setSecureHeLtfEnabled(scanResult.isSecureHeLtfSupported()) - .setRangingFrameProtectionEnabled(scanResult.isRangingFrameProtectionRequired()) + .setRangingFrameProtectionEnabled(true) .build(); } - /** * Creates a Responder configuration from a MAC address corresponding to a Wi-Fi Aware * Responder. The Responder parameters are set to defaults. -- cgit v1.2.3-59-g8ed1b