Closed Bug 1505720 Opened 2 years ago Closed 1 year ago
crash [@ java
.lang .Null Pointer Exception: at org .mozilla .gecko .fxa .authenticator .Android Fx Account .rename Account If Necessary(Android Fx Account .java)]
797.13 KB, text/plain
Bug 1505720 crash [@ java.lang.NullPointerException: at org.mozilla.gecko.fxa.authenticator.AndroidFxAccount.renameAccountIfNecessary(AndroidFxAccount.java)] r=JanH
47 bytes, text/x-phabricator-request
|Details | Review|
Device: - Sony Xperia Z5 Premium (Android 6.0.1); Build(s): - Nightly 65.0a1 (2018-11-07); - Release 63.0.2. Steps to reproduce: 1. Go to Settings -> Sign in, and sign in with valid credentials. 2. Go to Settings -> you FxA and Disconnect 3. Nightly crashes Expected result: The user should be disconnected. Actual result: Nightly crashes.
Hi, I reproduced the crash on Nightly 65.0a1 (2018-11-08) on Samsung Galaxy Note 8 (Android 8.0). crash signature: [@ java.lang.NullPointerException: at org.mozilla.gecko.fxa.authenticator.AndroidFxAccount.renameAccountIfNecessary(AndroidFxAccount.java) ]
If we crash every time people log out of accounts we should set this as a P1 to investigate. It doesn't look like a huge crash volume on release, but it's significant.
Priority: -- → P1
Refreshing account (triggered by signing in) can cause a NullPointerException after quickly signing out. This defect consists in bad sync between the signing in state and signing out state.
Request from Janet to find a proper reviewer.
Hi, I have just reproduced the crash with this id on the Sony Xperia Z5 with Android 7.0 on Fennec 64.0.1 - build id: 20181213132357 https://crash-stats.mozilla.com/report/index/aed7858f-d040-44da-96a6-edeba0181214#tab-details
@Susheel - both Edourard and Nick Alexander provided reviews for this fix. @Alexandru, it's unclear to me whether you reproduced this issue before or after the fix was merged. @Susheel, I'm NI'ing back to you to follow up on the details here.
Flags: needinfo?(jdragojevic) → needinfo?(sdaswani)
Hey Janet, I am still working on this, nothing has been merged. Will make requested changes ASAP
Thanks Janet, Andrei is working on it so removing my NI until further 'needed' :) .
You need to log in before you can comment on or make changes to this bug.