Closed Bug 1025937 Opened 6 years ago Closed 6 years ago

Crash java.lang.NullPointerException at org.mozilla.gecko.background.healthreport.HealthReportBroadcastService.onHandleIntent(HealthReportBroadcastService.java:104)

Categories

(Firefox for Android :: General, defect, critical)

31 Branch
ARM
Android
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Firefox 33
Tracking Status
firefox31 + fixed
firefox32 --- fixed
firefox33 --- fixed

People

(Reporter: aaronmt, Assigned: rnewman)

References

Details

(Keywords: crash)

Attachments

(1 file)

java.lang.NullPointerException
at org.mozilla.gecko.background.healthreport.HealthReportBroadcastService.onHandleIntent(HealthReportBroadcastService.java:104)
at android.app.IntentService$ServiceHandler.handleMessage(IntentService.java:65)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:137)
at android.os.HandlerThread.run(HandlerThread.java:61)

There is still a large volume of crash reports on the Android Google Play Dashboard with this signature.

User messages seem to indicate that this happens on startup.
You say "still" - what's the history here?
nm, was thinking of the org.mozilla.gecko.background.healthreport.prune.HealthReportPruneService ANR which is still high volume
See also Bug 903136.
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Attached file Proposed patch. v1
Attachment #8440842 - Flags: review?(michael.l.comella)
Comment on attachment 8440842 [details] [review]
Proposed patch. v1

r+ w/ nits.
Attachment #8440842 - Flags: review?(michael.l.comella) → review+
This warrants uplift after bake.
Flags: needinfo?(rnewman)
https://hg.mozilla.org/mozilla-central/rev/e1c288cf638a
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
Comment on attachment 8440842 [details] [review]
Proposed patch. v1

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
  Old work for HealthReport service, but generally all services.

User impact if declined: 
  Occasional crashes.

Testing completed (on m-c, etc.): 
  Only positive testing: doesn't regress normal behavior. This patch addresses edge case crashes.

Risk to taking this patch (and alternatives if risky): 
  Null checks, so should be ~none.

String or IDL/UUID changes made by this patch:
  None.
Attachment #8440842 - Flags: approval-mozilla-aurora?
Flags: needinfo?(rnewman)
Attachment #8440842 - Flags: approval-mozilla-beta?
Attachment #8440842 - Flags: approval-mozilla-beta?
Attachment #8440842 - Flags: approval-mozilla-beta+
Attachment #8440842 - Flags: approval-mozilla-aurora?
Attachment #8440842 - Flags: approval-mozilla-aurora+
Duplicate of this bug: 1030740
Depends on: 1030740
You need to log in before you can comment on or make changes to this bug.