1317785 - Crash in java.lang.UnsupportedOperationException: JNI Function called before it was loaded at org.mozilla.gecko.GeckoAppShell.onLocationChanged(Native Method)

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Marcia Knous [:marcia]]

This bug was filed from the Socorro interface and is 
report bp-08e8484d-85b4-439a-97ee-aba972161115.
=============================================================

Seen while looking at crash stats: http://bit.ly/2fV9QTK. This jumped right to the top of crash stats - wondering if it is the zh-cn issue?

cc'ing a few folks who might be able to shed some light.

Comment 1

[[:philipp]]

hi, could you take a look at this?

Comment 2

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Hi Sebastian, James, Kevin, this is a top crasher on Fennec 50 since we pushed it out on GooglePlay this morning. We are doing a staged rollout and currently configured to update at 10%. Could you please investigate the crash reports? Should I stay at 10%, disable updates or too soon to tell?

Comment 3

[Kevin Brosnan [Ex-Mozilla]]

I can't say for sure that this is a zh-cn issue. Though there are a number of things that point to this crash being unique to zh-cn. Useragent locale field is not returning any data. The build id for the ARM version of 50 from mozilla.org is 20161105201442 the only build ids found for this crash are 20161107135757 and 20161107132644 [1]. The comments are nearly all in Chinese. The device list does not contain normal American/European models. 

My recommendation is to keep Firefox for Android on Play enabled.

We should get a bug on file for user agent locale not reporting data. It may be one of several problems Socorro might not be processing the data correctly, the crash reporter is not collecting this data any more or an issue with the zh-cn build.  

[1] https://crash-stats.mozilla.com/search/?signature=%3Djava.lang.UnsupportedOperationException%3A%20JNI%20Function%20called%20before%20it%20was%20loaded%20at%20org.mozilla.gecko.GeckoAppShell.onLocationChanged%28Native%20Method%29&product=FennecAndroid&version=50.0&date=%3E%3D2016-11-08T21%3A51%3A00.000Z&date=%3C2016-11-15T21%3A51%3A00.000Z&_sort=-date&_facets=build_id&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-build_id

Comment 4

[DongShengXue]

I have found the reason, it is our patch that support baidu geolocation API in China edition. Although it is not certain to occur, I have fixed it and I will update ASAP.

Comment 5

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

The crash volume on this one is extremely high, ~4500 crashes in the past 24 hrs. I plan to disable Fennec 50 updates on Google Play. Tracked as blocking 50.

Comment 6

[Marco Castelluccio [:marco]]

(In reply to Kevin Brosnan [:kbrosnan] from comment #3)
> We should get a bug on file for user agent locale not reporting data. It may
> be one of several problems Socorro might not be processing the data
> correctly, the crash reporter is not collecting this data any more or an
> issue with the zh-cn build.  

It's also possible that the crash is happening so early that we just don't get to the point where we set the annotation in the report.

Comment 7

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to DongShengXue from comment #4)
> I have found the reason, it is our patch that support baidu geolocation API
> in China edition. Although it is not certain to occur, I have fixed it and I
> will update ASAP.

Thanks! Did you already push this fix out?

(In reply to Ritu Kothari (:ritu) from comment #5)
> The crash volume on this one is extremely high, ~4500 crashes in the past 24
> hrs. I plan to disable Fennec 50 updates on Google Play. Tracked as blocking
> 50.

It seems like only the zh-cn build affected.

Comment 8

[DongShengXue]

(In reply to Sebastian Kaspari (:sebastian) from comment #7)
Yes， I already updated the version. And the new buildId is 20161116093641 and 20161116101130.

Comment 9

[Sebastian Kaspari (:sebastian; :pocmo)]

Thanks! Crashes are going down too. (No new crashes with other build ids).

Comment 10

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Removing the tracking flag.