summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Jiabin Huang <jiabin@google.com> 2024-03-07 00:27:30 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2024-03-07 00:27:30 +0000
commit84fbc6b6dcf84ef39b9b5de1892d0dab8cb125c9 (patch)
tree1e9596d327be9d92e292267b6aa5dfa697185ba7
parent511dd1695c6e28a91708a52c2595ebbd2cf2dada (diff)
parent66d7513001c4fd16eaead0f3a0322928c1d711ce (diff)
Merge "Use the descriptor passed in when adding USB device instead of querying again." into main
-rw-r--r--services/usb/java/com/android/server/usb/UsbHostManager.java8
1 files changed, 6 insertions, 2 deletions
diff --git a/services/usb/java/com/android/server/usb/UsbHostManager.java b/services/usb/java/com/android/server/usb/UsbHostManager.java
index 8b44579eda33..36adeec74e36 100644
--- a/services/usb/java/com/android/server/usb/UsbHostManager.java
+++ b/services/usb/java/com/android/server/usb/UsbHostManager.java
@@ -380,6 +380,11 @@ public class UsbHostManager {
return false;
}
+ if (descriptors == null) {
+ Slog.e(TAG, "Failed to add device as the descriptor is null");
+ return false;
+ }
+
UsbDescriptorParser parser = new UsbDescriptorParser(deviceAddress, descriptors);
if (deviceClass == UsbConstants.USB_CLASS_PER_INTERFACE
&& !checkUsbInterfacesDenyListed(parser)) {
@@ -462,8 +467,7 @@ public class UsbHostManager {
}
// Tracking
- addConnectionRecord(deviceAddress, ConnectionRecord.CONNECT,
- parser.getRawDescriptors());
+ addConnectionRecord(deviceAddress, ConnectionRecord.CONNECT, descriptors);
// Stats collection
FrameworkStatsLog.write(FrameworkStatsLog.USB_DEVICE_ATTACHED,