| commit | 2647c464f10ab0bf5ea8ea1a80d4b79e168f1ee6 | [log] [tgz] |
|---|---|---|
| author | Xinyi Zhou <zxy@google.com> | Wed Dec 13 21:14:30 2023 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Wed Dec 13 21:14:30 2023 +0000 |
| tree | 132f33f951c34fff9a4d1ed571c92f692e8bb67f | |
| parent | f1f8de49b155995f3a70747b2ef5ddf334889bbe [diff] | |
| parent | ebbfeee740e42732ace3b0d8ea63f3d96f27d94a [diff] |
[nearby] Catches NPE in ChreDiscoveryProvider am: e899d26468 am: ebbfeee740 Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2871061 Change-Id: I38deae29eae221ca76d4c5aa892ecd956c31c0c8 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/nearby/framework/java/android/nearby/NearbyManager.java b/nearby/framework/java/android/nearby/NearbyManager.java index 070a2b6..00f1c38 100644 --- a/nearby/framework/java/android/nearby/NearbyManager.java +++ b/nearby/framework/java/android/nearby/NearbyManager.java
@@ -284,6 +284,8 @@ */ public void queryOffloadCapability(@NonNull @CallbackExecutor Executor executor, @NonNull Consumer<OffloadCapability> callback) { + Objects.requireNonNull(executor); + Objects.requireNonNull(callback); try { mService.queryOffloadCapability(new OffloadTransport(executor, callback)); } catch (RemoteException e) {
diff --git a/nearby/service/java/com/android/server/nearby/provider/ChreDiscoveryProvider.java b/nearby/service/java/com/android/server/nearby/provider/ChreDiscoveryProvider.java index d69d42d..0302e4f 100644 --- a/nearby/service/java/com/android/server/nearby/provider/ChreDiscoveryProvider.java +++ b/nearby/service/java/com/android/server/nearby/provider/ChreDiscoveryProvider.java
@@ -155,7 +155,7 @@ builder.setFastPairSupported(version != ChreCommunication.INVALID_NANO_APP_VERSION); try { callback.onQueryComplete(builder.build()); - } catch (RemoteException e) { + } catch (RemoteException | NullPointerException e) { e.printStackTrace(); } });