RxFingerprint icon indicating copy to clipboard operation
RxFingerprint copied to clipboard

NPE on Xiaomi MI 5s

Open PblryH opened this issue 6 years ago • 3 comments

Fatal Exception: java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.String java.lang.CharSequence.toString()' on a null object reference at com.mtramin.rxfingerprint.FingerprintObservable$2.onAuthenticationError(FingerprintObservable.java:89) at android.hardware.fingerprint.FingerprintManager$MyHandler.sendErrorResult(FingerprintManager.java:848) at android.hardware.fingerprint.FingerprintManager$MyHandler.handleMessage(FingerprintManager.java:813) at android.os.Handler.dispatchMessage(Handler.java:102) at android.os.Looper.loop(Looper.java:163) at android.app.ActivityThread.main(ActivityThread.java:6321) at java.lang.reflect.Method.invoke(Method.java) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:880) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:770)

PblryH avatar Feb 28 '18 20:02 PblryH

My solution emitter.onError(new FingerprintAuthenticationException(errString != null ? errString : "Error code: " + errMsgId));

And solution for potential NPE on line FingerprintObservable.java:100 FingerprintObservable.this.onAuthenticationHelp(emitter, helpMsgId, helpString != null ? helpString.toString() : "Help code: " + helpMsgId);

PblryH avatar Feb 28 '18 20:02 PblryH

Interesting. This looks like Xiaomi devices might not supply a string in those callbacks at all under some conditions. Do you want to create a PR with your proposed fix for this?

Mauin avatar Mar 07 '18 19:03 Mauin

Do you want to create a PR with your proposed fix for this?

it would be nice.

PblryH avatar Mar 10 '18 11:03 PblryH