[nearby] Catches NPE in ChreDiscoveryProvider am: e899d26468

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2871061

Change-Id: I4a8a0358c93bf7146de9222e11ae4b12101a5071
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Xinyi Zhou
2023-12-13 20:42:32 +00:00
committed by Automerger Merge Worker
2 changed files with 3 additions and 1 deletions

View File

@@ -284,6 +284,8 @@ public class NearbyManager {
*/ */
public void queryOffloadCapability(@NonNull @CallbackExecutor Executor executor, public void queryOffloadCapability(@NonNull @CallbackExecutor Executor executor,
@NonNull Consumer<OffloadCapability> callback) { @NonNull Consumer<OffloadCapability> callback) {
Objects.requireNonNull(executor);
Objects.requireNonNull(callback);
try { try {
mService.queryOffloadCapability(new OffloadTransport(executor, callback)); mService.queryOffloadCapability(new OffloadTransport(executor, callback));
} catch (RemoteException e) { } catch (RemoteException e) {

View File

@@ -155,7 +155,7 @@ public class ChreDiscoveryProvider extends AbstractDiscoveryProvider {
builder.setFastPairSupported(version != ChreCommunication.INVALID_NANO_APP_VERSION); builder.setFastPairSupported(version != ChreCommunication.INVALID_NANO_APP_VERSION);
try { try {
callback.onQueryComplete(builder.build()); callback.onQueryComplete(builder.build());
} catch (RemoteException e) { } catch (RemoteException | NullPointerException e) {
e.printStackTrace(); e.printStackTrace();
} }
}); });