Logical operator short-circuit on Samsung

Question

I am getting a NullPointerException reported in one of my apps. The stacktrace gets me to a point where I check some BLE related features.

The code is the following :

private static boolean areHardwareFeaturesSupported(BluetoothManager bluetoothManager) {
    BluetoothAdapter bluetoothAdapter = bluetoothManager.getAdapter();
    return bluetoothAdapter != null
            && bluetoothAdapter.isOffloadedFilteringSupported()
            && bluetoothAdapter.isOffloadedScanBatchingSupported();
}

The exception is raised with the following reason :

java.lang.NullPointerException: Attempt to invoke interface method 'boolean android.bluetooth.IBluetooth.isOffloadedFilteringSupported()' on a null object reference

I am puzzled as this expression should not even try to invoke the function if the object is null (because of Java's short circuit).

This crash has only been reported on Samsung Galaxy S4 devices (SGH-I337M), running Lollipop (5.0.1).

Has anyone faced this kind of issue, or is something wrong with my code ?

I would add some log before the return to be sure that's the issue, because that behavior would be odd — , Apr 16 '16 at 16:52
bluetoothManager.getAdapter() seems to be returning null. What does that method look like? — Logan, Apr 16 '16 at 16:53
@RC. I got this report from Crashlytics, and it is pretty hard for me to reproduce. — Simon Guerout, Apr 16 '16 at 16:57
@LoganKulinski The method returns a reference to a BluetoothAdapter — Simon Guerout, Apr 16 '16 at 16:58

Logical operator short-circuit on Samsung

0 Answers0