diff options
author | Jakub Tyszkowski <jakub.tyszkowski@codecoup.pl> | 2022-04-04 10:21:25 +0000 |
---|---|---|
committer | Jakub Tyszkowski <jakub.tyszkowski@codecoup.pl> | 2022-04-14 09:04:05 +0000 |
commit | a6a4472ba7963bb0fe379d93c61660cd9e4dc9c6 (patch) | |
tree | 51b5927db5bf0889577a9a20f89564cce9b264de /framework/java | |
parent | c7ceab19a8fbea3f16c2e481ed0d67dd21bfe204 (diff) |
broadcaster: Add missing null checks at client side
These can be already checked at the frameworks side.
Bug: 150670922
Tag: #feature
Test: atest CtsBluetoothTestCases:android.bluetooth.cts.BluetoothHapClientTest
Sponsor: jpawlowski@
Change-Id: I1b3d68594f5c96343080d4cf693bd41cce8be581
Diffstat (limited to 'framework/java')
-rw-r--r-- | framework/java/android/bluetooth/BluetoothLeBroadcast.java | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/framework/java/android/bluetooth/BluetoothLeBroadcast.java b/framework/java/android/bluetooth/BluetoothLeBroadcast.java index 72eaf437ef..3cdf2cd129 100644 --- a/framework/java/android/bluetooth/BluetoothLeBroadcast.java +++ b/framework/java/android/bluetooth/BluetoothLeBroadcast.java @@ -542,8 +542,8 @@ public final class BluetoothLeBroadcast implements AutoCloseable, BluetoothProfi * * @param contentMetadata metadata for the default Broadcast subgroup * @param broadcastCode Encryption will be enabled when <var>broadcastCode</var> is not null - * @throws IllegalArgumentException if <var>contentMetadata</var> is null * @throws IllegalStateException if callback was not registered + * @throws NullPointerException if <var>contentMetadata</var> is null * @hide */ @SystemApi @@ -554,6 +554,8 @@ public final class BluetoothLeBroadcast implements AutoCloseable, BluetoothProfi }) public void startBroadcast(@NonNull BluetoothLeAudioContentMetadata contentMetadata, @Nullable byte[] broadcastCode) { + Objects.requireNonNull(contentMetadata, "contentMetadata cannot be null"); + if (DBG) log("startBroadcasting"); final IBluetoothLeAudio service = getService(); if (service == null) { @@ -579,6 +581,7 @@ public final class BluetoothLeBroadcast implements AutoCloseable, BluetoothProfi * @param broadcastId broadcastId as defined by the Basic Audio Profile * @param contentMetadata updated metadata for the default Broadcast subgroup * @throws IllegalStateException if callback was not registered + * @throws NullPointerException if <var>contentMetadata</var> is null * @hide */ @SystemApi @@ -589,6 +592,8 @@ public final class BluetoothLeBroadcast implements AutoCloseable, BluetoothProfi }) public void updateBroadcast(int broadcastId, @NonNull BluetoothLeAudioContentMetadata contentMetadata) { + Objects.requireNonNull(contentMetadata, "contentMetadata cannot be null"); + if (DBG) log("updateBroadcast"); final IBluetoothLeAudio service = getService(); if (service == null) { |