[WifiRtt] add check to verify bw and preamble combination valid

check the combination before send the request to the FW

Bug: 142147874
Test: atest android.net.wifi
Test: atest com.android.server.wifi

Change-Id: I98961412cc582ca7b2f1e596017ddc8cd3fee7ab

1 files changed, 15 insertions, 0 deletions

diff --git a/service/java/com/android/server/wifi/rtt/RttNative.java b/service/java/com/android/server/wifi/rtt/RttNative.java
index 06b2b6536..4b1524139 100644
--- a/service/java/com/android/server/wifi/rtt/RttNative.java
+++ b/service/java/com/android/server/wifi/rtt/RttNative.java
@@ -415,6 +415,7 @@ public class RttNative {
                 config.channel.centerFreq1 = responder.centerFreq1;
                 config.bw = halRttChannelBandwidthFromResponderChannelWidth(responder.channelWidth);
                 config.preamble = halRttPreambleFromResponderPreamble(responder.preamble);
+                validateBwAndPreambleCombination(config.bw, config.preamble);
 
                 if (config.peer == RttPeerType.NAN) {
                     config.mustRequestLci = false;
@@ -453,6 +454,20 @@ public class RttNative {
         return rttConfigs;
     }
 
+    private static void validateBwAndPreambleCombination(int bw, int preamble) {
+        if (bw <= RttBw.BW_20MHZ) {
+            return;
+        }
+        if (bw == RttBw.BW_40MHZ && preamble >= RttPreamble.HT) {
+            return;
+        }
+        if (bw >= RttBw.BW_80MHZ && preamble >= RttPreamble.VHT) {
+            return;
+        }
+        throw new IllegalArgumentException(
+                "bw and preamble combination is invalid, bw: " + bw + " preamble: " + preamble);
+    }
+
     private static ArrayList<android.hardware.wifi.V1_4.RttConfig>
             convertRangingRequestToRttConfigs14(
             RangingRequest request, boolean isCalledFromPrivilegedContext, Capabilities cap) {