1552152 - Crash in [@ mozilla::jni::NativeStub<T>::Wrap<T>]

Status: RESOLVED DUPLICATE of bug 1551342

(Core :: DOM: Web Authentication, defect, P1)

Description

[Dirkjan Ochtman (:djc)]

This bug is for crash report bp-d5f05780-a440-4ece-a1c4-77e3f0190516.

Top 2 frames of crashing thread:

0 libxul.so void mozilla::jni::NativeStub<mozilla::java::WebAuthnTokenManager::WebAuthnGetAssertionReturnError_t, mozilla::WebAuthnTokenManager, mozilla::jni::Args<mozilla::jni::StringParam const&> >::Wrap<&mozilla::WebAuthnTokenManager::WebAuthnGetAssertionReturnError> widget/android/jni/Natives.h:710
1 base.odex base.odex@0x102b7c 

Tried to use the webauthn.io login in Firefox Nightly on Android and got this. Probably related to bug 1391438?

Comment 1

[Dirkjan Ochtman (:djc)]

(It looks like the top-level crash signature does not distinguish between the types of T in mozilla::jni::NativeStub<T>; bug 1307868 exists to deal with NativeStub crashes but the signature seemed quite different.)

Comment 2

[J.C. Jones [:jcj] (he/they)]

Thanks! I'm trying to track this down, not quite sure if it's all one bug in the jni handling or several. I will likely end up either duping this on another bug, or duping a bunch of test errors onto this one.

Comment 3

[J.C. Jones [:jcj] (he/they)]

The fix for this should be, I think, in https://phabricator.services.mozilla.com/D31497, part of Bug 1551342. I'm going to go ahead and dupe this there, but thanks for the report! It was super-helpful to have a not-from-CI stack trace to confirm against.

Comment 4

[Dirkjan Ochtman (:djc)]

Happy to help, let me know if you need further guinea pigs. Do you have any plans to enable WebAuthn on Fenix?

Comment 5

[J.C. Jones [:jcj] (he/they)]

Yes, this is just the first step. Contact me off-bug (my email or twitter) if you want details.