diff options
| author | 2023-10-18 07:28:10 +0000 | |
|---|---|---|
| committer | 2023-10-18 07:28:10 +0000 | |
| commit | 4b80d89f3ad9bb9fb173cc4bc307d1528487e98b (patch) | |
| tree | d2057aee434e85e00734736e6cfdfb36e4b6b87b | |
| parent | 9fb7e4c796426e5d921fa3355a8ba75fa66c360c (diff) | |
| parent | 3962c485f814f1ef9c01f73c0765fb24c14babb5 (diff) | |
Merge "AudioDeviceBroker: fix regression in isValidCommunicationDevice" into main
| -rw-r--r-- | services/core/java/com/android/server/audio/AudioDeviceBroker.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/audio/AudioDeviceBroker.java b/services/core/java/com/android/server/audio/AudioDeviceBroker.java index 9cfac9af3991..eea3d3885b34 100644 --- a/services/core/java/com/android/server/audio/AudioDeviceBroker.java +++ b/services/core/java/com/android/server/audio/AudioDeviceBroker.java @@ -68,6 +68,7 @@ import java.util.HashSet; import java.util.LinkedList; import java.util.List; import java.util.NoSuchElementException; +import java.util.Objects; import java.util.Set; import java.util.UUID; import java.util.concurrent.atomic.AtomicBoolean; @@ -496,8 +497,9 @@ public class AudioDeviceBroker { AudioDeviceInfo.TYPE_AUX_LINE }; - /*package */ static boolean isValidCommunicationDevice(AudioDeviceInfo device) { - return isValidCommunicationDeviceType(device.getType()); + /*package */ static boolean isValidCommunicationDevice(@NonNull AudioDeviceInfo device) { + Objects.requireNonNull(device, "device must not be null"); + return device.isSink() && isValidCommunicationDeviceType(device.getType()); } private static boolean isValidCommunicationDeviceType(int deviceType) { |