Closed Bug 794908 Opened 7 years ago Closed 7 years ago

java.lang.NullPointerException: at org.mozilla.gecko.ScreenshotHandler.access$100(ScreenshotHandler.java)

Categories

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

18 Branch
ARM
Android
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Firefox 18

People

(Reporter: scoobidiver, Assigned: kats)

References

Details

(Keywords: crash, regression, Whiteboard: [native-crash])

Crash Data

Attachments

(1 file)

There's one crash in 18.0a1/20120927: bp-b130ad77-8d2a-4625-ac53-b678f2120927.

java.lang.NullPointerException
	at org.mozilla.gecko.ScreenshotHandler.access$100(ScreenshotHandler.java:27)
	at org.mozilla.gecko.ScreenshotHandler$1.run(ScreenshotHandler.java:309)
	at android.os.Handler.handleCallback(Handler.java:608)
	at android.os.Handler.dispatchMessage(Handler.java:92)
	at android.os.Looper.loop(Looper.java:156)
	at org.mozilla.gecko.util.GeckoBackgroundThread.run(GeckoBackgroundThread.java:31)

More reports at:
https://crash-stats.mozilla.com/report/list?signature=java.lang.NullPointerException%3A+at+org.mozilla.gecko.ScreenshotHandler.access%24100%28ScreenshotHandler.java%29
Attached patch PatchSplinter Review
Straightforward fix; this code is run on a separate thread and the ScreenshotHandler could get wiped out in the middle if a low-memory situation occurs.
Assignee: nobody → bugmail.mozilla
Attachment #665433 - Flags: review?(cpeterson)
Version: Trunk → Firefox 18
Comment on attachment 665433 [details] [diff] [review]
Patch

Review of attachment 665433 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Attachment #665433 - Flags: review?(cpeterson) → review+
https://hg.mozilla.org/mozilla-central/rev/563605815bc3
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
You need to log in before you can comment on or make changes to this bug.